ДЗ#1. Матрицы.

[basharin.egor]

Прошу проверить домашнее задание 1.

[Vladimir Rutsky]

Замечания:

1. Здесь происходит "срез" класса исключения (см "Propagating a polymorphic exception" ​здесь):

     } catch (const std::exception& e) {
         //throw std::runtime_error("Error while reading file from file");
         throw e;
     }

Используйте просто throw;.

2. На мой взгляд указание беззнаковости при инициализации переменной нулём излишне: nrow_ = 0u;. nrow_ = 0; не должно приводить ни к каким предупреждением.

3. Зачем вы возвращаете константную матрицу в const Matrix operator+(const Matrix& rhs) const;? Данный метод вернёт временный неименованный объект (prvalue) и мне не понятно зачем добавлять ему константность.

4. Код выделения памяти для буфера матрицы дублируется в Init() и operator>>.

[basharin.egor]

""" 3. Зачем вы возвращаете константную матрицу в const Matrix operator+(const Matrix& rhs) const;? Данный метод вернёт временный неименованный объект (prvalue) и мне не понятно зачем добавлять ему константность. """
Добавляю константность для того, чтобы нельзя было сделать так:
(A + B) = C;

[Vladimir Rutsky]

В (A + B) = C нет ничего страшного (по крайней мере не должно быть).
Конкретно (A + B) = C не особо осмысленно, но, например следующий код:

Matrix & m = A + B;
...
m += ...
m /= ...

для избежания лишнего копирования матрицы --- осмысленен*.

*С новым стандартом и современными компиляторами часто в данном случае можно обойтись без использования ссылки на матрицу, но не всегда.

[Vladimir Rutsky]

Замечания:

1. В этой конструкции нет смысла:

     try
     {
         // ...
     } catch (...) {
         throw;
     }

Вы с тем же успехом можете просто не ловить исключение здесь.

[basharin.egor]

Исправлено

[Vladimir Rutsky]

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