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

[alexey-malov]

class IShape
{
public:
    IShape();
    virtual double GetArea() const = 0;
    virtual double GetPerimeter() const = 0;
    virtual std::string ToString() const = 0;
    virtual std::string GetOutlineColor() const = 0;
//  virtual void SetOutlineColor(const std::string &color) = 0;
};

• [x] C.35: A base class destructor should be either public and virtual, or protected and nonvirtual. Разобраться и уметь объяснить

• [x] Убрать конструктор. У класса нет данных, значит и инициализировать нечего. Нет у него и базового класса без конструктора по умолчанию, значит и инициализировать особы образом не надо. Компилятор сам всё сделает. Это касается и других интерфейсов

class ISolidShape : public IShape
{
public:
    virtual ~ISolidShape() = default;
    virtual std::string GetFillColor() const = 0;
};

• [ ] Если в базовом классе есть виртуальный деструктор, то в классах наследниках он остаётся виртуальным. Дополнительно объявлять его не надо

[alexey-malov]

• [ ] Выпилить .cpp, которые не содержат ничего кроме #include-а заголовков

Остался ISolidShape.cpp

[alexey-malov]

double CTriangle::GetLength(CPoint const & point1, CPoint const & point2) const
{
    return sqrt(pow((point1.GetX() - point2.GetX()), 2)
        + pow((point1.GetY() - point2.GetY()), 2));
}

• [x] Сделать свободной функцией. Можно разместить в тех же файлах, что и CPoint

• [x] Использовать эту функцию как для нахождения длины LineSegment, так и периметра треугольника.

[alexey-malov]

class CShapeCreator
{
public:
    CShapeCreator(std::istream &input);
    ~CShapeCreator();
    std::shared_ptr<IShape> HandleCommand() const;

private:
    std::map<std::string, std::function<std::shared_ptr<IShape>(std::istringstream &strm)>> m_actionMap;
    std::shared_ptr<IShape> CreateLine(std::istringstream & strm) const;
    std::shared_ptr<IShape> CreateTriangle(std::istringstream & strm) const;
    std::shared_ptr<IShape> CreateRectangle(std::istringstream & strm) const;
    std::shared_ptr<IShape> CreateCircle(std::istringstream & strm) const;
    std::istream& m_input;
};

• [ ] Если это создатель, то он должен иметь методы для создания. Возможно, стоит передавать ему в метод создания фигуры поток или строку, а из конструктора выпилить

• [ ] Методы создания фигур не используют состояние объекта. Их можно сделать статическими или свободными функциями в безымянном пространстве имён.

[alexey-malov]

• [ ] Опечатка в слове GetLenghtToPoint

[alexey-malov]

• [ ] Защитить GetMaxShapeArea и GetMinShapePerimeter от разыменования end-указателей при пустом массиве