Méthode pour rendre un évênement paramétrable

Version imprimable

Bonjour à tous,

j'ai dans ma form 6 combo et 6 textbox qui reprennent un code couleur et un libellé couleur chacune.
j'ai mis à côté de chaque textbox, un bouton pour supprimer une couleur (en fait ce sont des boîtes sur lesquelles on peut mettre plusieurs couleurs, d'où les 6 combo et textbox).

Voici mon code pour mes boutons "supprimer" :

Code:

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
32
33
34
35
36
37
38
39
40
41
42
 
 private void button1_Click_1(object sender, EventArgs e)
        {
            rowCurrent = ((DataRowView)clichesBindingSource.Current).Row as Suivi_cliches_formes.Datasets.DS_cliches_formes.ClichesRow;
            iDCouleur1ComboBox.SelectedItem = null;
            ValeursComboCouleurs(iDCouleur1ComboBox, libelle_couleurTextBox1);
        }
 
        private void button2_Click(object sender, EventArgs e)
        {
            rowCurrent = ((DataRowView)clichesBindingSource.Current).Row as Suivi_cliches_formes.Datasets.DS_cliches_formes.ClichesRow;
            iDCouleur2ComboBox.SelectedItem = null;
            ValeursComboCouleurs(iDCouleur2ComboBox, libelle_couleurTextBox2);
        }
 
        private void button3_Click(object sender, EventArgs e)
        {
            rowCurrent = ((DataRowView)clichesBindingSource.Current).Row as Suivi_cliches_formes.Datasets.DS_cliches_formes.ClichesRow;
            iDCouleur3ComboBox.SelectedItem = null;
            ValeursComboCouleurs(iDCouleur3ComboBox, libelle_couleurTextBox3);
        }
 
        private void button4_Click(object sender, EventArgs e)
        {
            rowCurrent = ((DataRowView)clichesBindingSource.Current).Row as Suivi_cliches_formes.Datasets.DS_cliches_formes.ClichesRow;
            iDCouleur4ComboBox.SelectedItem = null;
            ValeursComboCouleurs(iDCouleur4ComboBox, libelle_couleurTextBox4);
        }
 
        private void button5_Click(object sender, EventArgs e)
        {
            rowCurrent = ((DataRowView)clichesBindingSource.Current).Row as Suivi_cliches_formes.Datasets.DS_cliches_formes.ClichesRow;
            iDCouleur5ComboBox.SelectedItem = null;
            ValeursComboCouleurs(iDCouleur5ComboBox, libelle_couleurTextBox5);
        }
 
        private void button6_Click(object sender, EventArgs e)
        {
            rowCurrent = ((DataRowView)clichesBindingSource.Current).Row as Suivi_cliches_formes.Datasets.DS_cliches_formes.ClichesRow;
            iDCouleur6ComboBox.SelectedItem = null;
            ValeursComboCouleurs(iDCouleur6ComboBox, libelle_couleurTextBox6);
        }

Comment pourrais-je faire pour rendre ça moins répétitif dans le code? Y a-t-il moyen de regrouper tout ça dynamiquement du genre :

Code:

1
2
3
4
5
6
7
8
9
10
 
for (int i= 0;i<=6;i++)
{
       private void button + i + _Click(object sender, EventArgs e)
        {
            rowCurrent = ((DataRowView)clichesBindingSource.Current).Row as Suivi_cliches_formes.Datasets.DS_cliches_formes.ClichesRow;
            iDCouleur + i + 6ComboBox.SelectedItem = null;
            ValeursComboCouleurs(iDCouleur + i + ComboBox, libelle_couleurTextBox + i + );
        }
}

D'avance merci

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 private void button_Click(object sender, EventArgs e)
        {
            rowCurrent = ((DataRowView)clichesBindingSource.Current).Row as Suivi_cliches_formes.Datasets.DS_cliches_formes.ClichesRow;
 
             if (sender is iDCouleur2ComboBox)
             {
                  iDCouleur2ComboBox.SelectedItem = null;
                  ValeursComboCouleurs(iDCouleur2ComboBox, libelle_couleurTextBox2);
              }
              else if (sender is iDCouleur3ComboBox)
               {
                ....
                }
           }

