WW_exceptions bukreev.roman

[Дмитрий Лапшин (lapshin)]

    while (true) {
        std::cin >> command;

Мне уже лень повторять где здесь беда. А ещё я бы переменную утащил внутрь цикла.

            int r;
            std::string name;
            std::cin >> r >> name;

Кого читаем?.. Инт?.. А условие прочитать?)

Зачем писать так:

            std::unique_ptr<Matrix> temp;
            std::size_t rows, cols;
            if (!(fin >> rows >> cols)) {
                std::cout << "LOAD: invalid file format.\n";
                continue;
            }
            temp = std::make_unique<Matrix>(Matrix(rows, cols));

Когда можно:

            std::size_t rows, cols;
            if (!(fin >> rows >> cols)) {
                std::cout << "LOAD: invalid file format.\n";
                continue;
            }
            Matrix temp(rows, cols); // или вариацию на тему

? И нет кучи, и переменная объявляется где надо.

Зачем чтение прямо в мейне, это же прям просится в matrix.cpp.

А ловить исключения кто будет?..

    std::string get() const;

Жууутко неэффективно, особенно для исключений. Кстати, где what()?

Какой медленный +=... А const мы не пишем особо?) Почему-то прибавление к самому себе привело к ошибке. Кстати, почему-то формат сообщения об ошибках в разных местах разный, но ты пока не начнёшь их в мейне ловить не заметишь особо. А зачем ты в мейне перепроверяшь то, что проверит метод, кинув исключение?

Чтение ещё и лажает что сбивает другие тесты... Короче проблем хватает и их много. Пока 2.

Кажется что хватает мест, которые недопонимаешь, и из-за этого делаешь не до конца осозновая проблемы. Можем устроить интерактивную сессию на следующей неделе, чтобы разобраться.

[Дмитрий Лапшин (lapshin)]

Формат вывода матриц ну близко не по заданию)

И не обработал когда произведение матриц было бы больше чем памяти на компьютере. Полезно ловить std::bad_alloc кроме MatrixException (я бы кстати ловил в одном месте, но это так).

А так годно, 8.