Opened 4 years ago
Closed 4 years ago
#539 closed ожидается проверка (задача сдана)
WW_shared_ptr
Reported by: | tarasov.denis | Owned by: | Sokolov Viacheslav |
---|---|---|---|
Component: | WW shared_ptr | Version: | 3.0 |
Keywords: | Cc: |
Description
Change History (6)
comment:1 Changed 4 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
comment:2 Changed 4 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 1.0 → 2.0 |
comment:3 Changed 4 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
Инвариантов больше.
comment:4 Changed 4 years ago by
Version: | 2.0 → 3.0 |
---|
comment:5 Changed 4 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
comment:6 Changed 4 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
19 void shared_ptr::Storage::incr() 20 { 21 assert(ref_count_ != 0); 22 ref_count_++; 23 } 24 25 void shared_ptr::Storage::decr() 26 { 27 assert(ref_count_ > 0); 28 ref_count_--; 29 }
Разное количество отступов. В первом случае лучше бы проверить, что ref_count_ > 0.
Note: See
TracTickets for help on using
tickets.
Сейчас не выполнено требование
У класса есть понятные инварианты, которые стоит проверить с помощью assert-ов
стоит вынести в функцию