命令模式撤消/重做：我的方法有什么问题吗？

Question

我正在寻求实现一个命令模式来支持我的应用程序中的撤消/重做。数据紧密地联系在一起，因此修改我的一些对象会产生一些下游后果，我也希望能够撤消这些后果。我主要关心的是应该将执行下游命令的代码放在哪里。例如：

class:MoveObjectCommand
{
    private hierarchicalObject:internalObject;

    public MoveObjectCommand(hierarchicalObject:newObject)
    {
        internalObject = newObject;
    }

    public Execute()
    {
        internalObject.Location = someNewLocation;

        foreach(hierarchicalObject:child in internalObject.Children)
        {
            if(someNewLocation = specialPlace)
            {
                var newCommand:MoveObjectCommand = new MoveObjectCommand(child)

                CommandManager.add(newCommand);
            }
        }

    }

    public Undo()
    {
        internalObject.location = oldLocation;
    }
}

据我所知，这样的事情会很好，但我无法理解大部分执行代码实际上应该去哪里。 hierarchicalObject 是否应该有一个 .changeLocation() 方法来添加所有后续命令，或​​者它们应该像上面一样位于命令本身中？我能想到的唯一区别是，在上面的示例中，必须调用 MoveObjectCommand 才能进行后续更改以进行处理，而另一种方式可以在不需要命令的情况下调用它，并且仍然以相同的方式处理（可能会产生负面后果）用于跟踪撤消/重做步骤）。我是不是想太多了？你会把它放在哪里以及为什么（显然这个例子并没有涵盖所有角度，但有命令模式的一般最佳实践吗？）。

Answer 1

听起来你应该在模型中使用changeLocation()方法（我认为是hierarchicalObject）。只需将新位置和对象存储在命令中即可。

对于撤消/重做，您将需要一两个命令列表。

听起来你的hierarchicalObject可能是一个http://en.wikipedia.org/wiki/Composite_pattern，所以看一下四人帮书中的宏命令。另请参阅：http://en.wikipedia.org/wiki/Command_pattern。

Christopher Alexander 说：“每种模式都描述了一个在我们的环境中反复出现的问题，然后描述了该问题解决方案的核心，这样您就可以使用该解决方案一百万次，而无需执行任何操作以同样的方式两次”。