Opened 4 years ago
Closed 4 years ago
#856 closed ожидается проверка (задача сдана)
WW #12 lopatin.mikhail
| Reported by: | lopatin.mikhail | Owned by: | Дмитрий Лапшин (lapshin) |
|---|---|---|---|
| Component: | WW_vector | Version: | 1.0 |
| Keywords: | Cc: |
Description
Change History (3)
comment:1 Changed 4 years ago by
| Owner: | changed from Дмитрий Лапшин (lapshin) to lopatin.mikhail |
|---|---|
| Summary: | WW #12 → WW #12 lopatin.mikhail |
| Type: | ожидается проверка → ожидаются исправления |
comment:2 Changed 4 years ago by
| Owner: | changed from lopatin.mikhail to Дмитрий Лапшин (lapshin) |
|---|---|
| Type: | ожидаются исправления → ожидается проверка |
так до конца и не понял что конкретно было не так
comment:3 Changed 4 years ago by
| Resolution: | → задача сдана |
|---|---|
| Status: | assigned → closed |
Список инициализации не в том порядке, на -Werror новый компилятор за такое в утиль кидает. Вокруг всяких равенств куча ерунды.
Конструктор копирования делает кучу лишних действий, в том числе требует конструктора по умолчанию + копирующее присваивание вместо одного конструктора. resize не всё умеют. operator = теряет объекты, которые надо уничтожить.
На исключениях что-то не так (ты ловишь std::exception, никто не обещал такой щедроты). Более того, ты его проглатываешь, а не прокидываешь! Операторы обращения по индексу стрёмные. Циклы уничтожения в обработке исключения не с того индекса, delete[] не от того типа. resize/reserve имеют слишком слабую гарантию.
Пробелы. Код дублируется (расчёт степени двойки). Делать delete[] array нельзя, если память не была выделена парным new T[].
4.
Тесты не компилируются: нет перемещающего присваивания, clean не работает на классах без конструктора по умолчанию.