Discussion :

[Arno2xF]

Bonjour,

Suite à une création dynamique d'un événements sur un contrôle (en passant par un delegate), je souhaiterai pouvoir le supprimer dynamiquement.
Création de l'événement:

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
 
        private void InitialiseEvents()
        {
            // Suite au click, modifier le curseur
            this.m_image.MouseDown += delegate
            {
                this.m_image.Visible = false;
                Bitmap b = new Bitmap(GetImagePath());
                IntPtr s = b.GetHicon();
                Cursor c = new Cursor(s);
                this.m_image.Parent.Parent.Cursor = c;
            };
        }
        private void DisableEvents()
        {
             ?????????
        }

Est-ce possible ? En accédant à la collection d'EventHandler liés au contrôle (si ça existe) ?
Merci

[Guulh]

Je crois pas qu'il soit possible de désabonner une méthode anonyme.

=> Solution : que la méthode ne soit plus anonyme, ce qui sera pas difficile ici vu que la méthode en question n'utilise pas de variable de la méthode dans laquelle elle a été définie.

Par ailleurs, je connais pas le contexte de ce bout de code, mais il me parait un brin bourrin d'aller rechercher sur le disque le curseur à chaque MouseDown... tu devrais stocker le curseur dans une variable de ta classe, par exemple.

[Arno2xF]

Donc ça n'est pas possible
Je voulais éviter de stocker les délégués anonymes en variable membres (du coup, elles ne seront plus anonymes ^^)

Effectivement, l'exemple fourni est un peu hard mais c'était juste un projet test pour les délégués

Merci.

[Guulh]

De toutes façon, tu n'as pas vraiment besoin de désabonner. Il te suffit d'ajouter au début de ta méthode abonnée un "if (!activated) return;", où activated est une variable membre de ta classe, qu'il te suffit de modifier pour activer / désactiver.

[Arno2xF]

Bonne idée mais le pointeur de fonction ne sera pas détruit par le garbage collector

N'empêche que je suis surpris qu'il n'y ai pas moyen de retrouver tous ces délégués via un accesseur à partir du contrôle

[Guulh]

Bonne idée mais le pointeur de fonction ne sera pas détruit par le garbage collector

Loin de moi l'idée de jouer au tatillon, mais ce que tu dis est faux à plusieurs niveaux
Déjà, qu'un pointeur (ou plutôt une référence) soit détruit ou pas, on s'en fiche, puisque n'étant qu'une adresse, ça ne prend pas de place.

Ensuite, la méthode anonyme est transformée par le compilo en méthode d'instance de ta classe (puisqu'elle ne capture aucune variable locale de la méthode où elle est déclarée, sinon le compilo aurait dû créer un objet, lui coller les variables capturées, et faire de la méthode anonyme une méthode de cet objet).
Et donc, quand un objet A s'abonne à un objet B, ça ne crée pas une référence de A vers B (qu'en l'occurence tu as déjà, je suppose, puisque m_image est un membre de ta form ?), mais de B vers A. Il faut effectivement faire attention à désabonner des méthodes quand l'abonné a une durée de vie plus courte que l'émetteur, puisque l'émetteur garde une référence vers tous ses abonnés, et empêche donc le garbage collector de les collecter.

Donc là, de toutes façons, vu que la durée de vie de m_image est inférieure à celle de ta form, pas besoin de te désabonner. Et ce, pour des raisons de perf en tous cas, tu peux vouloir te désabonner pour que le code ne s'exécute plus, mais comme je te disais précédemment un bon gros booléen est plus simple dans ce cas.

Y'a peut être moyen de choper la collection de méthodes abonnées, mais perso j'en ai jamais eu besoin, et je pense partager ma situation avec 99.9% des participants à ce forum. En tous cas, clairement, vu le besoin que tu exprimes, t'en as pas besoin

[Graffito]

Y'a peut être moyen de choper la collection de méthodes abonnées

Oui, je crois que j'ai encorele bout de code quelque part (stocké en commentaire dans une bibli).
J'essaierai de le retrouver demain et de le poster.

[Graffito]

Voici le code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
RemoveEventHandlers(button1,"Click") ;
...
internal void RemoveEventHandlers(Control TheControl,string EventName)
{
  System.Reflection.FieldInfo f1 = typeof(Control).GetField("Event"+EventName,
       System.Reflection.BindingFlags.Static | System.Reflection.BindingFlags.NonPublic);
  object obj = f1.GetValue(TheControl); 
  System.Reflection.PropertyInfo pi = TheControl.GetType().GetProperty("Events",
       System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance);             
  EventHandlerList list = (EventHandlerList)pi.GetValue(TheControl, null);
  list.RemoveHandler(obj, list[obj]);
}

[MABROUKI]

bonjour
elle est disponible pour les classes controls dans Control.Events

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
28
29
30
31
 
 
// The class SampleControl defines two event properties, MouseUp and MouseDown.
class SampleControl: Component {
   // :
   // Define other control methods and properties.
   // :
 
   // Define the delegate collection.
   protected EventHandlerList listEventDelegates = new EventHandlerList();
 
   // Define a unique key for each event.
   static readonly object mouseDownEventKey = new object();
   static readonly object mouseUpEventKey = new object();
 
   // Define the MouseDown event property.
   public event MouseEventHandler MouseDown {  
      // Add the input delegate to the collection.
      add { listEventDelegates.AddHandler(mouseDownEventKey, value); }
      // Remove the input delegate from the collection.
      remove { listEventDelegates.RemoveHandler(mouseDownEventKey, value); }
   }
 
   // Define the MouseUp event property.
   public event MouseEventHandler MouseUp {
      // Add the input delegate to the collection.
      add { listEventDelegates.AddHandler(mouseUpEventKey, value); }
      // Remove the input delegate from the collection.
      remove { listEventDelegates.RemoveHandler(mouseUpEventKey, value); }
   }
}

bon code...............

[Arno2xF]

Parfait ! Merci à tous