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

[alexey-malov]

• [x] Выделите функцию bool ParseCommandLine(int argc, char* argv[], string& mat1FileName, string& mat2FileName) (альтернатива - вместо ссылок на строки с именами файлов принимать ссылку на структуру, содержащую имена файлов).

• [x] Сделать функцию bool LoadMatrixFromFile(const string& fileName, Matrix& mat). Открытие файлов должно выполняться внутри неё

Это упростит функцию main

[alexey-malov]

typedef vector<vector<float>>  Matrix;

• [x] Вместо вектора векторов можно использовать:
• структуру

  struct Matrix3x3
  {
  float items[3][3];
  };

• std::array<std::array<float, 3>, 3>, тоже назвав её matrix.

Вектор плох тем, что в нём каждая строка может хранить своё количество элементов

[alexey-malov]

void InitMatrix(Matrix &matrix)
{
    vector<float> currentRow(N, 0);
    for (int row = 0; row < N; row++)
    {
        matrix.push_back(currentRow);
    }
}

• [x] А что если массив не был пустым? И вообще, LoadMatrix мог бы внутри себя самостоятельно выполнить зачистку, если в ней вообще есть необходимость

[alexey-malov]

const int N = 3;
const string FILE_1 = "file1";
const string FILE_2 = "file2";

• [x] Имя N для глобальной переменной слишком короткое.
• [x] Можно обойтись без FILE_1 и FILE_2