当前位置导航:炫浪网>>网络学院>>网页制作>>ASP.NET教程

未释放事件Handler可能导致内存泄漏

欢迎进入.NET社区论坛,与200万技术人员互动交流 >>进入

  以前曾看见过这样一个问题:托管代码会不会导致内存泄漏。自己对GC的了解也不是很深,但还是比较赞成这样的观点:托管代码不会产生内存泄漏,除非你没有正确释放非托管资源。

  今天看到一个非常有趣的例子,关于没有释放事件的Handler导致的内存泄漏。

  以前对于释放Handler的观念是一点也没有,这主要因为没此方面的意识,没有养成好的习惯。只知道当关心这个事件的时候就注册一下, 暂时不关心了就移除掉。却从来没有想到最终不移除不必要的Handler会导致此类无法被正常回收,导致不必要的内存浪费。

  事情是这样的,今天在看项目Source Code的时候发现一个有趣的字眼:"WeakEvent". 自己以前对WeakReference有点了解,所以就好奇地看看这是个啥玩意。

  发现其是一种通过弱引用实现的Delegate.因为没有太多的注释,所有不知其为啥用此种方式来封装事件。于是顺手Google了一下,找到了一篇关于weak event的非常有意思的文章。

  文章里提出了一个问题,场景如下:

  UnRelease Event Handler

  using System;

  using System.Collections.Generic;

  using System.Text;

  using Microsoft.Win32;

  namespace ConsoleApplication16

  {

  class DisplaySettingsListener

  {

  byte[] m_ExtraMemory = new byte[1000000];

  public DisplaySettingsListener()

  {

  SystemEvents.DisplaySettingsChanged += new EventHandler(ehDisplaySettingsChanged);

  }

  private void ehDisplaySettingsChanged(object sender, EventArgs e)

  {

  }

  }

  class Program

  {

  static void DisplayMemory()

  {

  Console.WriteLine("Total memory: {0:###,###,###,##0} bytes", GC.GetTotalMemory(true));

  }

  static void Main()

  {

  DisplayMemory();

  Console.WriteLine();

  for (int i = 0; i < 5; i++)

  {

  Console.WriteLine("—— New Listener #{0} ——", i + 1);

  DisplaySettingsListener listener = new DisplaySettingsListener();

  listener = null;

  GC.Collect();

  DisplayMemory();

  }

  Console.Read();

  }

  }

  }

  运行的结果如下:

 

  虽然我们释放了对listener的引用,并且强制GC进行回收,但我们可以看到其内存占用量还是变大了,出乎了我的意料。

欢迎进入.NET社区论坛,与200万技术人员互动交流 >>进入

  这就是该文作者指出的事件列表里保存的是一个强命名的引用而非弱引用。虽然上面释放了listener变量对Listener实例的引用,但因为仍然在DisplaySettingsChanged事件列表里保存了对Listener实例的引用,导致Listener实例并不能被垃圾回收(有人引用,自然不会回收)。

  那么接下来看看下面的代码:

  Release Event Hanlder

  class DisplaySettingsListener : IDisposable

  {

  byte[] m_ExtraMemory = new byte[1000000];

  public DisplaySettingsListener()

  {

  SystemEvents.DisplaySettingsChanged += new EventHandler(ehDisplaySettingsChanged);

  }

  private void ehDisplaySettingsChanged(object sender, EventArgs e)

  {

  }

  IDisposable Members#region IDisposable Members

  public void Dispose()

  {

  SystemEvents.DisplaySettingsChanged -= new EventHandler(ehDisplaySettingsChanged);

  }

  #endregion

  }

  class Program

  {

  static void DisplayMemory()

  {

  Console.WriteLine("Total memory: {0:###,###,###,##0} bytes", GC.GetTotalMemory(true));

  }

  static void Main()

  {

  DisplayMemory();

  Console.WriteLine();

  for (int i = 0; i < 5; i++)

  {

  Console.WriteLine("—— New Listener #{0} ——", i + 1);

  DisplaySettingsListener listener = new DisplaySettingsListener();

  listener.Dispose();

  listener = null;

  GC.Collect();

  DisplayMemory();

  }

  Console.Read();

  }

  }

  运行结果如下:

 

  结果是不是正如您猜测的呢:)。已经成功地回收了listener实例。 不知为何从432944字节变到446980字节,哪位高手赐教一下啊:)

  详情可以看原文 The Problem With Delegates

  在后续的文章中作者类似文章开头提到的Weak Event来解决这个问题: Solving the Problem with Events: Weak Event Handlers

  也许您觉得写这样的一个Weak Event没有必要或者显得麻烦,但您一定要记得及时地在必要的地方调用 -= 取消不再关心的事件。本文的目的也只是在此方面提个善意的提醒。

相关内容
赞助商链接