Discussion :

[penchu]

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 :

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
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 :

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

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

[guitoux1]

Code :

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

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

[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.

[penchu]

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

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
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?

[guitoux1]

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 :

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

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

[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