Opened 7 years ago

Closed 7 years ago

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

Home assignment #1

Reported by: valiullin.aleksandr Owned by: rutsky,grabovoy.philipp
Priority: проверка Milestone: ha1-deadline
Component: HA#1 matrices Version: 1.0
Keywords: Cc:

Description


Change History (3)

comment:1 Changed 7 years ago by Vladimir Rutsky

Milestone: ha1-milestone2ha1-deadline
Type: ожидается проверкаожидаются исправления

Замечания:

  1. В вашем решений присутствуют ошибки из списка из условия задачи "часто возникающие ошибки":
    • 10: Matrix(std::string path);, Matrix matrix_sum(Matrix m1, Matrix m2);,
    • 13,
    • 24.
  1. Не используйте альтернативные написания операторов (not, and, or): if (not matrix_file.good()), используйте здесь if (!matrix_file.good()) (а вместо and и or используйте && и || соответственно). Альтернативные написания операторов существуют для поддержки систем, в кодировках которых или на клавиатурах не таких символов, как !, |, &, в современном C++ их использование не принято.
  1. Нет необходимости явно приводить char const * к std::string здесь:
         std::string operation(argv[i]);
         if (operation == std::string("--add"))

Для std::string определено сравнение с char const * (или у std::string есть неявный конструктор от char const *).

comment:2 Changed 7 years ago by valiullin.aleksandr

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

Внесены исправления по замечаниям.

comment:3 Changed 7 years ago by Vladimir Rutsky

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

Замечания:

  1. Лучше использовать стиль написания приведения типов C++: вместо (std::string) argv[i] используйте std::string(argv[i]).

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

Note: See TracTickets for help on using tickets.