Opened 4 years ago
Closed 4 years ago
#630 closed ожидается проверка (задача сдана)
WW #10
Reported by: | lopatin.mikhail | 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.
Используйте ключевое слово override
Circle::is_inside реализован неверно
В zoom стоило бы проверить, что factor >= 0 (и в конструкторе неотрицательность радиуса, высоты, ширины). assert-ов совсем нет, а контракты, как обычно, есть.
Поиск фигуры по id стоило бы вынести в отдельную функцию в Scheme; не хватает assert-ов, что поиск выполнен успешно
в Scheme::remove_figure: figures_[i] == nullptr равносильно i >= size_
36 figures_[size_] = nullptr;
эта строчка некорректна, потому что если size_ == capacity_, происходит выход за границы массива (аналогичная проблема есть и в других местах)
55 for (i = 0; i <= size_; ++i)
если i == size_, произойдет выход за граница; если необходимой фигуры нет и size_ == capacity_, произойдет выход за границы и не вернется nullptr