¿El uso de lambdas como controladores de eventos causa una pérdida de memoria?

Question

Digamos que tenemos el siguiente método:

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(); 
} 

Si la clase con este método se crea una instancia y el método PotentialMemoryLeaker se llama varias veces, ¿tenemos pérdida de memoria?

¿Hay alguna manera de desenganchar ese controlador de eventos lambda después de que terminemos de llamar al MethodThatFiresAnEvent?

Answer 1

Sí, guárdelo en una variable y desengánelo.

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

Y sí, si no lo hace, se le hay fugas de memoria, como podrá conectar un nuevo objeto delegado cada vez. También te darás cuenta de esto porque cada vez que llamas a este método, irá a la consola un número creciente de líneas (no solo un número creciente, sino que por una llamada a MethodThatFiresAnEvent arrojará cualquier cantidad de elementos, una vez por cada método anónimo conectado).

Answer 2

Sí, de la misma manera que los controladores de eventos normales pueden causar fugas. Debido a que el lambda es en realidad cambiado a:

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

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

Así que básicamente es sólo corta la mano por lo que hemos estado utilizando en todos estos años 2.0.

Answer 3

No solo perderá memoria, también recibirá su lambda llamada varias veces. Cada llamada de 'PotentialMemoryLeaker' agregará otra copia de la lambda a la lista de eventos, y se llamará a cada copia cuando se active 'AnEvent'.

Answer 4

Su ejemplo simplemente se compila en una clase interna privada con el nombre de compilador (con el campo firedCount y un método con nombre de compilador). Cada llamada a PotentialMemoryLeaker crea una nueva instancia de la clase de cierre a la que foo guarda una referencia por medio de un delegado al único método.

Si no hace referencia al objeto completo que posee PotentialMemoryLeaker, entonces todo será basura. De lo contrario, puede establecer foo en nulo o evento lista manejador de foo vacío escribiendo esto:

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

Por supuesto, que había necesidad de acceso a los miembros privados del MiObjeto de clase.

Answer 5

bien que se puede extender lo que se ha hecho para hacer here delegados más seguro de usar (no hay pérdidas de memoria)