Opened 7 years ago

Closed 7 years ago

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

Домашнее задание "Операции над матрицами"

Reported by: tereschenko.sergej Owned by: rutsky,grabovoy.philipp
Priority: проверка Milestone: ha1-deadline
Component: HA#1 matrices Version:
Keywords: Cc: tereshchenko.sergei@…

Description


Change History (1)

comment:1 Changed 7 years ago by Vladimir Rutsky

Resolution: задача НЕ сдана
Status: newclosed

Замечания:

  1. В вашем решений присутствуют ошибки из списка из условия задачи "часто возникающие ошибки":
  1. Заголовочный файл должен быть самодостаточным. Сейчас вы используете в нём, например, size_t и std::string, но соответствующие заголовочные файлы не включаете.
  1. В matrices.cpp стоит включить matrices.hpp.
  1. Проверка существования файла с помощью check_file малоосмысленна: после того, как вы проверили, что файл существует и его можно открыть, но перед тем, как вы открыли его второй раз на чтение, файл может быть удалён. В любом случае во время фактического чтения файла надо проверять ошибки открытия/чтения.
  1. Вы не освобождаете память от результирующей матрицы.
  1. Вы не освобождаете память, выделенную при инициализации Matrix left_matrix = {0, 0, create_array2d(1,1)};, при запуске ./matrices 5.txt.
  1. Память в right_matrix не освобождается при запуске ./matrices 5.txt --add 2.txt.
  1. std::string по умолчанию инициализируется пустой строкой, явно делать это не стоит: std::string operation = ""; (используйте std::string operation;).

Решение не зачтено.

Note: See TracTickets for help on using tickets.