你将如何改进这个浅复制类？

Question

我编写了一个带有单个静态方法的类，该方法将属性值从一个对象复制到另一个对象。 它不关心每个对象是什么类型，只关心它们具有相同的属性。 它满足了我的需要，因此我不会进一步对其进行设计，但是您会做出哪些改进？

这是代码：

public class ShallowCopy
{
    public static void Copy<From, To>(From from, To to)
        where To : class
        where From : class
    {
        Type toType = to.GetType();
        foreach (var propertyInfo in from.GetType().GetProperties(BindingFlags.GetProperty | BindingFlags.Public | BindingFlags.Instance))
        {
            toType.GetProperty(propertyInfo.Name).SetValue(to, propertyInfo.GetValue(from, null), null);
        }
    }
}

我使用它的方式如下：

EmployeeDTO dto = GetEmployeeDTO();
Employee employee = new Employee();
ShallowCopy.Copy(dto, employee);

Answer 1

您的 DTO 可以序列化吗？ 我希望如此，在这种情况下：

MemberInfo[] sm = FormatterServices.GetSerializableMembers(typeof(From));
object[] data = FormatterServices.GetObjectData(from, sm);
FormatterServices.PopulateObjectMembers(to, sm, data);

但请注意，我并不真正同意这种一般方法。 我更喜欢在每个 DTO 实施的 DTO 上进行复制的强有力的合同。

Answer 2

• 更改类型参数名称以符合命名约定，例如 TFrom 和 TTo，或 TSource 和 TDest（或 TDestination）。

  更改类型参数名称以符合命名

• 在泛型类型中完成大部分工作，而不仅仅是在泛型方法中。 这允许您缓存属性，并允许类型推断。 类型推断对于“TFrom”参数很重要，因为它将允许使用匿名类型。

• 您可以通过动态生成代码来进行属性复制并将其保存在对“from”类型有效的委托中，从而使其速度快得令人眼花缭乱。 或者可能为每个从/到对生成它，这意味着实际复制根本不需要使用反射！ （准备代码将是每对类型的一次性命中，但希望您不会有太多对。）

Answer 3

创建 To 的新实例并在返回之前调用 Copy() 方法的新方法可能会很有用。

像这样：

public static To Create<From, To>(From from)
    where To : class, new()
    where From : class
{
    var to = new To();
    Copy(from, to);
    return to;
}

Answer 4

确定如果传递的对象类型共享某些属性但不是全部属性，您要做什么。 在尝试设置属性值之前，请检查 From 对象和 To 对象中的属性是否存在。 当您来到一个不存在的房产时，做“正确的事”。 如果所有公共属性都需要相同，那么您需要检查是否已在 To 对象上设置所有属性，并处理未正确设置的情况。

我还建议您可能想要使用属性来装饰需要复制的属性并忽略其他属性。 这将使您能够更轻松地在两个不同的对象之间来回切换，并继续维护一些派生的公共属性，而不是存储在业务对象上的公共属性。