Проверка hw_1

[Vladimir Rutsky]

Замечания:

1. Папка с решением первой задачи должна называться hw_1.

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

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

void write_error(string const & message) {

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

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

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

7. Память для массива указателей в clear_matrix() вы освобождаете с помощью delete, а нужно delete[].

8. Используйте &&= вместо &= для логических переменных: &= это побитовое "и-присвоить". В C++ bool приводится к 0 и 1 и обратно, поэтому это не должно быть ошибкой (в отличие от того же выражения в Си), но в &&= использовать логичней.

9. Если не удалось открыть файл матрицы вы в дальнейшем пытаетесь освободить память неинициализированного класса Matrix.

[golovanov.egor]

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

[Vladimir Rutsky]

Замечания:

1. Принимайте аргументы, которые не должны меняться, константными:

bool read_matrix(Matrix & matrix, char const * file) {
...

bool read_from(char const * file);

2. Используйте nullptr вместо NULL.

3. Папка с решением первой задачи должна называться hw_1 - это всё ещё не исправлено.

4. В Matrix::read_from используйте Matrix::create_matrix.

Ваше решение вполне рабочее, исправьте оставшиеся замечания и я зачту задачу.

[golovanov.egor]

Специально не стал использовать create_matrix в прошлый раз, чтобы не делать двойную работу. (инициализировать сначала нулями, а потом перезаписывать)
В итоге все поправил, кроме названия папки.
У меня она уже с первой попытки называлась hw_1.
Вот такой путь к ней ​http://trac.compscicenter.ru/svn/cpp16/golovanov.egor/hw_1/

[Vladimir Rutsky]

Я опечатался --- папка с решением должна называться ha1 (как указано в условии задачи), а у вас называется hw_1.

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

[Vladimir Rutsky]

Milestone ha1-deadline deleted