20

假设我们有以下方法：

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？

4

5 回答 5

16

是的，将它保存到一个变量并解开它。

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

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

于 2008-08-19T16:08:24.663 回答
4

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

于 2008-08-19T16:11:30.227 回答
3

好吧，您可以扩展此处所做的工作，以使代表更安全地使用（无内存泄漏）

于 2008-08-19T16:31:26.020 回答
2

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

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

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

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

于 2008-08-19T16:19:24.453 回答
0

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

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

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

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

于 2008-08-19T16:09:37.283 回答