c# - 我应该创建很多 PropertyChangedEventHandler 还是测试 PropertyChangedEventArgs？

Question

到目前为止，我的模型实现了 INotifyPropertyChanged，并且每个属性都会引发此事件。几乎所有 ViewModel 都通过 PropertyChangedEventHandler 监听这些变化。

问题是模型中的每次更改都会调用此处理程序，即使属性更改对 View 而言并不重要。

一种选择是检查引发事件的属性。但是，我不喜欢测试 PropertyName 字符串的想法。它需要对属性名称进行硬编码，我已经在模型中通过调用 PropertyChanged.Notify(()=> PropertyName)

避免了这一点

我看到的第二个选项是为我的所有属性实现单个事件:

public event PropertyChangedEventHandler LayerChanged;
public event PropertyChangedEventHandler FieldChanged;
public event PropertyChangedEventHandler LinkDictionaryChanged;

....

最佳做法是什么？我更喜欢第二个选项。

编辑:我尝试更具体

我的模型类是这样工作的:

  public bool IsFeatureLayer
        {
            get { return _isFeatureLayer; }
            set { PropertyChanged.ChangeAndNotify(ref _isFeatureLayer, value, () => IsFeatureLayer);}
        }

或者

  PropertyChanged.Notify(() => LinkDictionary);

所以问题不在于如何使通知调用更安全，因为我已经使用扩展方法来执行此操作而无需属性的字符串名称。

问题是如何在不使用字符串的情况下找出谁调用了事件。

 void _MCDAExtensionPropertyChanged(object sender, PropertyChangedEventArgs e)
        {
            if(e.PropertyName.Equals("LinkDictionary"){
              //event handling
           }
        }

这是完全不安全的，因为我的模型中的 Property 名称可能会更改，我必须将其固定在不同的地方。

Answer 1

如果您的目标是 .NET 4.5，则使用新的 CallerMemberName 属性实现 INotifyPropertyChanged 会更加容易和安全。

简而言之，CallerMemberName 属性允许您获取调用成员的名称作为方法参数。这样，您就可以拥有这样的东西:

private string name;
public string Name
{
    get { return name; }
    set { SetProperty(ref name, value); }
}

private void SetProperty<T>(ref T field, T value, [CallerMemberName] string callerMemberName = "")
{
    // callerMemberName = "Name" (the property that called it).

    // Set the field value and raise PropertyChanged event.
}

您可以查看如何使用它的示例 here .

至于选择哪个选项 - 我认为执行时间方面的差异可以忽略不计，而不是编码开销和代码困惑(在代码本身和智能感知中)你有一个额外的事件每个属性。我肯定会选择第一个选项。

编辑:

不幸的是，在处理 PropertyChanged 事件时，您只能针对 PropertyName 字符串进行测试，并且无法以保持一致的方式获取该字符串，即使当属性名称更改时。对于依赖属性，您有 MyDependencyProperty.Name，但这不适用于常规属性。

最终，您的选择是要么为每个属性使用不同的事件，要么在类中定义一个常量来定义包含属性名称的属性，希望您在/如果更改属性时记得修改它姓名。假设您没有很多实现 INotifyPropertyChanged 的类，您自己在其中附加处理程序，那么为这些特定类中的每个属性设置一个事件并不是那么糟糕。