在 ViewModel 中取消挂钩事件处理程序的正确方法是什么

Question

所以我一直在研究如何从我的视图模型中正确解除事件处理程序以防止内存泄漏。

假设我有一个像这样的视图模型：

class MyViewModel
{
    private List<MyObject> _myObjects;
    public List<MyObject> MyObjects
    {
        get { return _myObjects; }
        set { _myObjects = value; }
    }

    public MyViewModel()
    {
        for (int i = 0; i < 10; i++)
        {
            var obj = new MyObject();
            obj.MySampleEvent += Obj_MySampleEvent ;
        }
    }

    private void Obj_MySampleEvent (object sender, EventArgs e)
    {
        //do something
    }
}

现在最初我找到了This link，其中说实现IDisposable 并添加Dispose 方法：

public void Dispose()
{
   foreach (var obj in MyObjects)
   {
       obj.MySampleEvent -= Obj_MySampleEvent;
   }
}

但是当我想到时，这并没有被调用。它似乎是不稳定的，有时甚至根本没有打电话？所以我决定搜索“什么时候被调用”，这导致我找到this link，解释说Dispose被Finaliser/Destructor调用

让我进行最后一项研究的原因是，我记得有人说不要在 Destructor 中取消挂钩事件处理程序，因为它永远不会被从 this link 调用。

所以我只是想最后澄清一下。在 ViewModel 中解开事件处理程序的正确方法是什么？

Answer 1

我通常在导航期间在我的视图模型上取消注册事件处理程序。

例如，当引发 OnNavigatedFrom 事件时（在您的视图上），您可以取消注册当前视图模型上的事件处理程序。然后，当引发 OnNavigatedTo 事件时，您可以重新注册事件处理程序。

关于 IDisposable，我不确定。 我认为 IDisposable 是用于管理资源而不是业务逻辑。

Answer 2

我是否建议将 ViewModel 转换为 ObservableCollection 的原因是要绑定一些东西。然后，您可以直接在 ViewModel 属性上订阅任何事件，这将允许您订阅 Window_Closing 并在该事件中进行清理。

class MyViewModel
{
    private ObservableCollection<MyObject> _myObjects;

    public ObservableCollection<MyObject> MyObjects
    {
        get { return _myObjects; }
        set { _myObjects = value; }
    }

    public MyViewModel()
    {
        for (int i = 0; i < 10; i++)
        {
            var obj = new MyObject();
        }
    }
}