Opened 4 years ago
Closed 4 years ago
#585 closed ожидаются исправления (задача сдана)
WW #10
Reported by: | korolev.dmitriy | Owned by: | korolev.dmitriy |
---|---|---|---|
Component: | WW figures | Version: | 1.0 |
Keywords: | Cc: |
Description
Change History (2)
comment:1 Changed 4 years ago by
Owner: | changed from Дмитрий Свиридкин to korolev.dmitriy |
---|---|
Type: | ожидается проверка → ожидаются исправления |
comment:2 Changed 4 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
В Rectangle.hpp отсутствует include_guard
Зачем в Scheme.hpp подключены заголовки для конкретных реализаций фигур?
А std::stack тут при чем?
поля x,y лучше переименовать в соответсвии с тем, что эти координаты означают.
У вас все методы именуются в snake_case, getId в camelCase.
Дефолтные деструкторы можно сгенерить с помощью = default прямо в заголовке;
push_back получился совсем не back, а front. А после удаления -- вообще куда получится.
Удаление работает не так, как требуется в условии.
is_inside для прямоугольника можно написать проще и с меньшей вероятностью ошибиться. Используйте abs.
А зачем повторно инициализировать поля базового класса, если вы уже ему эту инициализацию делегировали?
is_inside для круга работает не верно. Не стоит переходить к float, если можно посчитать в целых числах.
5