WW_vector Mittcev lab_11

[Антон Филатов]

автотесты оказались не готовы к тому, чтобы вручную подключать ещё и "_impl.h" Кажется, вам надо самим об этом позаботиться.

Падает тест на копирование, reserve и resize (попробуйте разные кратные и не кратные 2 длины то поувеличивать, то поуменьшать). Эти тесты довольно большие, я не очень хочу их сбда копипастить. Попишите юнит тесты, если сами не найдёте ошибку - пишите мне, я скину код тестов.

Стиль:

• не хватает своп трика в операторе=
• может (1 << i) * capacity_ - это в отдельную функцию?
• кажется, что в delete_array и в pop_back есть копипаста

6,5/10

[mitcev.nikita]

Пожалуйста, проверьте, я забыл 2 дня назад статус поменять

[Антон Филатов]

Упал тест на resize.
Если взять вектор из 5 элементов, заполнить их чем-то ненулевым, а потом сделать resize до 10 и проверить от 6 до 10 на vec[i] == T() (где T = int), то такой ассерт упадёт.
Стиль:

• а представьте как было бы красивы не делать отдельную функцию call_destructor, а в delete_array сделать while(size) {pop_back;}
• зря вы в reserve пользуетесь memcpy, мы же на паре это обсуждали. У вас же есть замечательные конструкторы копирования, которые в купе с placement new будут очень хорошо себя вести, и по скорости не уступать memcpy

9.3/10