Discussion :

[Nico28]

Bonjour à tous.


Dans le code suivant,

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
private void Form1_Load(object sender, EventArgs e)
{
       listBox1.Items.Add("aaaaa");
       listBox1.Items.Add("bbbb");
       listBox1.Items.Add("cccc");
       listBox1.SelectedIndex = 1;
}
 
private void listBox1_SelectedIndexChanged(object sender, EventArgs e)
{
        MessageBox.Show("Activé");
}

la ligne "listBox1.SelectedIndex = 1;" a pour effet de déclencher la méthode "listBox1_SelectedIndexChanged". Normal !!

Mais si je ne veux pas que cela arrive, comment faire ? on peut se débrouiller avec un quelconque flag mais n'y a t il pas un moyen plus "propre" ?

Je pensais que le "SuspendLayout" et le "ResumeLayout" auraient pu m'aider mais non !

Avez-vous des idées ?

[tomlev]

on peut se débrouiller avec un quelconque flag mais n'y a t il pas un moyen plus "propre" ?

Pas vraiment... tu peux te désabonner temporairement de l'évènement et te réabonner ensuite, mais c'est pas vraiment plus pratique

[Bluedeep]

Avez-vous des idées ?

Tout sîmplement (dans le cas que tu donnes) abonner tes événements en fin de Form_load plutôt que dans le constructeur ou dans le InitializeComponent

Mais, sinon, non, il n'existe pas de méthode externe à une instance permettant de lui dire de ne pas "lever" ses évenements.

[Graffito]

Comme l'indique tomlev, encadres le changement d'index par une déactivation/réactivation de l'event :

Code :

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

1
2
3
4
 
listBox1.SelectedIndexChanged -= listBox1_SelectedIndexChanged ;
listBox1.SelectedIndex = 1;
listBox1.SelectedIndexChanged += listBox1_SelectedIndexChanged ;

Remarque : la désactivation d'un event ne gènére pas d'exception si l'event n'est pas encore activé ou a déjà été désactivé.

[Nico28]

Merci de vos réponses.

J'avais pensé à cela mais il faut effectivement désabonner TOUS les évenements qui me gêne.

J'aurais aimé une technique globale de déactivation de tous les événements pour l'objet considéré.


En fait, le problème se pose souvent à l'initialisation d'un formulaire, lorsqu'on configure/met à jour les différentes listes, cases à cocher et autres boutons radio. Les événements sont générés en cascade et on obtient parfois des situations incohérentes (à l'intérieur du programme).

[tomlev]

J'aurais aimé une technique globale de déactivation de tous les événements pour l'objet considéré.

Ca n'existe pas... Certains composants ont une propriété EnableRaisingEvents, mais ce n'est pas le cas des contrôles

[micka132]

Ben si c'est un problème récurrent lors de l'initialisation tu peux toujours mettre un boolean et dans tout tes events tu tests cette variable.
Quand tu estimes que les initialisations sont finis tu changes le boolean.

Bon ca marche que dans un contexte "d'initialisation", et les evenements sont quand meme appelé...

[Nico28]

Ben si c'est un problème récurrent lors de l'initialisation tu peux toujours mettre un boolean et dans tout tes events tu tests cette variable.
Quand tu estimes que les initialisations sont finis tu changes le boolean.

C'est ce qui est fait pour le moment.

[ctxnop]

Ca n'existe pas... Certains composants ont une propriété EnableRaisingEvents, mais ce n'est pas le cas des contrôles

Au pire ça s'implémente, il suffit de surcharger le contrôle et ses méthodes On[...] afin de ne pas raise l'event si la propriété est à true.

[tomlev]

Au pire ça s'implémente, il suffit de surcharger le contrôle et ses méthodes On[...] afin de ne pas raise l'event si la propriété est à true.

Oui mais bon, s'il faut se faire une version customisée de chaque contrôle standard, c'est un peu galère...

[ctxnop]

Bah, ça n'est à faire qu'une fois, et seulement pour les contrôles pour lesquels ont veut la fonctionnalité. C'est pas bien la mort et ça évite de refaire la même astuce à chaque fois qu'on en à besoin.
Ceci-dit, je trouve très con de la part de Microsoft de ne pas avoir intégré ça en standard dans les winforms.

[Graffito]

Ceci-dit, je trouve très con de la part de Microsoft de ne pas avoir intégré ça en standard dans les winforms.

On peut créer une fonction statique récursive pour sauvegarder/désactiver les events de chaque control et de ses enfants. On passera à cette fonction le control (la form au départ) et elle mettra à jour un objet contenant la sauvegarde.

Une deuxième procédure statique avec comme paramètre d'entrée l'objet de sauvegarde permettra de restaurer les events sauvegardés.

On pourra se baser sur ce lien pour les fonctions permettant de supprimer/restaurer un event ou tous les events d'un control:
http://www.frafru.info/index.php?db=so&id=91778

[ctxnop]

C'est ce que j'appelle une usine à gaz.
1. Tu ne peux pas choisir unitairement les contrôles.
2. Tu ne peux pas choisir unitairement l'évènement à désactiver.
3. Tu boucle récursivement sur une arborescence de contrôle qui peut être TRES grosse, contenant également les contrôles internes (genre la scrollbar d'un panel trop grand pour son parent), et donc c'est potentiellement bien lent.
4. Tu te sert de la reflexion pour retrouver la liste des events, ce qui est très lent (sans compter le manque d'optimisation vis à vis du fait si tu fais deux System.Windows.Forms.Button à la suite, tu refais le même code de reflexion très lent)

Alors a moins que ce soit justifié par le fait qu'il ait besoin de désactiver tous les évents, souvent, à pleins d'endroits différents, quelque soit le contrôle, la solution d'origine (celle du flag) me parait plus adaptée.

[Graffito]

C'est ce que j'appelle une usine à gaz.

C'est aussi mon avis, mais je voulais juste indiquer que c'était faisable.