Opened 3 years ago

Closed 3 years ago

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

HW #12

Reported by: Georgiy Angeni Owned by: Святослав Власов
Component: WW_exceptions Version: 3.0
Keywords: Cc:

Description


Change History (5)

comment:1 Changed 3 years ago by Святослав Власов

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

Все тесты прошли.

  1. MatrixException лучше отнаследовать от std::logic_error, тогда вообще ничего не придется переопределять.
  2. В конструкторе матрицы от файла можно воспользоваться swap-trick и избавить себя от мороки с освобождением памяти.
  3. То же самое касается умножения
  4. Лучше чтобы все методы матрицы бросали только исключение MatrixException наверх, а вызывающий код уже выводил сообщение.

В остальном всё хорошо, 8/10

comment:2 Changed 3 years ago by Georgiy Angeni

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

comment:3 Changed 3 years ago by Святослав Власов

Type: ожидается проверка → ожидаются исправления
  1. Зачем ты делаешь пустые блоки catch(...) { throw; } ? Они не делают ничего, можно их просто убрать.
  2. Код, проверяющий состояние потока и бросающий эксепшн прям просится в отдельную функцию.

9.5/10

comment:4 Changed 3 years ago by Georgiy Angeni

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

comment:5 Changed 3 years ago by Святослав Власов

Resolution: → задача сдана
Status: assigned → closed

10/10

Note: See TracTickets for help on using tickets.