Discussion :

[giova_fr]

Bonjour
tentant de débuger une fuite mémoire dans mon logiciel, je crois que je suis en train de découvrir une chose qui me térrifie (car je ne l'ai absolument pas gérée).

J'ai l'impression que si mon objet s'abonne à l'event de quelque chose de persistant. Alors mon objet ne sera jamais détruit.

exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
public class MaClasse1()
{
   public MaClasse1(IContent obj)
   {
         obj.UnEvent +=new EventHandler(Test);
   }
 
   private void Test(object sender,EventArg e)
  {
     //blablabla
  }
 
}
 
public class Program
{
    public static void Main()
   {
       IContent content = new Content();
       MaClasse1 obj = new MaClasse1(content);
       obj = null; // Bien que obj etait l'unique référent l'instance n'est pas détruite.
       //Plein de choses se passent, pour moi l'instance de MaClasse1 n'est plus, et pourtant : 
      content.DeclencheEvent(); //Fait que l'ex obj recoit l'event alors qu'il est cencé etre détruit ...
 
   }
}

Pourriez vous confirmer cette théorie?

Merci

[giova_fr]

Arg ! le couperet vient de tomber, j'ai trouvé la réponse sur un autre site :

publisher.SomeEvent += target.DoSomething;

then publisher has a reference to target but not the other way round.
[...]
The tricky case is when the publisher is long-lived but the subscribers don't want to be - in that case you need to unsubscribe the handlers.

la réponse est donc oui, ca empêche la destruction car celui qui publie l'événement garde du coup une référence vers l'abonné

C'est vrai que ca parrait logique apres tout.

Du coup je trouve que Visual Studio nous pousse vers un TRES mauvais reflexe :
Lorsque l'on s'abonne à un événement, Visual nous propose de créer la callback pour nous : "Press Tab to generate new EventHandler..."

C'est très mauvais car on n'a pas la main pour nous désabonner par la suite. Il aurait été préférable qu'il crée dans la classe, une variable EventHandler que l'on pourrait par la suite utiliser pour se désabonner. Même ajouter dans la Méthode Dispose un désabonnement n'aurait pas été un luxe !

Bon et bien je vais en avoir pour des jours à décrasser mon code

[BenoitM]

Tu peux faire une recherche sur "memory leak with event" dans google, j'ai vu aussi il y a un petit temps post sur le sujet si tu es réellement interresé par la problématique

[micka132]

Le "Tab" c'est fait pour aller plus vite. Maintenant il est toujours possible de faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
toto.click += new eventhandler(xxxx) // Abonnement
toto.click -= new eventhandler(xxxx) //Desabonnement

Tu peux par exemple implémenter l'interface IDisposable et dans le Dispose mettre tout les désabonnements.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
public static void Main()
   {
       IContent content = new Content();
       MaClasse1 obj = new MaClasse1(content);
obj.Dispose();
       obj = null; // Bien que obj etait l'unique référent l'instance n'est pas détruite.
       //Plein de choses se passent, pour moi l'instance de MaClasse1 n'est plus, et pourtant : 
      content.DeclencheEvent(); //Fait que l'ex obj recoit l'event alors qu'il est cencé etre détruit ...
 
   }

[giova_fr]

toto.click -= new eventhandler(xxxx) //Desabonnement

es tu sur de ca?
parceque qui dit new, dit une nouvelle instance d'objet. Je serai donc étonné que ca désabonne...

[tomlev]

Du coup je trouve que Visual Studio nous pousse vers un TRES mauvais reflexe :
Lorsque l'on s'abonne à un événement, Visual nous propose de créer la callback pour nous : "Press Tab to generate new EventHandler..."

C'est très mauvais car on n'a pas la main pour nous désabonner par la suite. Il aurait été préférable qu'il crée dans la classe, une variable EventHandler que l'on pourrait par la suite utiliser pour se désabonner. Même ajouter dans la Méthode Dispose un désabonnement n'aurait pas été un luxe !

C'est pas un problème en fait... quand tu fais ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

publisher.SomeEvent += target.DoSomething;

C'est en fait un raccourci pour

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

publisher.SomeEvent += new EventHandler(target.DoSomething);

(conversion implicite de groupe de méthode en délégué)

Et si tu fais ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

publisher.SomeEvent -= new EventHandler(target.DoSomething);

Tu utilises effectivement une nouvelle instance de EventHandler pour te désabonner, mais comme elle est "égale" (même cible et même méthode) à celle que tu as utilisé pour l'abonnement, ça fonctionne quand même

[giova_fr]

ok merci pour cet éclairsissement.

C'est génial, mais il y a quand meme une chose que je ne comprend pas trop.
ok la cible est la même.

Mais il y a quand même un objet qui contient cette cible (un Delegate?)
comment un "new" peut il retourner une référence unique?

[tomlev]

comment un "new" peut il retourner une référence unique?

Il ne retourne pas une référence unique, mais deux instances peuvent être considérées comme identiques (en redéfinissant la méthode Equals). C'est le même principe que quand tu retires un objet d'une liste : ça retire le premier objet qui est considéré comme égal à l'objet spécifié.

Soit dit en passant, il y a au moins un cas où l'opérateur new renvoie une instance existante au lieu d'en créer une nouvelle :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
    string s1 = new string('x', 0);
    string s2 = new string('x', 0);
    bool sameInstance = ReferenceEquals(s1, s2); // true

Mais c'est juste une optimisation du CLR pour ne pas multiplier les instances de chaine vide...

[giova_fr]

Je ressort grandi de cette discution encore merci !

PS : la boutique de free mobile toujours fermée, 4 ans que je l'attends, je n'en peux plus !

[chrisdot]

Si c'est en WPF, il y a moyen d'utiliser le weak event pattern
Si c'est du Silverlight, tu trouveras des implémentations du pattern sur le web.

A+