WW #7

[Святослав Власов]

Выводить usage при запуске CLI не нужно, это сломало все тесты)

1. Валгринд изрыгает тонны ошибок. Проверяй им и ищи что не так.
2. Подумай как можно выделять память под матрицу меньшим количеством вызовов new, ты даже знаешь как это делать
3. Зачем ты в print делаешь приведение f к типу FILE*, когда он и так имеет тип FILE*?
4. Реализация присваивающих операторов через неприсваивающие приведет к лишним выделениям памяти и копированиям элементов. Так делать не нужно. Лучше наоборот, реализовать неприсваивающие операторы через присваивающие.
5. Хочется увидеть ассерты вместо сегфолтов, если размеры матриц вдруг не совпадают.

[Святослав Власов]

UPD:

6. operator!= работает неверно!

[Святослав Власов]

1. Аллоцируешь слишком много памяти. Аргументом оператора new T[...] выступает количество элементов типа T, память под которые должна быть выделена, а не количество байт, как у функции malloc.
2. В операторах += и -= ты по прежнему выделяешь память под еще одну матрицу. Это избыточно. Ты можешь просто добавлять элементы матрицы-аргумента к текущей матрицы, зачем новую-то выделять? Новую обязательно выделять только в операторе *=, потому что в этом случае результат -- это матрица другого размера.

8.5/10