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

[alexey-malov]

https://github.com/PetrovIliya/OOP/blob/759ce4b4189ab7d53e1bbc12349b68819f9af02f/lw1/invert/invert.cpp#L162

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

[alexey-malov]

https://github.com/PetrovIliya/OOP/blob/759ce4b4189ab7d53e1bbc12349b68819f9af02f/lw1/invert/invert.cpp#L17-L25

• [ ] Здесь не надо парсить строки вручну. Используйте stof или stod. Тем более, что эта функция не сможет разобрать дробные числа и выявить ошибки вроде 3-8

[alexey-malov]

https://github.com/PetrovIliya/OOP/blob/759ce4b4189ab7d53e1bbc12349b68819f9af02f/lw1/invert/invert.cpp#L52-L55

• [ ] Неудачное имя для аргумента

[alexey-malov]

https://github.com/PetrovIliya/OOP/blob/759ce4b4189ab7d53e1bbc12349b68819f9af02f/lw1/invert/invert.cpp#L60-L69

• [ ] Почему нельзя индексы строки и столбца начать с 0. Тогда не придётся вычитать единицу

[alexey-malov]

https://github.com/PetrovIliya/OOP/blob/759ce4b4189ab7d53e1bbc12349b68819f9af02f/lw1/invert/invert.cpp#L112

• [ ] i != j - избыточное условие

[alexey-malov]

https://github.com/PetrovIliya/OOP/blob/759ce4b4189ab7d53e1bbc12349b68819f9af02f/lw1/invert/invert.cpp#L207-L213

• [ ] Нулевой детерминант - не повод бросать исключение. Можно было бы возвращать bool, а в вызывающей функции проверять возвращённое значение. Либо можно было бы не менять исходную матрицу, а возвращать optional<Matrix3x3>

[[nodiscard]] bool Invert3x3Matrix(Matrix3x3 &matrix)
{
    float det = GetDeterminant3x3(matrix);
    if (det == 0)
    {
        return false;
        //throw std::logic_error("Determinant is zero. Inverted matrix do not exist");
    }
    TransposeMatrix(matrix);
    matrix = GetMinorMatrix(matrix);
    MultiplyMatrixByNumber(matrix, 1 / det);
    return true;
}

[alexey-malov]

• [ ] программа не может инвертировать эту простую матрицу (использует табуляции)

input.txt