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

[alexey-malov]

class IShape
{
public:
    IShape() = default;
    virtual ~IShape() = default;

    virtual double GetArea() const = 0;
    virtual double GetPerimeter() const = 0;
    virtual std::string ToString() const = 0;
    std::string GetOutlineColor() const;

protected:
    std::string m_outlineColor;
};

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

[alexey-malov]

• [x] Реализация ToString в разных классах фигур очень похожа друг на друга. Примените паттерн "шаблонный метод". Посмотрите пример с объемными телами, там эта проблема решена подобным способом

[alexey-malov]

• [x] метод DoCommand должен принимать команду по значению, либо по константной ссылке

[alexey-malov]

• [x] DoCommand разбить на подфункции

[alexey-malov]

• [ ] В фабрике был бы смысл, если бы она массив параметров принимала бы

[alexey-malov]

• [ ] В фабрике (которая просто набор функций) следует возвращать конкретные класса, а не CShape. Будет больше пользы.

[alexey-malov]

• [ ] В CConsoleMenu::Create-методы содержат дублирующийся код вставки элемента в массив. Пусть лучше возвращают объект, которые вызывающим кодом бы вставлялся в вектор. В фабрике нет смысла, можно из ее методов код перенести в ConsoleMenu::Create*