Opened 3 years ago

Closed 3 years ago

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

WW_09

Reported by: Tatiana Pridonyants Owned by: Святослав Власов
Component: WW figures Version: 3.0
Keywords: Cc:

Description


Change History (5)

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

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

Все тесты попадали, потому что ты выводишь фигуры в неверном формате. Наши тесты сравнивают вывод посимвольно, поэтому формат должен точно совпадать с тем, что требуется в задании.
Тебе нужно это исправить, без этого я не могу проверить корректность.
Еще некоторые тесты попадали по сегфолту. Подозреваю, что причина в функции remove_figure. Кажется, ты её плохо тестировала.

По стилю:

  1. Функцию get_index нужно сделать приватной. У тебя снаружи класса нет доступа к массиву фигур, а значит функция которая возвращает индекс этого массива оказывается неприменимой за пределами класса.
  2. Вот такая конструкция избыточна:
        if (_label != nullptr) {
            delete[] _label;
        }
    

потому что операция delete nullptr; не является ошибкой, этот вызов просто не делает ничего.

comment:2 Changed 3 years ago by Tatiana Pridonyants

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

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

Type: ожидается проверкаожидаются исправления
  1. Формат по прежнему хромает. Ты поменяла местами width и height в строке printf, а сами переменные в аргументах поменять забыла
  2. Circle::is_inside работает неправильно.

6/10

comment:4 Changed 3 years ago by Tatiana Pridonyants

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.