WW_exceptions milshin.maksim lab_12

[Дмитрий Лапшин (lapshin)]

Формат выыыыывода! Кроме кучи лишних сообщений ещё пробелы в матрицах и ошибки, пожалуйста, в cout.

Утечки памяти. Я схожу вижу opeartor * возвращающий сырой указатель на кучу (зачем лишний раз на кучу?!). Я бы ещё не перепроверял в add что будет ошибка, пусть += сам же проверит.

Ну и bad_alloc вылететь может запросто, не ловишь.

5, но легко исправимо.

[milshin.maksim]

• поправил формат вывода матриц(убрал лишние пробелы в конце строк) и сделал вывод ошибок в std::cout
• перенес выброс исключений при проверки корректности матриц в приватные методы: operator+= и operator* соответсвенно

Но мне кажется, что вы смотрели не последнюю мою версию. Я там к сожалению, сделал много посылок до открытия тикета (потому что не могу запустить валгринд со своего мака). И в последней версии я ловил bad_alloc, а оператор operator * возвращал std::unique_ptr<Matrix>, поэтому там валгринд вроде не жаловался. Но если много посылок это плохо, и это вам неудобно, то я постараюсь так больше не делать.

[Дмитрий Лапшин (lapshin)]

Всё норм, но осталось

std::unique_ptr<Matrix> matrix(new Matrix(rows, cols)); 

Зачем?

Matrix matrix(rows, cols);

9.