使用 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 方法被多次调用，是否会泄漏内存？

在我们完成调用后，有什么方法可以取消 lambda 事件处理程序 MethodThatFiresAnEvent?

Answer 1

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

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

是的，如果你不这样做，你就会 泄露 内存，因为您每次都会连接一个新的委托对象。您还会注意到这一点，因为每次调用此方法时，它都会向控制台转储越来越多的行（不仅仅是越来越多的行，而且对于 MethodThatFiresAnEvent 的一次调用，它会转储任意数量的项目，一次每个连接的匿名方法）。

Answer 2

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

Answer 3

好吧，你可以扩展已经完成的事情 这里 使委托使用起来更安全（无内存泄漏）

Answer 4

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

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

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

当然，您需要访问 我的对象 类的私有成员。

Answer 5

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

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

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

所以基本上它只是我们这些年来在 2.0 中使用的简写。