Change History (2)

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

Type: ожидается проверкаожидаются исправления
  1. Служебные методы нужно убрать из интерфейса в приватную область видимости
  2. shared_ptr::decrease лучше сделать так, чтобы он оставлял объект в консистентном состоянии (т.е. чтобы storage_ не указывал на освобожденную память.
  3. Метод shared_ptr::isNull можно сократить до одной строчки: return !storage_;. Кстати, с помощью тернарного оператора метод shared_ptr::create тоже можно сократить до одной строчки: storage_ = obj ? new Storage(obj) : nullptr. shared_ptr::ptr аналогично сокращается.
  4. Проверку на isNull логичнее перенести в decrease, т.к. она у тебя стоит перед каждым вызовом decrease

8/10

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

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