task1

[Luchko Alexander]

Please review this solution. Task1 matrices!

[Vladimir Rutsky]

Замечания:

1. Решение необходимо оформить выделив функции/классы для работы с матрицами в файлы matrices.cpp/matrices.hpp.

2. Передавайте объекты константными, если не собираетесь их изменять внутри функции:

bool createMatrix(char const * file)
...
int add(Matrix const & y)

3. Используйте одинаковый стиль отступов, сейчас у вас где-то отступ 4 проблела, где-то 2.

4. Не вся выделенная память освобождается. Например, в main() вы создаёте Matrix() в куче, и не удаляете его.

[Vladimir Rutsky]

Замечания:

1. При обработке ошибок вы не всегда освобождаете выделенную память. Например, если файл не удалось открыть.

[Luchko Alexander]

Здравствуйте
Всё исправил, но не понял зачем освобождать память перед выходом из программы, ведь она автоматически освободится. Во всех местах там идёт затем return -1;

[Vladimir Rutsky]

Replying to luchko.aleksandr:

Здравствуйте
Всё исправил, но не понял зачем освобождать память перед выходом из программы, ведь она автоматически освободится. Во всех местах там идёт затем return -1;

Вся явно выделенная память, должна быть явно освобождена --- это "хороший тон" при написании программ.

У такого поведения есть ряд преимуществ: всегда можно извлечь замкнутый кусок функциональности и превратить его в библиотеку или использовать в другой программе; при использовании инструментов проверки утечки памяти не нужно показывать инструменту, где настоящие утечки, а где утечки, которые автор посчитал допустимыми; при необходимости в дальнейшем модифицировать программу не придётся разбираться какая память будет освобождена в ходе работы программы, а какая память не освобождается до конца работы программы, чтобы случайно дважды не освободить одну и ту же память.

В общем случае в дальнейшем вам нужно будет использовать умные указатели, контейнеры, RAII, и другие средства, которые значительно облегчают работу с памятью (и которые запрещены к использованию в первой программе).

[Vladimir Rutsky]

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

[Vladimir Rutsky]

Milestone ha1-deadline deleted