取消钩子之后，C＃仍然迷上了事件

如何解决取消钩子之后，C＃仍然迷上了事件

| 我目前正在调试一个包含内存泄漏的大型C＃应用程序。它主要使用Winforms作为GUI，尽管有一些控件是在WPF中制作的，并由ElementHost托管。到现在为止，我发现许多内存泄漏是由于事件未被取消（通过调用-=）引起的，并且我已经能够解决问题。但是，我遇到了一个类似的问题。有一个名为WorkItem的类（短寿命），该类在构造函数中注册到另一个名为ClientEntityCache的类的事件（长寿命）。这些事件从未解开，我可以在.NET探查器中看到WorkItem实例由于这些事件而不应该保持活动状态。因此，我决定使WorkItem实现IDisposable，并在Dispose（）函数中以以下方式取消挂接事件：

public void Dispose()
{
  ClientEntityCache.EntityCacheCleared -= ClientEntityCache_CacheCleared;
  // Same thing for 10 other events
}

编辑这是我用于订阅的代码：

public WorkItem()
{
  ClientEntityCache.EntityCacheCleared += ClientEntityCache_CacheCleared;
  // Same thing for 10 other events
}

我还更改了注销代码，以不调用新的EntityCacheClearedEventHandler。编辑结束我在使用WorkItem的代码中的适当位置进行了Dispose的调用，并且在调试时可以看到该函数确实已被调用，并且对每个事件都执行-=。但是我仍然遇到内存泄漏，工作项在被Dispose之后仍然保持活动状态。在.NET探查器中，我可以看到实例保持活动状态，因为事件处理程序（例如EntityCacheClearedEventHandler）仍将它们保留在调用列表中。我试图不止一次地将它们摘钩（多个-=），只是为了确保它们没有被超过一次的钩钩，但这无济于事。任何人都知道为什么会发生这种情况，或者我可以做些什么来解决问题？我想我可以将事件处理程序更改为使用弱委托，但是这将使大量的旧代码陷入混乱。谢谢！编辑：如果有帮助，这是.NET探查器描述的根路径：很多事情都指向ClientEntityCache，它指向EntityCacheClearedEventHandler，它指向Object []，它指向EntityCacheClearedEventHandler的另一个实例（我不明白为什么），它指向WorkItem。

解决方法

可能是多个不同的委托函数连接到事件。希望下面的小例子可以使我的意思更清楚。

// Simple class to host the Event
class Test
{
  public event EventHandler MyEvent;
}

// Two different methods which will be wired to the Event
static void MyEventHandler1(object sender,EventArgs e)
{
  throw new NotImplementedException();
}

static void MyEventHandler2(object sender,EventArgs e)
{
  throw new NotImplementedException();
}


[STAThread]
static void Main(string[] args)
{
  Test t = new Test();
  t.MyEvent += new EventHandler(MyEventHandler1);
  t.MyEvent += new EventHandler(MyEventHandler2); 

  // Break here before removing the event handler and inspect t.MyEvent

  t.MyEvent -= new EventHandler(MyEventHandler1);      
  t.MyEvent -= new EventHandler(MyEventHandler1);  // Note this is again MyEventHandler1    
}

如果在删除事件处理程序之前中断了操作，则可以在调试器中查看调用列表。如下所示，有2个处理程序，一个用于MyEventHandler1，另一个用于方法MyEventHandler2。现在，在两次删除MyEventHandler1之后，仍会注册MyEventHandler2，因为只剩下一个委托，它看起来有些不同，因此不再显示在列表中，但是在删除MyEventHandler2的委托之前，该事件仍将引用它。。 ,取消挂接事件时，它必须是同一委托人。像这样：

public class Foo
{
     private MyDelegate Foo = ClientEntityCache_CacheCleared;
     public void WorkItem()
     {
         ClientEntityCache.EntityCacheCleared += Foo;
     }

     public void Dispose()
     {
         ClientEntityCache.EntityCacheCleared -= Foo;
     }
}

原因是，您为此使用的是语法糖：

public class Foo
{
     public void WorkItem()
     {
         ClientEntityCache.EntityCacheCleared +=
new MyDelegate(ClientEntityCache_CacheCleared);
     }

     public void Dispose()
     {
         ClientEntityCache.EntityCacheCleared -=
new MyDelegate(ClientEntityCache_CacheCleared);
     }
}

因此，-=不会解开您所订阅的原始代表，因为他们是不同的代表。 ,您是否正在寻找正确的参考？当您使用ѭ5进行摘机时，不会产生任何错误，并且如果您取消摘录的事件，则不会发生任何错误。但是，如果您使用ѭ7进行添加，并且该事件已被挂接，则会收到错误消息。现在，这只是诊断问题的一种方法，而是尝试添加事件，如果您没有收到错误，则问题在于您使用错误的引用取消了事件的处理。 ,如果实例被事件处理程序保持活动状态，则GC不会调用Dispose，因为该实例仍被事件源引用。如果您自己调用了Dispose方法，则引用将超出范围。 ,也许尝试：

 public void Dispose()
    {
      ClientEntityCache.EntityCacheCleared -= ClientEntityCache_CacheCleared;
      // Same thing for 10 other events
    }

您正在创建一个新的事件处理程序并将其从ѭ9中删除-这实际上什么也没做。通过删除对原始订阅事件方法的引用来删除事件订阅。您总是可以将ѭ10设置为10，我认为那会比ѭ11更好。

取消钩子之后，C＃仍然迷上了事件

如何解决取消钩子之后，C＃仍然迷上了事件

解决方法

相关推荐