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

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

您可以扩展已完成的工作here以使代理更安全（没有内存泄漏）

Answer 4

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

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

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

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

Answer 5

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

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

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

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

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

5 个答案: