WW #14

[Sokolov Viacheslav]

Makefile:
стоит разбить компиляцию main.cpp и компоновку lab_14

По какой причине какие-то определения находятся в заголовочном файле?

MatrixException::check - из названия совершенно непонятно, что делает этот метод

Почему struct, а не class?

final, explicit

138 std::pair<int, int> get_dimensions() const;
предпочтительнее использовать struct, а не pair, потому что меньше вероятность совершить ошибку

36 void read(std::string path);

const std::string& ?

41 void elem(int row, int col) const; можно просто at

53 catch (const std::runtime_error& e) {
54 std::cout << e.what() << std::endl;
55 }

возможно, лучше ловить const std::exception&

всегда выставляйте .exceptions на failbit, badbit для стандартных потоков ввода/вывода, если не считаете их ожидаемыми

8 char sp;

sp это space?

39 int reg = read_reg(), row, col;

это нечитаемо. Если уж декларируйте несколько переменных разом (зачем?), то не смешивайте объявления с определениями

36 *this = tmp;

на самом деле здесь должен быть std::move, но мы пока об этом не знаем

59 tmp >> in;

лучше сделать in >> tmp, что будет согласовано с обычным использованием

[Sokolov Viacheslav]

по стандарту int может быть 16-битным, поэтому лучше везде использовать типы фиксированного размера

[samoylov.viktor]

Все замечания исправлены, кроме std::move, так как про него мы ещё не знаем

[Sokolov Viacheslav]

16 explicit MatrixException?(ERRORS err);
стоит пометить как noexcept этот и другие методы

17 void what() const;
все еще не очень понятно)
можно const char* what() const noexcept, как в стандартной библиотеке, можно назвать как-нибудь вроде printToStdout