使用反射在 ViewModel 中创建通用类型命令有哪些缺点？

Question

我有一个具有许多不同列表属性的数据对象。我想使用单个 AddObject 命令，而不是为每个列表创建不同的命令，因此提出了以下代码。

您认为使用它有什么缺点吗？我认为性能可能会很慢，但老实说我没有看到区别。

public MyViewModel()
{ 
    _addCommand = new RelayCommand<IEnumerable>(AddGenericObject);

    // Old code.... defines an Add command per list
    // _addAddressCommand = new RelayCommand(() => AddObject<Address>(AddressList));
    // _addPhoneCommand = new RelayCommand(() => AddObject<Phone>(PhoneList));
    // ... etc
}

private void AddGenericObject(IEnumerable list)
{
    // Find Add Method
    var addMethod = this.GetType()
        .GetMethod("AddObject", BindingFlags.NonPublic | BindingFlags.Instance);

    // Created Generic Add Method
    Type genericType = list.GetType().GetGenericArguments()[0];
    var genericAddMethod = addMethod.MakeGenericMethod(genericType);

    // Invoke Method
    genericAddMethod.Invoke(this, new object[] { list });
}

private void AddObject<T>(EntityCollection<T> list)
   where T : EntityObject, new()
{
    var newItem = new T();
    list.Add(newItem);
}

它在 XAML 中的使用方式如下：

<Button Content="New Address" 
        Command="{Binding AddCommand}"
        CommandParameter="{Binding AddressList}" />

<Button Content="New Phone" 
        Command="{Binding AddCommand}"
        CommandParameter="{Binding PhoneList}" />

Answer 1

一言以蔽之 - 性能，但在您摆脱代码基准测试之前 - 它可能足够快以满足您的需求。

Answer 2

在考虑使用反射时，性能始终是最大的因素。一般来说，如果可以的话，您应该尽量避免使用反射。

Answer 3

正如詹姆斯和德罗所说，表现是一个重要因素。您可以执行以下操作来缓存该方法。

    private MethodInfo cachedMethod = null;
    private void AddGenericObject(IEnumerable list)
    {
        if (cachedMethod == null)
        {
            // Find Add Method
            var addMethod = this.GetType()
                 .GetMethod("AddObject", BindingFlags.NonPublic | BindingFlags.Instance);

            // Created Generic Add Method
            var genericType = list.GetType().GetGenericArguments()[0];
            cachedMethod = addMethod.MakeGenericMethod(genericType);
        }

        // Invoke Method
        cachedMethod.Invoke(this, new object[] { list });
    }

Answer 4

我的方法是使用 Freezable 作为 Command 对象，给它一个 Action(of Object) 类型的 Delegate 作为 DependencyProperty，并使用一个转换器，该转换器接受方法名称作为 ConverterParameter 并将对集合的引用转换为绑定到集合时，委托集合的 Add 方法（或用作转换器参数的任何方法名称）。这样，只有在交换绑定集合对象本身时才会产生反射成本，这种情况应该相当罕见，因为您希望保留相同的集合并仅添加和删除项目。

如果您正在寻找示例代码，您可能需要查看此博客文章：http://wpfglue.wordpress.com/2012/05/07/commanding-binding-controls-to-methods/