Créer une classe dérivée d'un objet label

Version imprimable

Bonjour à tous,

Voilà je commence à comprendre l'utilité des classes...
Dans mon projet j'aimerais bien créer une classe car je fais une tâche répétitive mais je ne sais absolument pas comment m'y prendre car j'ai un objet label.

Voici mon code :

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
43
44
45
46
47
private void label1Joueur_Click(object sender, EventArgs e)
        {
            label1Joueur.BackColor = Color.Gray;
            label1Joueur.ForeColor = Color.White;
            label2Joueur.BackColor = default(Color);
            label2Joueur.ForeColor = default(Color);
            label3Joueur.BackColor = default(Color);
            label3Joueur.ForeColor = default(Color);
            label4Joueur.BackColor = default(Color);
            label4Joueur.ForeColor = default(Color);
        }
 
        private void label2Joueurs_Click(object sender, EventArgs e)
        {
            label2Joueur.BackColor = Color.Gray;
            label2Joueur.ForeColor = Color.White;
            label1Joueur.BackColor = default(Color);
            label1Joueur.ForeColor = default(Color);
            label3Joueur.BackColor = default(Color);
            label3Joueur.ForeColor = default(Color);
            label4Joueur.BackColor = default(Color);
            label4Joueur.ForeColor = default(Color);
        }
 
        private void label3Joueurs_Click(object sender, EventArgs e)
        {
            label3Joueur.BackColor = Color.Gray;
            label3Joueur.ForeColor = Color.White;
            label1Joueur.BackColor = default(Color);
            label1Joueur.ForeColor = default(Color);
            label2Joueur.BackColor = default(Color);
            label2Joueur.ForeColor = default(Color);
            label4Joueur.BackColor = default(Color);
            label4Joueur.ForeColor = default(Color);
        }
 
        private void label4Joueurs_Click(object sender, EventArgs e)
        {
            label4Joueur.BackColor = Color.Gray;
            label4Joueur.ForeColor = Color.White;
            label1Joueur.BackColor = default(Color);
            label1Joueur.ForeColor = default(Color);
            label2Joueur.BackColor = default(Color);
            label2Joueur.ForeColor = default(Color);
            label3Joueur.BackColor = default(Color);
            label3Joueur.ForeColor = default(Color);
        }

J'aimerais le simplifier en créant une classe
Merci d'avance le forum :D

Valentin.

18/04/2018, 12h03
WaterTwelve21
Bonjour VTT85,

C'est très bien de vouloir automatiser une tâche répétitive ( c'est un peu notre boulot :aie: ).

Le comportement commun içi c'est le click qui change les propriétés BackColor et ForeColor du label visée , et remet à zéro les autres.
Tu peux donc refactoriser en un seul événement click , c'est à dire tous les labels abonnés au même événement click.

Caster le sender te permet de récupérer le label visé , ainsi :
Code:

1 2 3 4 5 6 Label label = sender as Label; if (label != null) { // tu sais ce qu'il te reste à faire içi. }
Il te reste maintenant à modifier les couleurs du label , et remettre à défaut les autres.

Bon code!

Super merci beaucoup pour l'info. +1 :mrgreen:

J'ai donc fais ceci comme code pour également récupérer le nombre de joueurs:

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
private void labelJoueur_Click(object sender, EventArgs e)
        {
            int nbJoueurs = 0;
            //Initialisation des couleurs des labels
            label1Joueur.BackColor = default(Color);
            label1Joueur.ForeColor = default(Color);
            label2Joueur.BackColor = default(Color);
            label2Joueur.ForeColor = default(Color);
            label3Joueur.BackColor = default(Color);
            label3Joueur.ForeColor = default(Color);
            label4Joueur.BackColor = default(Color);
            label4Joueur.ForeColor = default(Color);
 
            Label label = sender as Label;
            if (label != null)
            {                
                label.BackColor = Color.Gray;
                label.ForeColor = Color.White;
            }
            if (label == label1Joueur)
            {
                nbJoueurs = 1;
            }
            if (label == label2Joueur)
            {
                nbJoueurs = 2;
            }
            if (label == label3Joueur)
            {
                nbJoueurs = 3;
            }
            if (label == label4Joueur)
            {
                nbJoueurs = 4;
            }
        }

Ceci je ne peux pas le factoriser plus ? :calim2:
Merci WaterTwelve21 :)

A quoi te sert le nombre de joueur ?

Le but de refactoriser c'est éviter cette partie :

Code:

1
2
3
4
5
6
7
8
 label1Joueur.BackColor = default(Color);
            label1Joueur.ForeColor = default(Color);
            label2Joueur.BackColor = default(Color);
            label2Joueur.ForeColor = default(Color);
            label3Joueur.BackColor = default(Color);
            label3Joueur.ForeColor = default(Color);
            label4Joueur.BackColor = default(Color);
            label4Joueur.ForeColor = default(Color);

Donc oui tu peux refactoriser plus que ça :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
  private void labelJoueurs_Click(object sender, EventArgs e)
        {
            Label label = sender as Label;
 
            if (label != null)
            {
                label.BackColor = Color.Gray;
                label.ForeColor = Color.White;
 
                ToogleOtherLabels(label);
                // => remettre a default(Color) les autres labels , en paramétre le label visé pour ne pas toucher à celui visé.
               // tu peux boucler sur les contrôles de ta form en bouclant sur this.Controls
               // à toi de coder cette le contenu de cette méthode.
 
            }
        }

18/04/2018, 13h12
VTT85

J'ai besoin de récupérer le nombre de joueurs pour ensuite mettre un menu différents selon le nombre de joueurs.

Je ne vois vraiment pas quoi mettre dans la méthode ToogleOtherLabels(label)...
Comment est-il possible de récupérer tous les autres labels en fonction de celui visé pour les faire appliquer backcolor et forecolor ?

Désolé je débute :(

Cela peut être plus optimisé pour éviter de relire à chaque fois l'intégralité des controls mais dans l'idée c'est ca :

Code:

1
2
3
4
5
6
7
8
9
10
11
 private void ToogleOtherLabels(Label label)
        {
            foreach (Control c in this.Controls)
            {
                if(c is Label && !c.Name.Equals(label.Name) && (c.Name.Contains("Joueur")))
                {   
                   c.BackColor = default(Color);
                   c.ForeColor  = default(Color);
                }
            }
        }

18/04/2018, 13h37
VTT85

C'est gentil merci beaucoup :)

Je ne connaissais pas les fonctions Control et In..

J'ai juste une petite question, est-il vraiment utile de vérifier que c soit un label au début du if ?

Merci en tout cas :D
18/04/2018, 14h03
WaterTwelve21

Citation:

Envoyé par VTT85

J'ai juste une petite question, est-il vraiment utile de vérifier que c soit un label au début du if ?

Oui parce que tu peux avoir toute sortes de controls , des boutons , des panels , des combobox etc ... .
18/04/2018, 14h11
VTT85

Oui je viens juste de comprendre le fonctionnement de la méthode qui reprend tous les labels.

Merci beaucoup :)