Opened 3 years ago

Closed 3 years ago

#294 closed ожидается проверка (задача сдана)

WW_vector Kiselev Igor lab_06

Reported by: kiselev.igor Owned by: Антон Филатов
Component: WW_vector Version: 3.0
Keywords: Cc:

Description


Change History (5)

comment:1 Changed 3 years ago by Антон Филатов

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

ломается тест на insert
resize 5
info
set 3 -1
insert 0 7
info

Вот такой код делает вектор из 7 элементов, в последнем дежит мусор.

И везде ругается валгринд. На всех тестах

Стиль:

  • Хорошо бы пустой вектор делать с capacity = 2, а не 0 - это просто общая практика
  • Кажется, что реализация insert-а может стать лучше, если воспользоваться memmove или memcpy
  • Ваш erase тоже можно немного проапгредить. Не буду сейчас говорить, как именно. Сначала допилите insert, а потом по аналогии erase

Любопытно, что вы оператор присваивания делаете через цикл, а конструктор копирования через memcpy

Пока что 2/10

comment:2 Changed 3 years ago by kiselev.igor

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

comment:3 Changed 3 years ago by Антон Филатов

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

инсерт всё ещё падает

resize 5 > insert 0 7 > insert 6 5

Остальные тесты проходят
По стилю. По-прежнему хочется, чтобы ваш конструктор копирования и оператор присваивания были больше похожи друг на друга. Но без копипасты

8/10

comment:4 Changed 3 years ago by kiselev.igor

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

comment:5 Changed 3 years ago by Антон Филатов

Resolution: задача сдана
Status: assignedclosed

Принимается
10/10

Note: See TracTickets for help on using tickets.