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

[alexey-malov]

double GetMinor2x2Matrix(Matrix3x3& matrix, int col, int row)

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

[alexey-malov]

double GetСofactor2x2Matrix(double& minor, int minorF, int minorS)
{
    //домножать на -1 в зависимости от нечетности minorF + minorS
    double сofactor = pow(-1, minorF + minorS) * minor;
    return сofactor;
}

• [x] нет смысла передавать double по ссылке. можно передать по значению
• [x] minorF и minorS - плохие имена
• [x] GetСofactor2x2Matrix - ПолучитьМатрицуАлгебраических дополнений. 2*2 тут никак не фигурирует.
• [x] Вообще лучше выпилить и менять знак переменной с -1 на +1 и обратно вычитая ее из нуля

[alexey-malov]

• [x] Выделить из main функцию инвертирования матрицы

[alexey-malov]

double FindCofactor(double minor, int minorOne, int minorTwo)
{
    double сofactor;
    ((minorOne + minorTwo) % 2 == 0) ? сofactor = minor : сofactor = -minor;

    return сofactor;
}

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

  return (условие) ? minor : -minor;

[alexey-malov]

• [ ] Выделить из main функцию загрузки матрицы из файла по имени:

if (ReadMatrix(argv[1], inputMatrix)
{
    Matrix3x3 invertedMatrix;
    if (InvertMatrix(inputMatrix, invertedMatrix))
    {
          PrintMatrix(invertedMatrix;)
    }
    else
    {
    ...
    }
}
else
{
    ...
}

[alexey-malov]

3>c:\teaching\2018\oop\grebneva\oop\lr_01\invert\invert.cpp(85): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data
3>c:\teaching\2018\oop\grebneva\oop\lr_01\invert\invert.cpp(86): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data

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

[alexey-malov]

    const string nameInputFile = argv[1];
    Matrix3x3 matrix;
    if (ReadMatrix(matrix, nameInputFile))

• [ ] nameInputFile -> inputFileName

[alexey-malov]

void GetСofactorOfMatrix(const Matrix3x3& matrix, Matrix3x3& algebraicComplementsMatrix)
{
    for (size_t row = 0; row < SIZE_OF_MATRIX; ++row)
    {
        for (size_t col = 0; col < SIZE_OF_MATRIX; ++col)
        {
            double minor = GetMinorOfMatrix(matrix, row, col);
            algebraicComplementsMatrix[row][col] = FindCofactor(minor, row + 1, col + 1);
        }
    }
}

• [ ] Использовать единую терминологию в имени функции и её выходных аргументов

[alexey-malov]

• [ ] MatrixTranspose -> TransposeMatrix

[oMystique]

• [x] Добавить тест, в котором детерминант = 0

[oMystique]

• [x] В функции GetNextLine следует использовать тернарный оператор

[oMystique]

• [x] Нужны нормальные названия для аргументов функции FindCofactor

[oMystique]

• [x] Функционал по работе с матрицами нужно вынести в отдельную пару файлов .cpp/.h