Home assignment #1

[Nicholas Kapralov]

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

Посмотрите, пожалуйста, мой текущий вариант решения. Smoke test без valgrind'а был пройден успешно, но при включении проверки на утечки памяти все обрушивается на первом тесте, и я не понимаю, почему так происходит.

С уважением,
Николай.

[Vladimir Rutsky]

Здравствуйте, Николай!

Замечания:

1. Страж включения должен оборачивать весь файл:

#include <string>

#ifndef MATRICES
#define MATRICES

2. Передавайте неизменяемые объекты по константной ссылке:

double ** read (string const & filename, int * rows, int * cols) {

3. Используйте тип size_t для размеров и индексов.

4. Вынесите код выделения и освобождения памяти для матриц в отдельные функции.

5. Вы не всегда освобождаете память, выделенную для матрицы (об этом в частности говорят сообщения valgrind). Например, вы не освобождаете память для answer в конце программы (это не единственное место с утечкой памяти). При обработке ошибок также вся выделенная память должна быть освобождена.

[Nicholas Kapralov]

Исправил указанные Вами замечания.
Также разобрался с утечками памяти, теперь smoke test с valgrind'ом тоже проходится успешно.
Посмотрите, пожалуйста, ещё раз, когда у Вас будет возможность.

[Vladimir Rutsky]

Замечания:

1. В mult стоило использовать destroy и create, вместо перевыделения памяти вручную.

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

[Vladimir Rutsky]

Milestone ha1-deadline deleted