WW_14

[Sokolov Viacheslav]

10 explicit MatrixException?(const char* val);
noexcept

12 int convertReg(string s)
13 {
14 return s[1] - '0';
15 }

assert?

34 std::ifstream is;
35 is.exceptions(std::ios::eofbit | std::ios::failbit | std::ios::badbit);
зачем здесь std::ios::eofbit ?

39 catch (const std::ifstream::failure& e)
40 {

выбивается по стилю

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

75 swap(_matrix, _tmp_matrix);
ADL здесь намеренно используется?

checkDimensions - это проверка инварианта => assert

[Jura Khudyakov]

ADL - намеренно. Я посчитал, что вызывать swap-функцию класса лучше, чем std::swap.

[Sokolov Viacheslav]

79 if (cin.eof())
80 break;

не понял, зачем это

[Sokolov Viacheslav]

116 if (!ok)

117 assert(false

"error: _matrix dimensions != _rows and _cols");

-> assert(ok && "...")?

[Sokolov Viacheslav]

35 is.exceptions(std::ios::eofbit | std::ios::failbit | std::ios::badbit);

кажется, здесь eofbit все же нужен, но вопрос остается в силе: по какой причине?