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

[alexey-malov]

https://github.com/Curpiq/oop_labs/blob/327d52561429f14093e24d7d0615aebc8c5ea27a/lab01/invert/invert/invert.cpp#L131-L142

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

https://github.com/Curpiq/oop_labs/blob/ae2067cfcd37311708784e984721d1e7967103d3/lab01/invert/invert/invert.cpp#L132-L143 Уже лучше, но надо дожать

• [ ] Неудачные имена функции DivisionMatrixByDet и имя аргумента det. Как эта функция определяет, что мы передаем ей именно определитель? Имя функции, как правило, должно обозначать действие, то есть быть глаголом. Division (деление) - имя существительное.

[alexey-malov]

• [x] У вас есть тип Matrix3x3. Используйте его. Нечего туда-сюда массивы гонять

[alexey-malov]

https://github.com/Curpiq/oop_labs/blob/327d52561429f14093e24d7d0615aebc8c5ea27a/lab01/invert/invert/invert.cpp#L116

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

[alexey-malov]

https://github.com/Curpiq/oop_labs/blob/327d52561429f14093e24d7d0615aebc8c5ea27a/lab01/invert/invert/invert.cpp#L144

• [x] Не нужно аргумент оборачивать в optional, а также передавать по значению

[alexey-malov]

https://github.com/Curpiq/oop_labs/blob/327d52561429f14093e24d7d0615aebc8c5ea27a/lab01/invert/invert/invert.cpp#L157

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

[alexey-malov]

https://github.com/Curpiq/oop_labs/blob/327d52561429f14093e24d7d0615aebc8c5ea27a/lab01/invert/invert/invert.cpp#L116

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

[alexey-malov]

https://github.com/Curpiq/oop_labs/blob/327d52561429f14093e24d7d0615aebc8c5ea27a/lab01/invert/invert/invert.cpp#L15-L18

• [x] Следует конкретизировать, что за fileName

https://github.com/Curpiq/oop_labs/blob/ae2067cfcd37311708784e984721d1e7967103d3/lab01/invert/invert/invert.cpp#L15-L18

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

[alexey-malov]

https://github.com/Curpiq/oop_labs/blob/327d52561429f14093e24d7d0615aebc8c5ea27a/lab01/invert/invert/invert.cpp#L38 https://github.com/Curpiq/oop_labs/blob/ae2067cfcd37311708784e984721d1e7967103d3/lab01/invert/invert/invert.cpp#L32

• [ ] Незачем аргумент оборачивать в optional.
• [x] Для чего аргумент передаётся по ссылке, допускающей модификацию аргумента?

[alexey-malov]

• [ ] Код ReadMatrix по прежнему сложен

[alexey-malov]

https://github.com/Curpiq/oop_labs/blob/327d52561429f14093e24d7d0615aebc8c5ea27a/lab01/invert/invert/invert.cpp#L13

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

[alexey-malov]

• [x] Тесты проверяют лишь код возврата программы. В случае, если программа вернула успешный код возврата, они должны сравнивать вывод программы с ожидаемым - вдруг вместо обратной матрицы программа выводит левый текст