HW1

[Vladimir Rutsky]

Замечания:

1. В std::ifstream режим открытия std::ifstream::in по умолчанию, нет необходимости его явно указывать ("i f stream" это "input file stream"):

std::ifstream file(argv[i], std::ifstream::in);

2. Страж включения должен оборачивать весь файл:

#include <fstream>
#include <cstddef>

#ifndef MATRIX
#define  MATRIX

...

3. Файлы с функциями работы с матрицами должны называться matrices.hpp и matrices.cpp.

4. При умножении матриц вы не обнуляете результирующую матрицу. Используйте ​value initialization чтобы инициализировать матрицу нулями при создании.

5. Вместо 0/1 для action используйте enum.

6. Нет необходимости явно закрывать файл:

{
    std::ifstream file(argv[i], std::ifstream::in);

    ....

    file.close();
}

он будет закрыт, когда переменная file будет уничтожена при выходе из её области видимости.

7. В случае возникновения ошибки (например, при вызове ./matrices A_3x7.txt --mult B_7x13.txt --add B_7x13.txt) в строке 89:

  delete my_matrix;

происходит обращение к уничтоженному объекту.

[Vladimir Rutsky]

Решение зачтено.

[Vladimir Rutsky]

Milestone ha1-milestone2 deleted