Opened 4 years ago

Closed 4 years ago

#622 closed ожидается проверка (задача сдана)

WW #10

Reported by: Solovyev Gleb Owned by: Sokolov Viacheslav
Component: WW figures Version: 1.0
Keywords: Cc:

Description

Залил в svn в срок, но ticket сразу забыл повесить.

Change History (2)

comment:1 Changed 4 years ago by Solovyev Gleb

Version: → 1.0

comment:2 Changed 4 years ago by Sokolov Viacheslav

Resolution: → задача сдана
Status: assigned → closed

24 if (count_distance_square(x, y, x_, y_) > radius_square) {
25 return false;
26 }
27 return true;

эквивалентно

return count_distance_square(x, y, x_, y_) <= radius_square;

В zoom стоило бы проверить, что factor >= 0 (и в конструкторе неотрицательность радиуса, высоты, ширины)

Вместо
if (index == NOT_FOUND_INDEX) {

return;

}
лучше было поставить assert(NOT_FOUND_INDEX != index);
потому что в названии функции не отражена возможность того, что фигура не присутствует
(иначе стоило бы именовать функцию remove_figure_if_present или вроде того)

Note: See TracTickets for help on using tickets.