ДЗ №1. Матрицы

[kaminskij.vasilij]

Здравствуйте!

Прошу проверить моё решение.

С уважением,
Каминский Василий

[Vladimir Rutsky]

Замечания:

1. В вашем решений присутствуют ошибки из списка из условия задачи "часто возникающие ошибки":

• № 9 (matrix::print()),
• 18.c) (std::ifstream::failure),
• 13 (цикл для освобождения памяти реализован два раза в operator= и ~matrix).

2. В matrix::mult происходит утечка памяти: вы создаёте matrix в динамической памяти (matrix* tmp = new matrix(N_, m.M_);), но никогда не освобождаете память, выделенную для объекта matrix (т.е. где-то должен быть аналог delete tmp).

3. В operator= вы делаете value_ = matrix.value_, что приводит к тому, что один и тот же буфер используется в нескольких экземплярах класса matrix, что скорее всего приведёт к двойному освобождению памяти буфера.

[kaminskij.vasilij]

Здравствуйте!

Внёс изменения в соответствии с вашими замечаниями. Прошу проверить.

С уважением,
Каминский Василий.

[Vladimir Rutsky]

Василий, вы закоммитили исправления в Subversion? Последний раз ваше решение менялось в Subversion 20 сентября.

[kaminskij.vasilij]

Приношу свои извинения. Действительно, не закоммитил исправления. Теперь всё должно быть хорошо.

С уважением,
Каминский Василий

[Vladimir Rutsky]

Замечания:

1. В аргументах matrix::allocation(int N_, int M_) стоит использовать тип size_t, вместо int. Плюс аргументы стоит назвать без подчеркивания.

Решение зачтено (уверен, что вы бы исправили замечание выше в оставшуюся попытку сдачи).