issues
search
daria-grebneva
/
OOP
0
stars
0
forks
source link
Замечания по лабораторной работе Shapes
#17
Open
oMystique
opened
6 years ago
oMystique
commented
6 years ago
[x] Простые типы данные следует передавать по значению
oMystique
commented
6 years ago
[x] Не следует заводить именованные константы, если подбираются слишком абстрактные имена для них, не передающие суть константы Пример норм. констант: M_PI = 3.14... Пример не норм. констант: TWO_COEFFICIENT = 2.0 HALF_COEFFICIENT = 0.5 ZERO = 0
oMystique
commented
6 years ago
[x] Разобраться с виртуальностью деструкторов
[x] Разобраться с лишним *.cpp файлом (IShape)
oMystique
commented
6 years ago
[x] m_color лучше переименовать в m_outlineColor
oMystique
commented
6 years ago
[x] Нужно предусмотреть вариант, когда пользователь вводит цвет вместе с прозрачностью
oMystique
commented
6 years ago
[x] Лучше бросать конкретные исключения, вместо базового std::exception