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

alexey-malov commented 4 years ago

1>C:\teaching\oop-ips\lobanova\oop\Invert\Invert.cpp(156,37): warning C4267: 'initializing': conversion from 'size_t' to 'int', possible loss of data

[ ] Программа должна компилироваться без предупреждений компилятора во всех конфигурациях и платформах.

alexey-malov commented 4 years ago

[ ] Дать функции более удачное имя (например ReadMatrixFromFile)
[ ] Read from file
[ ] Незачем передавать размер (все равно если он превысит размер матрицы, произойдёт выход за границы массива)

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

[ ] Рекомендую использовать std::array. Тогда сможете возвращать матрицу по значению и хранить ее внутри optional

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

[ ] InvertMatrix3x3 выглядит сложно. Выделить функции с понятными именами. Подсказка - как правило, цикл реализует некий алгоритм, которому можно дать имя

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

[ ] Тесты должны сравнивать, что вывела программа с тем, что должно получиться. Должны быть файлы с ожидаемыми значениями. Используйте перенаправление вывода для того, чтобы программа вместо консоли вывела итоговую матрицу в файл, который потом тесты сравнивали бы с эталоном

venera13 / oop