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: ожидается проверкаожидаются исправления
  1. Ворнинги компилятора мы не любим, их нужно исправлять.
  2. Вот так писать не надо: if (&oth == this) return *this; Это затрудняет читаемость. Переноси тело условия или цикла на отдельную строку.
  3. В MyVector::reserve ты выделяешь динамическую память на стеке. Во-первых, это не является стандартом, не все компиляторы это поддерживают, а, во-вторых, если вектор будет достаточно большой, ты получишь Stack Overflow.
  4. Когда ты вызываешь методы того же класса, не обязательно указывать имя класса MyVector::, можно просто вызвать метод по имени.
  5. const у методов не расставлены.

comment:2 Changed 3 years ago by movsin.marat

Type: ожидаются исправленияожидается проверка
Version: 1.02.0

comment:3 Changed 3 years ago by Святослав Власов

Type: ожидается проверкаожидаются исправления
  1. Дважды выделять память в MyVector::reserve и дважды копировать элементы туда-сюда избыточно, можно этого избежать
  2. В операторе присваивания и конструкторе копирования ты копируешь лишнюю память, в которой нет элементов

В остальном хорошо.

comment:4 Changed 3 years ago by movsin.marat

Type: ожидаются исправленияожидается проверка
Version: 2.03.0

comment:5 Changed 3 years ago by Святослав Власов

Type: ожидается проверкаожидаются исправления

Не вижу коммита в svn с исправлениями

comment:6 Changed 3 years ago by Святослав Власов

Resolution: задача сдана
Status: assignedclosed
Note: See TracTickets for help on using tickets.