Матрицы

[Alexander]

Здравствуйте, Владимир.

Прошу проверить задание 1, выложено к первому майлстоуну.
​http://trac.compscicenter.ru/svn/cpp16/bzikadze.aleksandr/

Спасибо.

С уважением,
Александр

[Vladimir Rutsky]

Замечания:

1. Файлы с функциями работы с матрицами должны называться matrices.hpp и matrices.cpp.

2. При умножении матриц вы не обнуляете результирующую матрицу. Используйте ​value initialization чтобы инициализировать матрицу нулями при создании.

3. Объявление parseSentence не должно быть в matrices.hpp. matrices.hpp должен объявлять функции, классы и переменные, определённые в matrices.cpp.

4. В случае возникновения ошибки вы не освобождаете память в std::string* arguments.

[Alexander]

Replying to rutsky:

Замечания:

1. Файлы с функциями работы с матрицами должны называться matrices.hpp и matrices.cpp.

2. При умножении матриц вы не обнуляете результирующую матрицу. Используйте ​value initialization чтобы инициализировать матрицу нулями при создании.

3. Объявление parseSentence не должно быть в matrices.hpp. matrices.hpp должен объявлять функции, классы и переменные, определённые в matrices.cpp.

4. В случае возникновения ошибки вы не освобождаете память в std::string* arguments.

Здравствуйте, Владимир.

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

С уважением,
Алекандр

[Vladimir Rutsky]

Александр,

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

вы правы насчет того, что операционная система освободит всю память процесса, при его завершении.

В данной программе не освобождение ресурсов или не выполнение деинициализации скорее всего даже никак не повредит (хотя, например, нет гарантий, что std::cout зафлашит вывод в консоль), но в общем случае в программах или библиотеках на C++ использование exit() для завершения программы недопустимо.

[Vladimir Rutsky]

В данной программе не освобождение ресурсов или не выполнение деинициализации скорее всего даже никак не повредит (хотя, например, нет гарантий, что std::cout зафлашит вывод в консоль), но в общем случае в программах или библиотеках на C++ использование exit() для завершения программы недопустимо.

Мой комментарий про exit() не относится к вашей работе --- вы не используете exit() и правильно делаете, я перепутал вашу работу с другой.

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

[Vladimir Rutsky]

Milestone ha1-milestone2 deleted