Opened 3 years ago

Closed 3 years ago

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

WW#08

Reported by: Daniil Lyubaev Owned by: Святослав Власов
Component: WW shared_ptr Version: 2.0
Keywords: Cc:

Description


Change History (4)

comment:1 Changed 3 years ago by Daniil Lyubaev

Version: 1.0

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

Type: ожидается проверкаожидаются исправления
  1. delete this; -- вот этого не надо делать. Деструктор вызывается перед освобождением памяти под объект, поэтому еще раз это делать уже в деструкторе не надо. Это плохой паттерн.
  2. Постарайся убрать явный вызов деструкторов, постарайся отрефакторить код, чтобы это не понадобилось.

В остальном всё хорошо -- 9/10.

comment:3 Changed 3 years ago by Daniil Lyubaev

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

Вроде всё поправил и ещё чуть-чуть код укоротил, используя reset()

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

Resolution: задача сдана
Status: assignedclosed

Всё круто, 10/10

Вот тут можно тернарным оператором сделать код покороче:

    if (obj != nullptr)
        storage_ = new Storage(obj);
    else
        storage_ = nullptr;
storage_ = obj ? new Storage(obj) : nullptr;
Note: See TracTickets for help on using tickets.