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

miredirex / oop-course

Объектно-ориентированное программирование

0 stars 0 forks source link

Open alexey-malov opened 4 years ago

alexey-malov commented 4 years ago

[x] Если сделали свойство determinant частью состояния матрицы, то оно должно быть всегда актуальным. Сейчас после изменения элементов матрицы будет возвращаться ранее вычисленный детерминант. Справедливо как для 2x2, так и 3x3

alexey-malov commented 4 years ago

alexey-malov commented 4 years ago

[x] Убрать конструктор, выполняющий чтение из потока. Сделать свободную функцию для чтения матрицы
[x] Убрать вывод матрицы в поток - сделать свободную функцию

alexey-malov commented 4 years ago

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

alexey-malov commented 4 years ago

[x] Метод должен быть константным, так как он не меняет наблюдаемое состояние объекта

alexey-malov commented 4 years ago

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

alexey-malov commented 4 years ago

[ ] Для вывода с указанной точностью воспользуйтес fixed + setprecision (см. пример у меня в репозитории oop - пример numericToStringConversion

alexey-malov commented 4 years ago

[ ] Выделите функцию чтения файла по его имени. Не нужно в main заниматься открытием файлов