Opened 3 years ago
Closed 3 years ago
#579 closed ожидается проверка (задача сдана)
HW #11
Reported by: | movsin.marat | Owned by: | Святослав Власов |
---|---|---|---|
Component: | WW_vector | Version: | 2.0 |
Keywords: | Cc: |
Description
Change History (3)
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
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
Базовые тесты прошли. Тесты на гарантии исключений попадали.
Много копипасты повсюду, тот же оператор присваивания прекрасно через swap trick пишется двумя строчками или конструктор от числа через resize.
8/10
Note: See
TracTickets for help on using
tickets.
Даже компиляция твоего собственного кода упала -- как так? Пришлось чинить :-/
Затем упала компиляция тестов :( Потому что твои файлы не инклюдать заголовочники, от которых они зависят. Например, ты объявляешь перегрузку оператора<<, у которого аргумент
std::ostream
вmy_vector.h
, но<iostream>
ты в него не заинклюдил. Из-за чего компиляция ожидаемо упала.Следующая причина падения компиляции -- отсутствие константного оператора[], из-за чего функция принимающая вектор по константной ссылке не может им воспользоваться.
Теперь тесты:
resize
с ошибками валгринда -- память течет.Замечания по дезайну
new T;
лучше всегда писатьnew T();
. Эти вызовы будут отличаться, если T -- это POD структура. Первый ничего не проинициализирует, а второй проинициализирует структуру нулями.resize
должен уметь работать в меньшую сторону, у тебя он работает только на расширениеПока что 6/10