Et tu abonne l'événement click de tous tes boutons à cet méthode

16/02/2007, 12h31
penchu

Ok, mais du coup le designer va me générer le même évenement pour tout mes boutons, non? Ca risque pas de poser problème?
Enfin, je teste et je reviens avec mes résultats.

Merci.

Oui, ok, ça fonctionne d'une certaine façon.
Voici comment j'ai fait.

Code:

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
32
33
34
35
36
37
38
 
 private void button_supprimer_Click(object sender, EventArgs e)
        {
 
            rowCurrent = ((DataRowView)clichesBindingSource.Current).Row as Suivi_cliches_formes.Datasets.DS_cliches_formes.ClichesRow;
            switch (((Button)sender).Name)
            {
                case "button1":
                    iDCouleur1ComboBox.SelectedItem = null;
                    ValeursComboCouleurs(iDCouleur1ComboBox, libelle_couleurTextBox1, button1);
                    break;
                case "button2":
                    iDCouleur2ComboBox.SelectedItem = null;
                    ValeursComboCouleurs(iDCouleur2ComboBox, libelle_couleurTextBox2, button1);
                    break;
                case "button3":
                    iDCouleur3ComboBox.SelectedItem = null;
                    ValeursComboCouleurs(iDCouleur3ComboBox, libelle_couleurTextBox3, button1);
                    break;
                case "button4":
                    iDCouleur4ComboBox.SelectedItem = null;
                    ValeursComboCouleurs(iDCouleur4ComboBox, libelle_couleurTextBox4, button1);
                    break;
                case "button5":
                    iDCouleur5ComboBox.SelectedItem = null;
                    ValeursComboCouleurs(iDCouleur5ComboBox, libelle_couleurTextBox5, button1);
                    break;
                case "button6":
                    iDCouleur6ComboBox.SelectedItem = null;
                    ValeursComboCouleurs(iDCouleur6ComboBox, libelle_couleurTextBox6, button1);
                    break;
            }
            //if (sender == this.iDCouleur1ComboBox)
            //{
            //    iDCouleur1ComboBox.SelectedItem = null;
            //    ValeursComboCouleurs(iDCouleur1ComboBox, libelle_couleurTextBox1, button1);
            //}
        }

Mais ce qui serait bien, ce serait de pouvoir donner les noms des contrôles dynamiquement dans une boucle pour pouvoir n'avoir que 5-6 lignes de codes... Mais je sais pas trop comment.
QQn une idée?

Tu pourrais utiliser la propriété Tag de tes contrôle pour y stocker un identifiant (1, 2, 3, ....).
Ensuite, tu utilise la collection de controls du container contenant tes combox et tes textbox, puis tu fais :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 private void button_supprimer_Click(object sender, EventArgs e)
        {
 
            rowCurrent = ((DataRowView)clichesBindingSource.Current).Row as Suivi_cliches_formes.Datasets.DS_cliches_formes.ClichesRow;
            string idControl = ((Button)sender).Tag.ToString();
 
            ComboBox iDCouleurCombo = monContainer.Controls["iDCouleur" + idControl + "Container"];
             TextBox txtCouleur = monContainer.Controls["libelle_couleurTextBox" + idControl];
 
              iDCouleurCombo.SelectedItem = null;
              ValeursComboCouleurs(iDCouleurCombo, txtCouleur, (TextBox)sender);
 
        }

16/02/2007, 13h58
penchu

Ben voilà, ça c'est ce que j'appelle une réponse simple, rapide et efficace car tout fonctionne.
Je te remercie énormément pour ton aide.

Donc pour rappeler :

1. mettre un tag sur chacun de mes boutons "supprimer".
2. créer la méthode de suppression
3. ajouter un handler de cette méthode sur l'évênement click des boutons

Au plaisir