Замечания по CCanvas::CheckIndex - Githubissues

7kia / ISpringPractic

0 stars 0 forks source link

Замечания по CCanvas::CheckIndex #13

Closed alexey-malov closed 7 years ago

alexey-malov commented 7 years ago

[x] переименовать в CCanvas::CheckShapeIndex
[x] CheckIndex для 0 при 0 фигур завершится успешно, но удалить итератор из пустого массива будет нельзя. Надо сделать так, чтобы CheckShapeIndex проверял, что index < количество фигур. >= 0 можно не проверять, потому что size_t не может быть меньше 0
```
void CCanvas::DeleteShape(size_t index)
{
CheckIndex(index);
m_shapes.erase(m_shapes.begin() + index);
}
```

alexey-malov commented 7 years ago

[x] Вместо передачи доп. параметр isInsert передавать, когда нужно, значение верхней границы, увеличенное на 1

alexey-malov commented 7 years ago

CCanvas::CheckShapeIndex сделать статическим. Может, лучше вообще выпилить этот метод и явно проверять индекс в методах?