Undo/redo 纪念图案 c#

Question

我正在尝试在 C#（treeView 组件）中为树的节点实现 Undo/Redo 功能。我使用了备忘录模式，但我在重做部分遇到了问题。我看不出我的逻辑哪里有问题。这是代码的一些快照

    private List<Memento> _mementoStateList= new List<Memento>();
    private List<Memento> _undoStateList= new List<Memento>();
    public Memento Memento { get{return null;}
        set{_mementoStateList.Add(value);} }

    public Memento Undo()
    {
        if (!_mementoStateList.Any()) return null;
        Memento m = _mementoStateList.Last();
        _undoStateList.Add(m);
        _mementoStateList.Remove(m);
        return m;
    }

    public Memento Redo()
    {
        if (!_undoStateList.Any()) return null;
        Memento m = _undoStateList.Last();
        _mementoStateList.Add(m);
        _undoStateList.Remove(m);
        return m;
    }

在我的表单中，在删除节点之前，我调用了 SaveMemento() 方法，该方法创建了一个表示当前状态的新 Memento 对象。对象被添加到 _mementoStateList。

当撤销和重做一个动作时，我调用上面的Undo()和Redo()方法。

我假设我没有在适当的时候拯救各州？非常感谢任何意见！

Answer 1

当您创建备忘录时，您必须对对象树进行深度克隆，否则备忘录将只是对当前状态的引用。在那种情况下，对状态的任何更改都会反映到所有纪念品上，从而破坏任何保留先前（或潜在的未来）状态历史的尝试。

Answer 2

如果您需要存储很多状态（实际上问题是您将支持多少撤消操作），那么您可能应该考虑使用 Command pattern, while with Memento 来实现 Undo/redo，那么实现可能会很繁重.

Undo/redo 纪念图案 c#

Undo/redo memento pattern c#

c#

design-patterns

undo

memento

redo