使用lambda作为事件处理程序会导致内存泄漏吗?

使用lambda作为事件处理程序会导致内存泄漏吗?

Can using lambdas as event handlers cause a memory leak?

说我们有以下方法:

1
2
3
4
5
6
7
8
9
private MyObject foo = new MyObject();

// and later in the class

public void PotentialMemoryLeaker(){
  int firedCount = 0;
  foo.AnEvent += (o,e) => { firedCount++;Console.Write(firedCount);};
  foo.MethodThatFiresAnEvent();
}

如果使用此方法的类被实例化并且多次调用PotentialMemoryLeaker方法,我们是否泄漏内存?

在调用MethodThatFiresAnEvent之后,有什么方法可以解除该lambda事件处理程序的挂接?


是的,将其保存到变量中并解钩。

1
2
3
4
DelegateType evt = (o, e) => { firedCount++; Console.Write(firedCount); };
foo.AnEvent += evt;
foo.MethodThatFiresAnEvent();
foo.AnEvent -= evt;

是的,如果不这样做,则会泄漏内存,因为每次都会连接一个新的委托对象。您还会注意到这一点,因为每次调用此方法时,它都会将越来越多的行转储到控制台(不仅是增加的行数,而且对于MethodThatFiresAnEvent的一次调用,它将转储任意数目的项目,一次用于每个都连接了匿名方法)。


您不仅会泄漏内存,还会多次调用lambda。每次调用" PotentialMemoryLeaker"都会将lambda的另一个副本添加到事件列表中,并且在触发" AnEvent"时将调用每个副本。


好了,您可以扩展此处执行的操作,以使委托使用起来更安全(不存在内存泄漏)


您的示例仅编译为以编译器命名的内部类(带有字段firedCount和以编译器命名的方法)。每次对PotentialMemoryLeaker的调用都会创建一个闭包类的新实例,其中foo通过对单个方法的委托保留引用。

如果您不引用拥有PotentialMemoryLeaker的整个对象,则将全部垃圾回收。否则,可以通过编写以下命令将foo设置为null或清空foo的??事件处理程序列表:

1
foreach (var handler in AnEvent.GetInvocationList()) AnEvent -= handler;

当然,您需要访问MyObject类的私有成员。


是,就像普通事件处理程序可能导致泄漏一样。因为lambda实际上更改为:

1
2
3
4
5
6
7
8
9
someobject.SomeEvent += () => ...;
someobject.SomeEvent += delegate () {
    ...
};

// unhook
Action del = () => ...;
someobject.SomeEvent += del;
someobject.SomeEvent -= del;

因此,基本上,这只是这些年来我们在2.0中一直使用的简写方式。


推荐阅读