matrices

[Vladimir Rutsky]

Замечания:

1. В заголовочном файле отсутствует страж включения.

2. Не используйте Си-заголовочные файлы: #include<stdlib.h>, используйте их C++ версии: #include <cstdlib> (если они вам необходимы, конечно, в данном случае я сомневаюсь, что вам нужен cstdlib).

3. Не указывайте this-> там, где это не требуется:

matrix::~matrix() {
	for (size_t i = 0; i < this->rows; ++i)
		delete[] this->elements[i];
	delete[] this->elements;
}

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

void print_matrix(matrix const * a);

matrix* read_matrix(char const * file_name);

5. При обработке ошибок наблюдаются утечки памяти. Например, при возникновении ошибки Unknown operation вы выходите из программы, не освобождая память от уже загруженных матриц.

Вы сдали работу на несколько часов позже milestone 2, плюс вносили существенные исправления после сдачи. Формально ваша попытка сдачи должна относиться к последней попытке сдачи данной задачи, и из-за указанных выше замечаний не должна быть зачтена.
Постарайтесь исправить указанные замечания в ближайшее время.

P.S. Я проверил вашу работу вчера, но не опубликовал результаты проверки. Я вижу, что вы с тех пор внесли ещё изменения --- их я не смотрел.

[Vladimir Rutsky]

Замечания:

1. При возникновении ошибки открытия не первой матрицы вы не освобождаете память от первой матрицы (matrices 1.txt --add xxx.txt).

Это критическая ошибка, но вы успешно исправили аналогичную ошибку ранее, поэтому, если вы корректно исправите данную ошибку в ближайшее время, я зачту вам задачу.

[Семён]

Исправил.

[Vladimir Rutsky]

Ещё небольшое замечание на будущее: в конце сообщений выводите перевод строки.

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

[Семён]

Спасибо!

[Vladimir Rutsky]

Milestone ha1-deadline deleted