Opened 3 years ago
Closed 3 years ago
#350 closed ожидаются исправления (задача сдана)
WW #6
Reported by: | movsin.marat | Owned by: | Святослав Власов |
---|---|---|---|
Component: | WW_vector | Version: | 3.0 |
Keywords: | Cc: |
Description
Change History (6)
comment:1 Changed 3 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
comment:2 Changed 3 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 1.0 → 2.0 |
comment:3 Changed 3 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
- Дважды выделять память в
MyVector::reserve
и дважды копировать элементы туда-сюда избыточно, можно этого избежать - В операторе присваивания и конструкторе копирования ты копируешь лишнюю память, в которой нет элементов
В остальном хорошо.
comment:4 Changed 3 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 2.0 → 3.0 |
comment:5 Changed 3 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
Не вижу коммита в svn с исправлениями
comment:6 Changed 3 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
if (&oth == this) return *this;
Это затрудняет читаемость. Переноси тело условия или цикла на отдельную строку.MyVector::reserve
ты выделяешь динамическую память на стеке. Во-первых, это не является стандартом, не все компиляторы это поддерживают, а, во-вторых, если вектор будет достаточно большой, ты получишь Stack Overflow.MyVector::
, можно просто вызвать метод по имени.const
у методов не расставлены.