将Lambda用作事件处理程序会导致内存泄漏吗？

Question

假设我们有以下方法：

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事件处理程序？

- user1228

---

如下所示，根据下面的答案，没有办法在不保留引用的情况下取消挂钩。然而，你可以让它自己取消挂钩：https://dev59.com/JnI-5IYBdhLWcg3wpaJ1#1747236 - Benjol

Answer 1

是的，将其保存到变量中并取消挂钩。

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

没错，如果你不这样做，你的内存将会泄漏，因为每次都会连接一个新的代理对象。你也会注意到每次调用此方法时，控制台会输出逐渐增加的行数（不仅仅是递增的数量，而且对于每个连接的匿名方法，它将为MethodThatFiresAnEvent转储任意数量的项，每个方法都会输出一次）。

- lassevk

Answer 2

你不仅会泄漏内存，还会使您的lambda被多次调用。每次调用“PotentialMemoryLeaker”都会向事件列表中添加lambda的另一个副本，并且当“AnEvent”被触发时，每个副本都将被调用。

- Wilka

Answer 3

你可以扩展这里做的事情，使委托更安全易用（没有内存泄漏）。

- Vaibhav

Answer 4

你的示例只是编译为一个编译器命名的私有内部类（带有firedCount字段和一个编译器命名的方法）。每次调用PotentialMemoryLeaker都会创建该闭包类的新实例，其中foo通过对单个方法的委托保留引用。

如果您不引用拥有PotentialMemoryLeaker的整个对象，那么所有内容都将被垃圾回收。否则，您可以将foo设置为null，或通过写入以下内容来清空foo的事件处理程序列表：

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

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

- marxidad

Answer 5

是的，就像普通事件处理程序可能会导致泄漏一样。因为lambda实际上被改变为：

someobject.SomeEvent += () => ...;
someobject.SomeEvent += delegate () {
    ...
};

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

基本上，这只是我们在2.0版本中多年来一直在使用的简写。

- Nick Berardi