WW_figures Tulchinskiy lab_09

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

Вот из-за того, что вы вот так пользуетесь конструктором Circle:

(char*)"very nice circle")

У меня не компилируются кастомные тесты. Лечится const-м в правильном месте

Что-то падают совсем все тесты. Вот этот тест вообще вызывает сегфолт:

cheme.push_back_figure(new Rectangle(1, 5, 5, 10, 10));
scheme.push_back_figure(new Circle(2, -3, -3, 1, "very nice circle"));
print(scheme.is_inside_figure(5, 5));
scheme.zoom_figure(1, 2);
scheme.remove_figure(1);

Кажется, что вы неправильно пользуетесь id. Id пользователь может задать любые. Например, один из тестов, который падает: пользователь создаёт фигуры с id 28, 233, 234, 12, а потом удаляет 234 и 28.
Ожидается, что останется 233 и 12, а у вас, как ни странно остаются 28 и 243

Стиль:

• хочется больше параметров в списках инициализации в конструкторах. Прям всё, что можно, туда добавьте
• вот для этого for (; label[label_size] != '\0'; label_size++); есть функция strlen

2/10

[tulchinskij.aleksandr]

В вашем примере кода:
print(scheme.is_inside_figure(5, 5));
это кастомная функция? Просто если вставить этот кусок, то код не скомпилируется, а если исправить на scheme.is_inside_figure(5, 5)->print();, то сегфолта не будет.

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

10/10