Opened 4 years ago

Closed 4 years ago

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

WW shared ptr

Reported by: Vavilov Mark Owned by: Sokolov Viacheslav
Component: WW shared_ptr Version: 2.0
Keywords: Cc:

Description


Change History (3)

comment:1 Changed 4 years ago by Sokolov Viacheslav

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

Сейчас не выполнено требование

Умный указатель, указывающий в nullptr, не должен требовать дополнительной памяти.

delete this конечно валидная конструкция
​https://isocpp.org/wiki/faq/freestore-mgmt#delete-this
но ни один программист не обрадуется увидеть это в чьем-либо коде.
в целом это тот редкий случай, когда можно, но лучше все-таки так не делать
можно сделать приватную функцию в shared_ptr (но можно и оставить, как есть, я не буду снижать баллы за это)

у класса есть понятные инварианты, которые легко проверить с помощью assert-ов

comment:2 Changed 4 years ago by Vavilov Mark

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

comment:3 Changed 4 years ago by Sokolov Viacheslav

Resolution: → задача сдана
Status: assigned → closed

61 Matrix* shared_ptr::operator->() const {
лучше было бы поставить assert(not isNull()) (как работает вызов x-> ?)

проверки инвариантов не увидел

Note: See TracTickets for help on using tickets.