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

[alexey-malov]

• [x] Зачем в CSelectedShape нужен масив m_moveShape, содержащий 1 элемент? Тем более, что в void CSelectedShape::SetMoveView() однозначно подразумевается, что в нем содержится только один элемент.

[alexey-malov]

    SFrameData                      m_frameData;
    SFrameData                      m_oldData;
    CShapePtr                       m_selectedShape;

    ArrayShapes                     m_resizeShapes;

    // For drag and drop
    boost::optional<Vec2f>              m_startMove;
    Vec2f                               m_start;
    Vec2f                               m_current;
    bool                                m_isUpdate = false;
    UpdateType                          m_updateType = UpdateType::None;

• [x] имя, содержащее data следует использовать, когда ничего о данных не известно. В вашем случае это oldFrame и newFrame.

• [x] Выпилить SFrameData, т.к. есть классы, решающие задачу хранения информации о прямоугольнике - CFrame

[alexey-malov]

В CSelectedShape выпилить неиспользуемые методы, либо сделать их приватными, если они снаружи не нужны.

[alexey-malov]

GetOldFrameData - SetOldFrameData - выпилить слово Data.

Зачем хранить OldFrame внутри selected shape, если можно хранить только текущий фрейм, а старый фрейм хранить в команде и восстанавливать при undo

[alexey-malov]

    bool    CheckBoundingRect(const D2D1_RECT_F & rect) const;

• [ ] Используется только внутри CSelectedShape, нет нужды делать публичным

[alexey-malov]

• [ ] Наличие метода SetOldFrame в CSelectedShape выглядит как костыль

[alexey-malov]

• [ ] ReturnToOldState тоже выглядит как костыль. Подумай, может где-то в другом месте следует запоминать старые координаты?

[alexey-malov]

• [ ] GetDirectionResize() используется только внутри, нет необходимости делать публичным. Переименовать в GetResizeDirection

[alexey-malov]

• [ ] GetFinalShift используется только внутри CSelectedShape. пересмотреть публичность методов

[alexey-malov]

• [ ] пересмотреть публичность методов

[alexey-malov]

• [ ] Вместо кучи методов In*Marker нельзя ли восмользоваться одним методом, возвращающим enum?

[alexey-malov]

• [ ] Есть ощущение, что класс нарушает Single Responsibility Principle. Вероятно, стоит разбить на несколько.