Замечания по паттерну "Компоновщик"

[alexey-malov]

void CShapeGroup::SetFrame(const RectD& rect)
{
	auto IsCurrentGroupFrame = GetFrame();
	if (!IsCurrentGroupFrame)
	{
		return;
	}

• IsCurrentGroupFrame - неудачное имя для переменной. Оно подходит для переменной типа bool

[alexey-malov]

std::shared_ptr<const IOutlineStyle> CShapeGroup::GetOutlineStyle() const
{
	if (!m_outlineStyle)
	{
		throw std::runtime_error("Outline style doesn't exist");
	}
	return m_outlineStyle;
}

std::shared_ptr<IStyle> CShapeGroup::GetFillStyle()
{
	if (!m_fillStyle)
	{
		throw std::runtime_error("Fill style doesn't exist");
	}
	return m_fillStyle;
}

• Для чего нужны проверки? Конструктор же создаёт m_fillStyle и m_lineStyle

[alexey-malov]

class CSimpleShape : public IShape
{
public:
	...
	virtual void DrawingBehavior(ICanvas& canvas) const = 0;
};

• Метод следует сделать приватным. Он предназначен не для вызова снаружи, а из метода Draw. Сделав этот метод приватным, наследники всё ещё смогут переопределить его (хотя не смогут вызвать напрямую).

[alexey-malov]

• Если используете SFML, то подключите его через nuget, либо через git submodule. Проект не собирается, если его просто вытянуть из репозитория