Opened 4 years ago

Closed 4 years ago

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

WW_10

Reported by: tarasov.denis Owned by: Sokolov Viacheslav
Component: WW figures Version: 2.0
Keywords: Cc:

Description


Change History (3)

comment:1 Changed 4 years ago by Sokolov Viacheslav

Type: ожидается проверка → ожидаются исправления

radius{radius} лучше так не делать (одно из имен стоит поменять)

В Circle::is_inside потенциальное переполнение (в Rectangle::is_inside будем считать, что не бывает переполнения)

21 bool Rectangle::is_inside(int x, int y) const
22 {
23 return (2 * x <= 2 * this->x + width) && (2 * x >= 2 * this->x - width) &&
24 (2 * y <= 2 * this->y + height) && (2 * y >= 2 * this->y - height);
25 }

по этой причине удобно в codestyle разделять имена членов класса и локальных переменных с помощью разного именования (например, с помощью _)

Scheme::push_back_figure
не хватает not null проверки

remove_figure
лучше считать, что фигура должна быть (иначе метод должен бы называться remove_figure_if_present или как-то так)

comment:2 Changed 4 years ago by tarasov.denis

Type: ожидаются исправления → ожидается проверка
Version: 1.0 → 2.0

comment:3 Changed 4 years ago by Sokolov Viacheslav

Resolution: → задача сдана
Status: assigned → closed
Note: See TracTickets for help on using tickets.