WW #14

[Дмитрий Свиридкин]

При перемножении вылетает не пойманный bad_alloc. Также при вылете пойманного bad_alloc исходная матрица ломается. А вроде как не должна. Можно сделать так, чтоб ничего не ломалось.

          bool flag = true;
            try {
                value = regs[index].get(row, col);
            } catch(MatrixException& e) {
                std::cout << e.what() << std::endl;
                flag = false;
            }
            if (flag)
                std::cout << value << "\n";

Зачем здесь flag?

возможно, стоит использовать более конкретный базовый класс для исключений, чем std::exception.
std::out_of_range_error, std::runtime_error... ​https://en.cppreference.com/w/cpp/error

---

7

[Дмитрий Свиридкин]

Чтение номера регистра стоит выделить в отдельную функцию. И завернуть все тело цикла обработки команд в один общий блок try, чтобы не ловить одно и то же в каждой ветке.

bad_alloc при чтении должен пролетать дальше методов класса Matrix, либо вызывать другое исключение -- иначе вызывающий код не узнает об ошибке.

Matrix::Matrix(int rows_, int cols_) : rows(rows_), cols(cols_) {}

После вызова этого конструктора, нельзя использовать метод get -- будет UB.
Публичный конаструктор должен создавать готовый к использованию объект. Либо не быть публичным.

---

8