Замечания по программе Invert

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

[x] Лучше объявите Matrix3x3 как std::array<std::array<double, 3>, 3>. Тогда можно будет принимать и возвращать матрицу по значению, а также завернуть в optional

alexey-malov commented 4 years ago

[x] Функция main не должна вычислять определитель. Вместо этого она должна вызывать функцию InvertMatrix(matrix), результат которой позволил бы функции main определить, есть существует ли обратная матрица

alexey-malov commented 4 years ago

[x] Элементы матрицы переписываются дважды
[x] Функция не выполняет нахождение обратной матрицы. Она транспонирует матрицу и делит ее на некоторый коэффициент.

alexey-malov commented 4 years ago

[ ] Лучше переименовать в InvertMatrix
[ ] Лучше, чтобы функция не изменяла входную матрицу. Можно возвращать optional<Matrix3x3> (nullopt в случае вырожденной исходной матрицы), либо принимать обратную матрицу в виде out-параметра и возвращать bool
[ ] Цикл лучше выделить в отдельную функцию
[ ] Вывод сообщений в cout отсюда убрать.

alexey-malov commented 4 years ago

[ ] Лучше возвращать матрицу миноров как возвращаемое значение функции, а не через out-параметр

IrinaZagaynova / oop