C# 简化并可能概括了我的对象克隆方法

Question

我写了一个方法:

class CopyableFloatCommand : FloatCommand
{
    public CopyableFloatCommand DeepCopy(LocationHeaderDTO locHeader, string commandId,
        List<FloatProductDetailsDTO> recountProuducts)
    {
        var newCommand = (CopyableFloatCommand)MemberwiseClone();
        newCommand.Location = locHeader ?? newCommand.Location;
        newCommand.CommandId = commandId ?? newCommand.CommandId;
        newCommand.RecountProducts = recountProuducts ?? newCommand.RecountProducts;
        return newCommand;
    }
}

然后通过以下方式调用它:

_tCheckinCommand = _pTCommand.DeepCopy(stagingLocHeadDto, SCICommand,
    new List<FloatProductDetailsDTO>(_pTCommand.MoveProducts));

为了深度复制一个 FloatCommand 类型的对象。

由于 MemberwiseClone() 是一种 protected 方法，因此必须按照您在上面看到的方式调用它 - 例如，不能在方法参数中解析 FloatCommand 类型并通过 fc.MemberwiseClone() 调用它。由于我的方法应该适用于 FloatCommand 类型，因此我创建了一个新的嵌套类 CopyableFloatCommand，它继承自 FloatCommand。 DeepCopy 方法然后浅克隆 FloatCommand，转换为子类型并在需要时更改一些属性。

专门为此目的创建一个新类似乎有点笨拙，当时我没有看到更明显的编写方法。就代码行而言，是否有更简单的方法来使用上述深度复制？如果另一个类 UserCommand 试图深度复制 User 对象怎么办？ UserComand 将是 FloatCommand 的兄弟，因此它们都继承自 Command。该方法将为不同类型解析不同的参数(尽管我可以完全删除参数并在需要时使用实例变量)，因为不同的子类型具有略微不同的属性。

鉴于此，是否有更通用的方法来编写 DeepCopy 方法，以供所有命令类型访问，以避免某些代码重复，给定上述限制？

谢谢!

Answer 1

我认为您怀疑克隆对象和在克隆后改变其状态的责任应该分开 - 因为您再次面临类似的任务(我的意思是 UserCommand)。

在这种情况下我会做以下事情:

创建一个突变接口(interface):

public interface ICopyCommandMutation
{
  void Mutate(Command target); 
}

为了可扩展性，我将创建默认的 muate 实现:

public class NoMutation : ICopyCommandMutation
{
  public void Mutate(Command target) {}
}

创建 CopyableCommand 类并将 DeepCopy() 方法移到那里(您还应该从 CopyableCommand 继承 FloatCommand):

public CopyableCommand : Command 
{
   public CopyableCommand DeepCopy(ICopyCommandMutation commandMutation = null)
   {
      var newCommand = (CopyableCommand)MemberwiseClone();
      if (commandMutation == null) commandMutation = new NoMutation();
      commandMutation.Mutate(newCommand);
      return newCommand;
   }
}

现在所有 CopyableCommand 继承者都可以使用“突变”进行复制 - 您只需要实现该类。例如，您问题中的 FloatCommand“突变”:

public class ChangeLocationRecountProducts : ICopyCommandMutation
{
  // these fields should be initialized some way (constructor or getter/setters - you decide
  LocationHeaderDTO locHeader;
  string commandId;
  List<FloatProductDetailsDTO> recountProducts;

  public void Mutate(Command floatCommand)
  {
     var fc = floatCommand as FloatCommand;
     if (fc == null) { /* handle problems here */ }
     fc.Location = locHeader ?? fc.Location;
     fc.CommandId = commandId ?? fc.CommandId;
     fc.RecountProducts = recountProuducts ?? fc.RecountProducts;
  }
}

用法如下:

var clrp = new ChangeLocationRecountProducts();
// ... setting up clrp
_tCheckinCommand = _pTCommand.DeepCopy(clrp);

现在，如果您需要“改变”UserCommand - 您可以为其创建单独的改变类，并将改变逻辑保留在那里。在不同的突变中进行不同突变的能力(仅通过定义单独的突变类)是免费的。我在这里看到的唯一问题是您可能无法创建 CopyableCommand 并从中继承其他命令(第 3 方库？)。解决方案是使用 CaSTLe 动态代理。

我没有使用过 Automapper，但我怀疑它在做类似的事情。

该解决方案不是“代码行最优”——但如果您在复制实例时必须改变大量命令类，您将从中受益。