#599 closed ожидается проверка (задача сдана)
WW #10
Reported by: | Ruslan Salkaev | Owned by: | Дмитрий Свиридкин |
---|---|---|---|
Component: | WW figures | Version: | 1.0 |
Keywords: | Cc: |
Description
Change History (1)
comment:1 Changed 4 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
Метод find_figure_by_id не должен быть публичным.
К нему точно стоит написать комментарий, что именно будет возвращено, если элемент не найден.
Для наследников с тривиальным дестркутором его можно вообще не писать, чтоб сработало правило нуля.
Проверку is_inside для прямоугольника можно успростить с помощью abs.
Алгоритм сдвига элементов можно написать проще и короче:
в цикле:
Делайте сразу return figures_[index];
8