Замечания по паттерну MVP

[alexey-malov]

• Команды Add и Remove не должны зависеть от документа напрямую. Вместо этого они должны зависеть от абстракции shapes, реализуемой документом или его частью.

[alexey-malov]

• Для определения z-order лучше не полагаться на id фигуры. Вместо этого в модели должен храниться массив фигур, а команды удаления должны знать индекс фигуры, по которому его удалять.

[alexey-malov]

• Сделайть ещё один холст, являющийся идентичным текущему. Изменения на одном холсте должны отражаться на другом и наоборот. Во время переноса или ресайза фигуры на одном холсте не должен происходить перенос и ресайз на другом до окончания операции.