Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#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: assignedclosed

Метод find_figure_by_id не должен быть публичным.
К нему точно стоит написать комментарий, что именно будет возвращено, если элемент не найден.

Для наследников с тривиальным дестркутором его можно вообще не писать, чтоб сработало правило нуля.

Проверку is_inside для прямоугольника можно успростить с помощью abs.

Алгоритм сдвига элементов можно написать проще и короче:
в цикле:

figures[i] = figures[i+1];


   result = figures_[index];
   break; 

Делайте сразу return figures_[index];


8

Last edited 4 years ago by Дмитрий Свиридкин (previous) (diff)
Note: See TracTickets for help on using tickets.