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

[alexey-malov]

    vector<IShape> shapes;

• [x] WTF?

[alexey-malov]

• [x] Проблема с ромбовидным наследованием

void DoSomething(IShape & shape)
{
    cout << shape.ToString() << endl;
}

int main(int argc, char* argv[])
{
    {
        CCircle c({ 0, 0 }, 42, "red", "green");
        DoSomething(c);
    }

[alexey-malov]

• [x] RemoteControl должен хранить массив указателей на IShape, а не на CShape. Для этого решите проблему ромбовидного наследования.

[alexey-malov]

• [x] Данные в классах должны быть объявлены как private

[alexey-malov]

double CTriangle::GetArea()const
{
    auto P = GetPerimeter() / 2;

    auto S = sqrt(P * (P - CLineSegment(m_vertex1, m_vertex2, m_outlineColor).GetPerimeter()) 
        * (P - CLineSegment(m_vertex2, m_vertex3, m_outlineColor).GetPerimeter()) 
        * (P - CLineSegment(m_vertex3, m_vertex1, m_outlineColor).GetPerimeter()));

    return S;
}

• [x] Стрельба из пушки по воробьям. Зачем для нахождения длины отрезка создавать полноценную фигуру CLineSegment?

double CTriangle::GetPerimeter()const
{
    return (CLineSegment(m_vertex1, m_vertex2, "white").GetPerimeter()
        + CLineSegment(m_vertex2, m_vertex3, "white").GetPerimeter()
        + CLineSegment(m_vertex3, m_vertex1, "white").GetPerimeter());
}

• [ ] Тут тоже не исправлено

[alexey-malov]

• [x] методы класса не меняющие его состояние, должны быть константными

[alexey-malov]

• [x] Уметь объяснить, в каких случаях следует объявлять деструктор виртуальным и почему
• [x] Разобраться и объяснить, почему в классах-потомках деструктор можно не объявлять виртуальным. И вообще не объявлять его в потомках (если тело пустое или реализация дефолтная устраивает)

[alexey-malov]

• [ ] Между классами фигур возможно дублирование кода, в частности, ToString(). Устранить его