Opened 3 years ago

Closed 3 years ago

#602 closed ожидается проверка (задача сдана)

WW_exceptions Kiselev Igor lab_12

Reported by: kiselev.igor Owned by: Антон Филатов
Component: WW_exceptions Version: 2.0
Keywords: Cc:

Description


Change History (3)

comment:1 Changed 3 years ago by Антон Филатов

Type: ожидается проверкаожидаются исправления

Падает тест на загрузку "плохой" матрицы:

если сделать load корректной матрицы, а потом в тот же регистр попробовать положить матрицу, у которой заявленная размерность больше, чем подано чисел, то регистр не откатывается к старому нормальному состоянию.

Стиль:

  • пробелы в конце строки выпилите
  • вы в странном месте кидаете bad_alloc - его надо кидать там, где он может выпасть. То есть в конструкторе. В вашем случае рискуете утекать по памяти, если bad_alloc возникнет на какой-нибудь итерации в середине цикла в конструктора.
  • не хватает слова noexcept в методах, которые ничего не кидают
  • если у вас есть констркутор без параметров для матрицы, то почему бы не положить load в матрицу?

7/10

comment:2 Changed 3 years ago by kiselev.igor

Type: ожидаются исправленияожидается проверка
Version: 1.02.0

comment:3 Changed 3 years ago by Антон Филатов

Resolution: задача сдана
Status: assignedclosed

Что-то у вас теперь прямо все тесты валятся на неравильный формат входных данных.
Даже на простой тест

load $0 smoke_2x3
print $0
exit

При этом

cat smoke_2x3
2 3
1 2 3
4 5 6

и

hexdump -C smoke_2x3
00000000  32 20 33 0a 31 20 32 20  33 0a 34 20 35 20 36 0a  |2 3.1 2 3.4 5 6.|
00000010

По стилю сейчас почти всё нравится, разве что в функции mul у вас между new и catch(bad_alloc) почти 20 строк кода

Но из-за упавших всех тестов могу только оставить старую оценку 7/10

Note: See TracTickets for help on using tickets.