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

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/e87485bd7ed048259d46114aa3a6d710b4596128/Lw%201/lab1.3%20invert/lab1.3%20invert/main.cpp#L15

• [x] Лучше дать имя ParseMatrixFileName

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/e87485bd7ed048259d46114aa3a6d710b4596128/Lw%201/lab1.3%20invert/lab1.3%20invert/main.cpp#L72

• [x] Задайте матрицу при помощи std::array<std::array<double, 3>, 3>.

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/e87485bd7ed048259d46114aa3a6d710b4596128/Lw%201/lab1.3%20invert/lab1.3%20invert/main.cpp#L82

• [x] статическая переменная существует в единственном экземпляре. Повторный вызов выполнит модификацию уже имеющейся переменной

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/e87485bd7ed048259d46114aa3a6d710b4596128/Lw%201/lab1.3%20invert/lab1.3%20invert/main.cpp#L84-L88

• [x] Лучше не бросать указатели на char. Посмотрите лекцию по исключениям

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/e87485bd7ed048259d46114aa3a6d710b4596128/Lw%201/lab1.3%20invert/lab1.3%20invert/main.cpp#L27

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

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/e87485bd7ed048259d46114aa3a6d710b4596128/Lw%201/lab1.3%20invert/lab1.3%20invert/main.cpp#L46-L49

• [x] Зачем в случае ошибки менять ее флаг на противоположный

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/e87485bd7ed048259d46114aa3a6d710b4596128/Lw%201/lab1.3%20invert/lab1.3%20invert/main.cpp#L53-L57

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

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/e87485bd7ed048259d46114aa3a6d710b4596128/Lw%201/lab1.3%20invert/lab1.3%20invert/main.cpp#L61-L64

• [x] Если в конце последней строки матрицы будет перевод строки, программа сообщит о пустом файле

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/e87485bd7ed048259d46114aa3a6d710b4596128/Lw%201/lab1.3%20invert/lab1.3%20invert/main.cpp#L149

• [x] Функция возвращает указатель на локальную статическую переменную. На надо так

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/e87485bd7ed048259d46114aa3a6d710b4596128/Lw%201/lab1.3%20invert/lab1.3%20invert/main.cpp#L160-L166

• [x] Выделить цикл в функцию

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/ce80b047ed5d138c564dad1dcdbd7f7d3e4b32bb/Lw%201/lab1.3%20invert/lab1.3%20invert/main.cpp#L27

• [ ] Зачем эта переменная, если сразу после ее установки в true происходит выход?

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/ce80b047ed5d138c564dad1dcdbd7f7d3e4b32bb/Lw%201/lab1.3%20invert/lab1.3%20invert/main.cpp#L60-L64

• [ ] Если будет 2 строки, напишет, что файл пустой

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/ce80b047ed5d138c564dad1dcdbd7f7d3e4b32bb/Lw%201/lab1.3%20invert/lab1.3%20invert/main.cpp#L117

• [ ] Что за x и y?

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/ce80b047ed5d138c564dad1dcdbd7f7d3e4b32bb/Lw%201/lab1.3%20invert/lab1.3%20invert/main.cpp#L136-L137

• [ ] Что такое mark?

• [ ] Почему newMatrix?

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/ce80b047ed5d138c564dad1dcdbd7f7d3e4b32bb/Lw%201/lab1.3%20invert/lab1.3%20invert/main.cpp#L171

• [ ] Что за coefficient? Назовите invDeterminant

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/ce80b047ed5d138c564dad1dcdbd7f7d3e4b32bb/Lw%201/lab1.3%20invert/lab1.3%20invert/main.cpp#L151

• [ ] Принимайте double по значению, а не по ссылке

[alexey-malov]

https://github.com/MaxMartST/OOP/blob/ce80b047ed5d138c564dad1dcdbd7f7d3e4b32bb/Lw%201/lab1.3%20invert/lab1.3%20invert/main.cpp#L207

• [ ] Лучше вместо invalid использовать термин singular (вырожденный)

[alexey-malov]

• [ ] ReadStream выглядит переусложнённой. Упростить код