Opened 4 years ago

Closed 4 years ago

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

WW #9

Reported by: Solovyev Gleb Owned by: Sokolov Viacheslav
Component: WW shared_ptr Version: 3.0
Keywords: Cc:

Description


Change History (6)

comment:1 Changed 4 years ago by Sokolov Viacheslav

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

assert(ref_count_);

лучше: assert(ref_count_ > 0);

delete this конечно валидная конструкция

​​https://isocpp.org/wiki/faq/freestore-mgmt#delete-this

но ни один программист не обрадуется увидеть это в чьем-либо коде.
в целом это тот редкий случай, когда можно, но лучше все-таки так не делать
можно сделать приватную функцию в shared_ptr (но можно и оставить, как есть, я не буду снижать баллы за это)

comment:2 Changed 4 years ago by Solovyev Gleb

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

Исправил.

comment:3 Changed 4 years ago by Sokolov Viacheslav

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

Есть еще как минимум один важный инвариант, который стоить проверить.

Кода будет меньше, если в shared_ptr::decr не предполагать, что storage_ != nullptr

comment:4 Changed 4 years ago by Solovyev Gleb

Last edited 4 years ago by Solovyev Gleb (previous) (diff)

comment:5 Changed 4 years ago by Solovyev Gleb

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

Исправил.

Last edited 4 years ago by Solovyev Gleb (previous) (diff)

comment:6 Changed 4 years ago by Sokolov Viacheslav

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