Opened 3 years ago

Closed 3 years ago

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

WW_vector volkov.ivan (lab_11)

Reported by: Ivan Volkov Owned by: Дмитрий Лапшин (lapshin)
Component: WW_vector Version:
Keywords: Cc:

Description


Change History (6)

comment:1 Changed 3 years ago by Дмитрий Лапшин (lapshin)

Owner: changed from Дмитрий Лапшин (lapshin) to Ivan Volkov
Type: ожидается проверкаожидаются исправления

operator = не возвращает ссылку. Зачем-то всю копию возвращаешь.

Иииии работает. Только копирующее присваивание что-то сломало, видимо ты капасити старое оставил.

А зачем тебе исключения?))

А так даже сложные тесты проходит. Восторг!

8.

comment:2 Changed 3 years ago by Ivan Volkov

Исключения убрал, ссылку в operator = поставил.

Почему копирующее присваивание не работало, так и не понял, capacity вроде там в reserve перепроставится. Возможно, проблема была именно с отсутствием & (если например конструктор копий T выводил что-нибудь куда-нибудь)

Last edited 3 years ago by Ivan Volkov (previous) (diff)

comment:3 Changed 3 years ago by Ivan Volkov

Owner: changed from Ivan Volkov to Дмитрий Лапшин (lapshin)
Type: ожидаются исправленияожидается проверка

comment:4 Changed 3 years ago by Дмитрий Лапшин (lapshin)

Owner: changed from Дмитрий Лапшин (lapshin) to Ivan Volkov
Type: ожидается проверкаожидаются исправления

Твой operator= не переживёт a = a.

Вопрос что он делает с капасити всё ещё актуален. Он у тебя конечно круче тем что переиспользует по возможности память, но получается что между конструктором копий и копирующим присваиванием эффекты разные.

9.

comment:5 Changed 3 years ago by Ivan Volkov

Owner: changed from Ivan Volkov to Дмитрий Лапшин (lapshin)
Type: ожидаются исправленияожидается проверка

Да, переписал operator = чтобы работал так же как конструктор копий. Проверку на a=a поставил, хотя теперь это вроде избыточно.

comment:6 Changed 3 years ago by Дмитрий Лапшин (lapshin)

Resolution: задача сдана
Status: assignedclosed

Да, нормась.

Note: See TracTickets for help on using tickets.