Opened 4 years ago
Closed 4 years ago
#621 closed ожидается проверка (задача сдана)
WW #10
Reported by: | sukhodolskiy.maksim | Owned by: | Sokolov Viacheslav |
---|---|---|---|
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.
3 #define MAX_VALUE 2147483647
плохая идея, потому что по названию константы абсолютно непонятно, откуда она и зачем, максимальное значение чего имеется в виду и тд. Кроме того, в c++ константы задаются с помощью ключевых слов static и constexpr, а макросы для этой цели использовать не стоит.
В C есть INT_MAX, в C++ есть std::numeric_limits<int>::max() (нам пока недоступен)
чтобы не путать x и this->x удобно гарантировать отличие этих сущностей на уровне codestyle. Например, дописывать в начало/конец членов класса underscore _.
Circle::is_inside реализован неверно
В zoom стоило бы проверить, что factor >= 0 (и в конструкторе неотрицательность радиуса, высоты, ширины).