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

[alexey-malov]

• Решить проблему с warning-ами при виртуальном наследовании

• Вернуть ISolidShape

[alexey-malov]

class IShape
{
public:
	IShape() = default;

	virtual ~IShape() = default;

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

protected:
	virtual std::string UniqueProperties() const = 0;
};

• Делать в интерфейсе область protected - плохая практика. Интерфейс не должен накладывать на детали реализации какие-то дополнительные требования.

[alexey-malov]

class CSolidShape : public CShape
{
public:
	CSolidShape(const std::string & type, const std::string& outlineColor, const std::string & fillColor);
	virtual ~CSolidShape() = default;
	virtual double GetArea() const { return 0; };
	virtual double GetPerimeter() const { return 0; };

	std::string GetFillColor() const;
protected:
	virtual std::string UniqueProperties() const override;
private:
	std::string m_fillColor;
};

• В CSolidShape нулевые площадь и периметр - неправильное обобщение. В CShape - аналогичная ситуация.