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

[alexey-malov]

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

[alexey-malov]

void GetMatrixFromStream(ifstream& matrixFileStream, double(&sourceMatrix)[3][3]);
void GetInvertMatrix(double(&sourceMatrix)[3][3], double(&invertMatrix)[3][3]);
void GetMinorMatrix(double(&sourceMatrix)[3][3], double(&minorMatrix)[3][3]);
void TransposeMatrix(double(&matrixToTranspose)[3][3]);
void InvertMatrix(double(&matrixToInvert)[3][3], double determinant);
void WriteMatrixToConsole(double(&matrix)[3][3]);

• [x] заведи typedef для обозначения матрицы

typedef double matrix[3][3];

[alexey-malov]

• [x] упростить чтение матрицы

[alexey-malov]

void InvertMatrix(double(&matrixToTranspose)[3][3], double determinant)
{
    double revertDeterminant = 1 / determinant;
    for (int i = 0; i < 3; ++i)
    {
        for (int j = 0; j < 3; ++j)
        {
            matrixToTranspose[i][j] *= revertDeterminant;
        }
    }
}

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

[alexey-malov]

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

[alexey-malov]

bool IsOnlyDigitString(const std::string &str)
{
    return all_of(str.begin(), str.end(), [](char i) { return isdigit(i) || i == '-' || i == '.'; });
}

• [x] функция не нужна

[alexey-malov]

    if (abs(determinant) <= DBL_EPSILON) // сравнение с нулем
    {
        return false;
    }

    if (abs(x - y) <= DBL_EPSILON) // сравнение с другим числом
    {
        return false;
    }

[alexey-malov]

    double (& sm)[3][3] = sourceMatrix;
    double determinant = (sm[0][0] * sm[1][1] * sm[2][2]) + (sm[0][1] * sm[1][2] * sm[2][0]) + (sm[0][2] * sm[1][0] * sm[2][1]) - 
        (sm[0][2] * sm[1][1] * sm[2][0]) - (sm[0][0] * sm[2][1] * sm[1][2]) - (sm[1][0] * sm[0][1] * sm[2][2]);

• [x] нахождение определителя выделить в отдельную функцию

• [ ] разбить по строкам для лучшей читаемости