Change History (2)

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

Type: ожидается проверкаожидаются исправления
  1. Твой метод shared_ptr::isNull можно сократить до одной строчки: return !storage_;
  2. То же самое про функцию shared_ptr::init, она тоже сокращается с помощью тернарного оператора в одну строчку: storage_ = obj ? new Storage(obj) : nullptr;

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

Дальше идут уже существенные замечания по стилю:

  1. В деструкторе, конструкторе копирования и операторе присваивания собрано много копи-пасты, попробуй отрефакторить код, чтобы от неё избавиться
  2. Деструктор лучше напрямую не вызывать, когда в этом нет необходимости, здесь её нет, можно написать reset без этого (и без копи-пасты)

К корректности замечаний нет. 9/10

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

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