c# - 对于生命周期较短的对象，是否需要取消订阅事件？

Question

我有一个 View 模型，它从某个服务中检索一个对象，并使其可用于数据绑定(bind)。该对象正在实现 INotifyPropertyChanged。在 View 模型中，我正在监听 PropertyChanged 事件，以便在修改对象中的某些属性时执行一些内部操作。

现在可以从服务请求一个新对象，完全替换旧对象。鉴于生命周期本质上受 View 模型本身的限制，并且没有其他人持有对它的引用(WPF 使用弱监听器)，在这种情况下我是否需要取消订阅该对象？当然，我应该并且在 setter 中这样做很简单，但我真的需要吗？

public class MyViewModel : INotifyPropertyChanged
{
    private DataType myData;
    public DataType MyData
    {
        get { return myData; }
        protected set
        {
            if (value == myData)
                return;

            if (myData != null)
                myData.PropertyChanged -= DataPropertyChanged;
            myData = value;
            myData.PropertyChanged += DataPropertyChanged;

            OnNotifyPropertyChanged("MyData");
        }
    }

    public void UpdateData ()
    {
        MyData = service.GetData();
    }

    // ...
}

Answer 1

您实际上不需要做任何事情，但您应该在完成后从事件中分离旧对象。有两个原因。

如果对象被垃圾回收并触发了事件，则需要花费一些时间来确定对象不再存在。 希望 然后它将从事件处理程序列表中删除。如果没有，下次也将花费更多时间，依此类推。

更重要的是，如果您的旧对象未被垃圾回收并且事件被触发，您将获得两个事件通知 - 一次在旧对象中并且一次在新的。您将需要专门在旧对象中处理这种情况(否则，会发生不好的事情)。

处理此问题的最简单方法是在完成后从事件中分离。