WW #10

[Денис Лочмелис]

V1.

[Денис Лочмелис]

Сразу комментарий: я слышал, что надо проверять на переполнения в проверке принадлежности точки, но даны координаты до 1000, поэтому зачем?

[Sokolov Viacheslav]

Координаты до 1000 стали в обновленной версии условия

[Sokolov Viacheslav]

Деструкторы тоже стоит помечать как override

Правильнее было бы поставить assert в find_figure, что переданный id присутствует, поскольку в местах использования предполагается, что такой элемент есть - в названии не отражено обратное. Иначе функции стоило бы именовать move_if_present и тп

При добавлении фигуры можно было бы проверить, что id уникален

 10     assert(width >= 0);
 11     assert(height >= 0);
 12     
 13     width = width_;
 14     height = height_;

здесь assert некорректны, потому что на этот момент эти поля еще не инициализированы.
Корень проблемы - в code style:

в одном месте именование полей класса

15 int radius;
16 char* label;

в другом

20 int capacity_;
21 int size_;

_ используется в разных местах для разных целей, что затрудняет написание (и чтение) кода