WW_exceptions vyguzov.artem lab_12

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

На загрузке плохих файлов валгринд!

И bad_alloc тебя убил наповал, кажись.

Я бы не разделял исключения как сделал ты, в чём разница между Add и Mul? Оба про то, что размерами не сошлись.

Ну и множественное наследование от bad_alloc это клёво, но зачем?

explicit у конструкторов без аргументов это странно.

        int *tmp;
        try {
            tmp = new int [rows_ * cols_];
            ...
        } catch (std::bad_alloc &e) {
            delete [] tmp;
            throw BadAllocMatrixException();
        }

А если первый new кинет что ты собираешься delete[]-ать? мусорный указатель?

7.

[Артем Выгузов]

Что нашёл - поправил.

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

Не думал what() перегрузить у исключения?

Мне кажется в конструкторе кучу кода можно вынести из try {}

Зачем закрывать файл руками, его закроет RAII? Точнее причина есть но ты её не знаешь и ̳̅зн̘̽а̨͈̜̟̦̙̑͐̑͋̿͝т͔͇̻͇̑͂̓͝ь̢̼̣̮̳͉̀͌̎̌͒̃͆͜ ̧̞͆̊п̺̟̤̪̈̊̀͘о͎̝̯͎̱̬̳͆̀̈́̑̂̔̏͢͞к̨̨̛̰̻̳͕̪̜̠̽̀̓̀̓͗̓̒̔͜а̡̣̺̲̰̋̒̾̒̋͋͟͢͡ ̡̢̻̫̥̺͙̘̝̼͌͐̌͋͆̉͆̄̒̇̀͜н̡̢̛̙͙̠̤̰̤͚̦͈̼̭̤̦̪͓͍̲͗̽̄͌͒̈́̊̂̂̽͗̈́͑͂̈̏̀̋͢͞е̧̨̡͙͉̟̼̳̺̝̫̙̺̲͚͍͙̺͇̪̒̍͂̆̈́͛́͑͑̈́̊̓̀̽͌̓͛̐̕͠ ̧̧̢̨̢̨̡̛̮̜̠̳͖̲͉̙̻̱͓̤̇̊̑̓͒͑̀̆̈̅͋͂̑̎̅̔̕̕͝ͅх̡̨̢͔̙͇͖͚̻̦̦͔͈̟̮͍̥͙̃̓̍̈́͒̒̒̔̀̊̎̀̀̃̈́͆̃̚̚͟͢͡о̨̛͓̻̙̺̥͎̞͖̭̳͍͇̜̪͔͍̋̑̾̌̅͋͊͛͗̿̋̍́̑̎͗̑͢͜͡͡ͅч̧̙̣͓̦̱̱͉̼̪̤̥͍̝̪͔̜̱͌̀͂̽̽́̊̍̃́̈͆̏̈̊̄̂̚̚͢͜͡е͔̜̜̩̺̘͚̻̖͙͕̲̪͈̺̜̰͔͛̐́̽̀̏̃̊̄͒͐͑͗͂͆̓̇͘͘̚͟ͅш̨̛̘̖̯͓̭̺̹̦͕͚̭͙̖̲̼͇̤̣̬̈́͐͂͆̅͋́́̽̐̈̿̽̋̓͐́͞͞ь̧̢̡͔̖̯͕̟͕̰̻͇̙̖͕̥͉͉͔͈͊͊̀̓͑͌̃̃́̀̂͑̾͐̉́̕͘͝͝.

8.

[Артем Выгузов]

Исправил.

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

Да, норм.