Change History (3)

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

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

Валгринд у тебя детектит ошибку в деструкторе. Состоит она в том, что аллоцируешь ты память оператором new[] для массивов, а освобождаешь обычным delete. Их нельзя смешивать.

Очень рекомендуется проверять работу своей программы валгриндом перед отправкой. Он поможет тебе отследить утечки и другие ошибки в работе с памятью.
Установить его можно командой apt-get install valgrind (если у тебя убунта или что-то дебиан-лайк, для других линуксов погугли), а запустить -- valgrind --leak-check=full ./lab_06

В MyVector::reserve нет нужды копировать всю выделенную память, достаточно лишь ту память, где есть элементы.

Давай ты не будешь писать тело цикла/условия на той же строке что и оператор. Для большинства стайлгайдов это недопустимо и для меня, как проверяющего, затрудняет чтение твоего кода.

comment:2 Changed 3 years ago by Маргарита Михельсон

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

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

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

Когда тело цикла или условия состоит из одной строки, в фигурные собки её брать не обязательно, главное не пиши их на одной строке.

Всё хорошо, зачтено.

Note: See TracTickets for help on using tickets.