Opened 3 years ago

Closed 3 years ago

#619 closed ожидается проверка (задача сдана)

WW#12

Reported by: Terova Valeriia Owned by: Святослав Власов
Component: WW_exceptions Version: 2.0
Keywords: Cc:

Description


Change History (7)

comment:1 Changed 3 years ago by Святослав Власов

Type: ожидается проверкаожидаются исправления
  1. Все тесты прошли кроме одного. Пропускаешь bad_alloc и не ловишь.
  2. Лучше эксепшн наследовать от std::logic_error, тогда вообще ничего переопределять не придется
  3. Этот код matrix = std::move(result); вызовет обычный оператор копирующего присваивания, т.к. оператор перемещающего присваивания у тебя не определен. И вы его еще не проходили. Зато у тебя есть замечатеьлный метод swap, который здесь прекрасно подойдет, который определени и о котором ты всё знаешь.

В остальном всё очень клёво, ты молодец! 9/10

comment:2 Changed 3 years ago by Terova Valeriia

Type: ожидаются исправленияожидается проверка
Version: 1.02.0

comment:3 Changed 3 years ago by Святослав Власов

Type: ожидается проверкаожидаются исправления

Упал тест на нехватку памяти.
Подсказка -- когда не хватает памяти, то bad_alloc летит прямо из new, nullptr оттуда не возвращается.

9/10

comment:4 in reply to:  3 Changed 3 years ago by Terova Valeriia

Replying to Святослав Власов:

Упал тест на нехватку памяти.
Подсказка -- когда не хватает памяти, то bad_alloc летит прямо из new, nullptr оттуда не возвращается.

9/10

Да, правда! На всякий случай уточню: обернуть это в try {} catch(std::bad_alloc&) {throw ...} должно все пофиксить, да же?)

comment:5 Changed 3 years ago by Terova Valeriia

Type: ожидаются исправленияожидается проверка

Ну, я надеюсь, что теперь работает как надо :)

comment:6 Changed 3 years ago by Святослав Власов

Задавай все вопросы в телеграм. Здесь я читаю комментарии только когда проверяю работы.

Да, теперь всё работает как надо, молодец.
10/10

comment:7 Changed 3 years ago by Святослав Власов

Resolution: задача сдана
Status: assignedclosed
Note: See TracTickets for help on using tickets.