ha1

[Vladimir Rutsky]

Замечания:

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

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

void set_correct(bool correct, std::string const & error_msg);

3. В данном конструкторе:

Matrix(char * const file_name);

вы принимаете неизменяемый указатель на изменяемые данные.

Следует использовать:

Matrix(char const * file_name);

или

Matrix(char const * const file_name);

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

5. Matrix::operator =, Matrix::operator +=, Matrix::operator *= не возвращают значения.

6. Случай, когда не удаётся открыть первую матрицу, обрабатывается некорректно.

Также, если матрицу не удалось открыть вы не инициализируете поля, которые потом читаете в деструкторе Matrix, что приводит к падению программы.

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

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

[Vladimir Rutsky]

Milestone ha1-deadline deleted