Замечания по фабрике

[alexey-malov]

• [x] При рисовании на холсте фигуры должны устаналивать цвет линий в свой собственный перед рисованием линий. При создании фигуры должна иметься возможность задать ее цвет.

[alexey-malov]

CShapeFactory::CreateShape

• [x] При чтении фигуры надо проверять ошибки чтения аргументов.

• [x] Разбить метод на более мелкие, чтобы добавление доп. фигур не сильно увеличивало размер метода. Есть несколько методов:

• сразу после определения типа фигуры вызвать соотв. метод

• завести map, отображающий тип фигуры в function, связанный с методом класса (можно посмотреть, как сделано в menu или в лабе 3 по oop (программа tvset класс remote control))

[alexey-malov]

    for (auto it = draft.begin(); it != draft.end(); it++)
    {
        it->Draw(canvas);
    }

• [x] Можно использовать range-based for.

[alexey-malov]

    canvas.SetColor(this->GetColor());

• [x] Можно просто GetColor(), без this->

[alexey-malov]

std::unique_ptr<CShape> CShapeFactory::GetRectangleWithParams(std::vector<std::string> params) const

• [x] вектор передавать по константной ссылке

[alexey-malov]

class CMockShape : public CShape
{
public:
    CMockShape(const string& descr)
        : descr(descr)
        , CShape()
    {
    }

• [ ] Если не указать вызов конструктора явно, то будет вызван конструктор по умолчанию. Явный вызов CShape() здесь избыточен

[alexey-malov]

1>c:\teaching\ood\2017-ood\koptelin\ood\lab04_factory\libpainter\utils.h(10): warning C4505: 'GetColorFromString': unreferenced local function has been removed
1>c:\teaching\ood\2017-ood\koptelin\ood\lab04_factory\libpainter\utils.h(38): warning C4505: 'GetParamsFromString': unreferenced local function has been removed
1>ShapeFactory.cpp
1>c:\teaching\ood\2017-ood\koptelin\ood\lab04_factory\libpainter\utils.h(24): warning C4505: 'ColorAsString': unreferenced local function has been removed
1>Shape.cpp

• [ ] Функцию следовало объявить в заголочном файле. а ее реализацию - в .cpp. При подключении библиотеки объявленные в ней функции будут прилинкованы к основной программе. Либо объявить функцию как inline, чтобы попросить компилятор выполнить ее встраивание