Замечания по Shapes

[alexey-malov]

• [x] Не надо добавлять суффикс Class к имени класса

[alexey-malov]

• [x] Исользуйте UpperCamelCase в именах методов

[alexey-malov]

• [ ] Разобраться, для чего в базовом классе нужен виртуальный деструктор и уметь объяснить преподавателю.

[alexey-malov]

• [x] Для именования полей класса используется префикс m_

[alexey-malov]

• [x] Периметр line segment-а не равен 0. Он равен длине отрезка

[alexey-malov]

• [x] Хранить 0 площадь в line segment-е излишне, потому что для всех экземпляров она равна 0.

[alexey-malov]

• [x] Конструктор PointClass не инициализирует поля. Деструктор в этом классе писать не нужно.

[alexey-malov]

• [ ] В разных классах фигур ToString содержит частично дублирующийся код. Воспользуйтесь паттерном "Шаблонный метод" (пример с объемными телами у меня в репозитории). В репозитории ood есть лекция, посвященная этому паттерну (template method)

[alexey-malov]

double TriangleClass::getPerimetr() const
{
    double side1 = sqrt((vertex2.x - vertex1.x)*(vertex2.x - vertex1.x) + (vertex2.y - vertex1.y)*(vertex2.y - vertex1.y));
    double side2 = sqrt((vertex3.x - vertex2.x)*(vertex3.x - vertex2.x) + (vertex3.y - vertex2.y)*(vertex3.y - vertex2.y));
    double side3 = sqrt((vertex3.x - vertex1.x)*(vertex3.x - vertex1.x) + (vertex3.y - vertex1.y)*(vertex3.y - vertex1.y));
    return side1 + side2 + side3;
}

• [x] Устранить дублирование кода

[alexey-malov]

• [x] Выделить из main функции, оставив в ней только высокоуровневый код

[alexey-malov]

• [x] Должен находиться минимальный периметр, а не максимальный

[alexey-malov]

• [x] При 0 фигур программа упадёт на поиске фигур с минимальным/максимальным периметром

[alexey-malov]

int main()
{
    vector <string> parametrSet;
    deque <CircleClass> circleSet;
    deque <RectangleClass> rectangleSet;
    deque <TriangleClass> triangleSet;
    deque <LineSegmentClass> lineSegmentSet;
    vector <ShapeInterface*> shapeSet;

• [x] Не нужно N коллекций - достаточно одной коллекции с vector<unique_ptr>. Но помните про виртуальный деструктор
• [x] parametrSet не нужен вне цикла. Стремитесь делать область видимости переменной как можно уже.

[alexey-malov]

• [x] В цикле чтения фигур будет куча if-ов при увеличении количества типов фигур. Посмотрите, как сделано в примере с классом RemoteControl для телевизора.

[alexey-malov]

• [x] ToString не протестирован ни у одного класса фигур

[alexey-malov]

double Triangle::GetLength(Point v1, Point v2) const
{
    return sqrt((v2.x - v1.x)*(v2.x - v1.x) + (v2.y - v1.y)*(v2.y - v1.y));
}

• [ ] Зачем делать GetLength методом класса?

[MiNeBudemRabami]

svg