WW11

[morozov.nikita]

FFF

[Святослав Власов]

Тесты прошли.

Замечания:

1. Не хватает неконстантного оператора [] значит через него нельзя менять элементы
2. Зачем в операторе присваивания выделять временный объект на куче? На стеке будет проще и чуть-чуть поэффективнее
3. Конструктор копий вектора будет течь, если конструктор копирования T швырнет какой-нибудь эксепшен
4. Аналогично с конструктором от size_t
5. Не нужно подменять исключение брошенное классом T на свое собственное. Так вызывающий код теряет возможность понять что же случилось и корректно на это среагировать.
6. В push_back у тебя происходит лишнее копирование объекта -- аргумент принимается по значению и потом копируется в буфер. Например, в таком коде произойдет два вызова конструктора копий, а хватило бы одного:

     Element e;
     v.push_back(e);
   

7. В main намешал пробелов и табуляций -- у меня всё съехало :-/

В остальном код очень хороший, ты молодец!
8/10

[Святослав Власов]

1. push_back должен принимать объект по константной ссылке, а не по обычной. Ты ведь не меняешь объект внутри.
2. С гарантиями исключений всё по прежнему не очень хорошо. reserve,resize,push_back не дают строгой гарантии.

9/10

[Святослав Власов]

С гарантиями исключений стало лучше, но не до конца -- на push_back ругается валгринд, а еще теперь упал тест на ресайз.

Еще у тебя бессмысленная копи-паста между resize и push_back. Зачем ты в resize рассматриваешь два случая когда хватает capacity и когда не хватает, если в push_back ты делаешь ровно то же самое?

9/10