Opened 4 years ago
Closed 4 years ago
#552 closed ожидается проверка (задача сдана)
WW #9
Reported by: | Maria Chizhova | Owned by: | Дмитрий Свиридкин |
---|---|---|---|
Component: | WW shared_ptr | Version: | 3.0 |
Keywords: | Cc: |
Description
Change History (5)
comment:1 Changed 4 years ago by
Owner: | changed from Дмитрий Свиридкин to Maria Chizhova |
---|---|
Type: | ожидается проверка → ожидаются исправления |
comment:2 Changed 4 years ago by
Owner: | changed from Maria Chizhova to Дмитрий Свиридкин |
---|---|
Type: | ожидаются исправления → ожидается проверка |
Version: | 1.0 → 2.0 |
comment:3 Changed 4 years ago by
Owner: | changed from Дмитрий Свиридкин to Maria Chizhova |
---|---|
Type: | ожидается проверка → ожидаются исправления |
Что-то поломалось. После присваивания потекла память.
А swap в операторе присваивания зачем убрали? У него же как раз сигнатура, чтоб вы swap сделали!
comment:4 Changed 4 years ago by
Owner: | changed from Maria Chizhova to Дмитрий Свиридкин |
---|---|
Type: | ожидаются исправления → ожидается проверка |
Version: | 2.0 → 3.0 |
comment:5 Changed 4 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
Теперь c reset что-то не так. Тоже память потекла
Что-то у вас код из второй попытки очень сильно похож на такой же но другой... А вот не было бы этой ерудны с const, может быть, и не заметно было. Проверяйте друг друга хотя бы и не делайте так больше.
Note: See
TracTickets for help on using
tickets.
Ради одного swap целый namespace тянуть -- как-то не хорошо.
Зачем вы еще раз копируете пришедший в оператор присваивания указатель? У вас там и так копия. Поэтому в исходниках задания нет const.
reset тогда можно реализовать с помощью уже определенного оператора присваивания.
Вы везде используете this, а в трех местах не используете. Как-то непоследовательно получается.
7