Замечания по редактору фигур

[alexey-malov]

class Command
{
public:
    virtual void execute() = 0;
    virtual void undo() = 0;
};

В C++ деструктор полиморфного (содержащего виртуальные методы или выступающего в роли базового) класса/структуры должен быть объявлен виртуальным http://cpp-reference.ru/articles/virtual-destructor/

[alexey-malov]

В MVC классы модели и представления должны быть разделены. В твоем же случае они совмещены воедино.

Т.е. правильно так:

• в модели отсутствуют какие-либо упоминания сущностей графической подсистемы (в данном случае - sfml)

[alexey-malov]

При Undo и последующем Redo добавлении команды добавления фигуры, объект размещается в случайных координатах, а не в тех, где был изначально добавлен

[alexey-malov]

void Workspace::RemoveSelectedShape()
{
    for (size_t i = 0; i < shapes.size(); i++)
    {
        if (shapes[i].IsSelect())
        {
            shapes.erase(shapes.begin() + i);
        }
        break;
    }
}

цикл выполнится 0 или 1 раз. на кой черт он нужен?