Opened 3 years ago

Closed 3 years ago

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

WW#12

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

Description


Change History (5)

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

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

16 из 20 тестов прошли
Упали тесты на умножение матриц.

  1. MatrixException проще отнаследовать от std::logic_error, он уже умеет хранить сообщение и тебе даже не придется ничего переопределять.
  2. Строки, если их нет намерения менять, передаются в функции по константным ссылкам
  3. std::bad_alloc лучше обернуть в MatrixExeption, чтобы матрица выбрасывала наружу только исключения своего типа и чтобы интерпретатор тоже его ловил.
  4. Вместо присваивания векторов лучше использовать метод std::vector::swap, это поможет избежать лишнего копирования данных
  5. Код кидающий исключение если встретился eof прям просится в отдельный метод

6/10 пока.

comment:2 Changed 3 years ago by Daniil Lyubaev

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

Пофиксил.

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

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

Зачем делать функцию, проверяющую eof у потока методом матрицы? Ей же не нужна матрица. Сделай её обычной функцией в matrix.cpp и оберни в анонимный неймспейс, чтобы наружу не торчала.

В остальном всё отлично, 9.9/10

comment:4 Changed 3 years ago by Daniil Lyubaev

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.