Home assignment #1 Anastasia Lavrenko

[Anastasia Lavrenko]

Check my solution, please.

[Vladimir Rutsky]

Замечания:

1. Объявление create_matrix_array() можно убрать из matrices.hpp, это можно считать деталью реализации.

2. В вашем решений присутствуют ошибки из списка из условия задачи "часто возникающие ошибки":
   • 9 (print()),
   • 12 (например, при ./matrices 5.txt --a 2.txt вы не освобождаете ).

3. Вы не освобождаете память от выводимой на экран результирующей матрицы.

4. *(data_[row] + column) эквивалентно data_[row][column], но последнее лучше читается.

[Anastasia Lavrenko]

Спасибо, поправила, можно снова смотреть.

[Vladimir Rutsky]

Замечания:

1. Не стоит использовать ​альтернативные написания логических операторов (and): if (matrix_left != nullptr and matrix_right != nullptr), используйте &&: if (matrix_left != nullptr && matrix_right != nullptr) --- это общепринято в C++, тогда как альтернативные написания были введены для кодировок или клавиатур, в которых нет таких символов, как &.

2. Память освобождается не во всех ошибочных ситуациях, например при вызове ./matrices 5.txt --a 2.txt происходит утечка памяти.

Исправьте, пожалуйста, в ближайшее время.

[Anastasia Lavrenko]

1. Спасибо, это по инерции с питона =)

Поправила, можно проверять

[Vladimir Rutsky]

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