可以使用lambda作为事件处理程序导致内存泄漏吗?
时间:2020-03-05 18:40:47 来源:igfitidea点击:
说我们有以下方法:
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事件处理程序?
解决方案
回答
是的,将其保存到变量中并解钩。
DelegateType evt = (o, e) => { firedCount++; Console.Write(firedCount); }; foo.AnEvent += evt; foo.MethodThatFiresAnEvent(); foo.AnEvent -= evt;
是的,如果不这样做,则会泄漏内存,因为每次都将连接一个新的委托对象。我们还会注意到这一点,因为每次调用此方法时,它都会将越来越多的行转储到控制台(不仅是增加的行数,而且对于MethodThatFiresAnEvent的一次调用,它将转储任意数目的项目,一次用于每个都连接了匿名方法)。
回答
是,就像普通事件处理程序可能导致泄漏一样。因为lambda实际上更改为:
someobject.SomeEvent += () => ...; someobject.SomeEvent += delegate () { ... }; // unhook Action del = () => ...; someobject.SomeEvent += del; someobject.SomeEvent -= del;
因此,基本上,这只是这些年来我们在2.0中一直使用的简写形式。
回答
我们不仅会泄漏内存,还会多次调用lambda。每次调用" PotentialMemoryLeaker"都会将另一个lambda副本添加到事件列表中,并且在触发" AnEvent"时将调用每个副本。
回答
示例仅编译为以编译器命名的内部类(带有字段firedCount和以编译器命名的方法)。每次对PotentialMemoryLeaker的调用都会创建一个闭包类的新实例,其中foo通过对单个方法的委托来保留对其的引用。
如果我们不引用拥有PotentialMemoryLeaker的整个对象,则将全部垃圾回收。否则,可以通过编写以下命令将foo设置为null或者清空foo的事件处理程序列表:
foreach (var handler in AnEvent.GetInvocationList()) AnEvent -= handler;
当然,我们需要访问MyObject类的私有成员。
回答
好了,我们可以扩展此处执行的操作,以使委托使用起来更安全(不存在内存泄漏)