Change History (2)

comment:1 Changed 4 years ago by Артур Гулецкий (huletski)

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

Глобально:

  • реализуйте хранение элементов Matrix в динамической памяти, а не в векторах (для тренировки);

Замечания:

  • MatrixException? наследуйте от std::exception (what нужно будет перекрыть)/std::logic_error (what ооtb), почему what не const?
  • matrix_exception.h:3. лишный include, не хватает подключения std::string; matrix.h лишний #include <fstream>. Проверьте код на недостаток/избыток подлюченных заголовков;
  • matrix.h. const не выставлены у всех методов;
  • Matrix::Matrix(std::string) - лучше принимать std::istream (проще тестировать, более общий интeрфейс) -> напрашивается перенос кода в operator>>; идея с guard (101) оригинальная, но лучше проверять флаги ошибок/ловить исключения в случае ошибок чтения;
  • Matrix::print. лучше перегрузить op<<;
  • Matrix::get_cell. Неочевидно почему в качестве аргументов не std::size_t и почему возвращается не int -> нужен коммент, поясняющий причину;
  • (опционально) по вектору лучше ходить переменой типа size_type, определенной в std::vector.

Баллы: 6

Last edited 4 years ago by Артур Гулецкий (huletski) (previous) (diff)

comment:2 Changed 4 years ago by Артур Гулецкий (huletski)

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

Дедлайн.

Note: See TracTickets for help on using tickets.