WW_figures Tyuryaev lab_09

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

Все тесты пройдены. Из всей сегодняшней партии посылок такое удалось всего 2 человекам. Горжусь.
Стиль:

• public int id у фигуры - это плохо. геттер - будет хорошо
• делить height и width на 2 в is_inside? Мьсе знает толк в целых числах ;)
• Кажется, что не хватает метода поиска фигуры по id - поможет избедать копипасты в паре мест. А если будете возвращать её номер в массиве, то можно будет и remove значительно упростить.

8,5/10

[tyuryaev.ilya]

делить height и width на 2 в is_inside? Мьсе знает толк в целых числах ;)
Но ведь центр прямоугольника у нас задается целыми координатами int x и int y, а значит у нас width и height кратны 2

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

• Слушайте, мне не очень нравится ваша реализация is_inside в прямоугльнике.

Столько умножений... У нас же прямоугольник не вращается. А, значит, координаты его вершин можно узнать за 4 сложения по х и 4 сложения по у

• Геттеру get_id не хватает константности

9,5/10