使用 lambda 作为事件处理程序会导致内存泄漏吗?
-
08-06-2019 - |
题
假设我们有以下方法:
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
?
解决方案
是的,将其保存到变量中并取消挂钩。
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 或清空:
foreach (var handler in AnEvent.GetInvocationList()) AnEvent -= handler;
当然,您需要访问 我的对象 类的私有成员。
是的,就像普通事件处理程序可能导致泄漏一样。因为 lambda 实际上改为:
someobject.SomeEvent += () => ...;
someobject.SomeEvent += delegate () {
...
};
// unhook
Action del = () => ...;
someobject.SomeEvent += del;
someobject.SomeEvent -= del;
所以基本上它只是我们这些年来在 2.0 中使用的简写。
不隶属于 StackOverflow