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

[alexey-malov]

include stdafx.h в заголовочных файлах не размещают

[alexey-malov]

• [ ] Узнать лучше о том, для чего нужен виртуальный деструктор, а также в каких ситуациях его неиспользование чревато проблемами, а в каких - нет

[alexey-malov]

• [ ] Структуры должны использовать UpperCamelCase, например: CoordinatesType

• [ ] pair плохо подходит для хранения координат, т.к. имена first и second вызывают слабые ассоциации с координатами

• [ ] Завести для этих целей структуру Point, либо Vector, либо Coordinates

[alexey-malov]

• [ ] Для чего окружность принимает в конструктор координаты своего центра и при этом никак их не использует

[alexey-malov]

• [x] префикс str в strColor абсолютно бесполезен. fillColor без этого префикса спокойно обходится

[alexey-malov]

• [x] Тесты разнести по разным файлам

[alexey-malov]

• [ ] почему-то Rectangle не считается сплошной фигурой.

[alexey-malov]

• [ ] .sln файл не открывается (не загружаются проекты)

[alexey-malov]

• [ ] При изменении размеров окна картинка не должна искажать свои пропорции

[alexey-malov]

• [ ] В файлах проектов не должно быть абсолютных путей к библиотекам третьих лиц

[alexey-malov]

• [ ] Отрисовкой должны заниматься сами классы фигур

class ICanvas
{
public:
    virtual void DrawLine(...) = 0;
    virtual void FillPolygon(...) = 0;
    // прочие методы рисования примитивов
};

class IShape
{
public:
    virtual void Draw(ICanvas & canvas) = 0;
};

[alexey-malov]

• [ ] В программе должен использоваться вектор интерфейсов IShape, а не классов CShape

[alexey-malov]

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

void Test()
{
    CCircle c({10, 20}, 30, "#000000", "#ffffff");
    DoSomething(c);
}

• [ ] Код не компилируется