Discussion :

[punisher999]

Salut à tous

Je voudrais savoir s'Il y a moyen d'optimiser ce code afin qu'il soit plus court.

Merci

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
private void btnModifier_Click(object sender, EventArgs e)
        {
 
            if (btnModifier.Text == "Modifier")
            {
                parAncien = construireParameter();
 
                edit();
                txtDomain.Focus();
                btnAjouter.Enabled = false;
                button3.Enabled = false;
                button4.Enabled = false;
                button5.Enabled = false;
                button6.Enabled = false;
                btnModifier.Text = "Sauvegarder";
            }
            else
            {
                par = construireParameter();
 
 
                if(parAncien.Equals(par))
                    memeValeur = true;
 
                else if (gp.parametreDifferent(par))
                {
                    gp.updateParameter(parAncien,par);
                    MessageBox.Show("Modification effectue avec succes");
                    btnModifier.Text = "Modifier";
                }
                else
                {
                    MessageBox.Show("ERREUR");
                }
                if (memeValeur == true)
                {
                    MessageBox.Show("Aucune modification apporte");
                }
 
            }
        }

[DonQuiche]

Bonjour.

Déjà, une simplification possible, supprimer "memeValeur" et afficher directement la MessageBox.

Chaîner les assignations "false" : button3.Enabled = bounton4.Enabled = ... = false; Ca ne change rien au code généré mais la source sera plus courte.

Supprimer des if avec des returns : "if (x) { y() } else { z() }" est parfois remplaçable par "if (x) { y(); return; } z(); ".

[hegros]

En plus de ce qui a été dis, une optimisation à faire est de supprimer les duplications (principes DRY don't repeat yourself(ne te répète pas))

Par exemple, construireParameter est appelée dans tous les cas donc ne l'appelle qu'une fois.

Ensuite tu pourrais ne pas mélanger français/anglais au moins dans un même nom (comme dans construireParameter :-) )

Les constantes genre "Modifier", "Sauvegarder" devrait être des constantes ou readonly de ta classe pour éviter à terme de se répéter.

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
 
 
const string MODIFIER = "Modifier";
const string SAUVEGARDER ="Sauvegarder";
 
private void btnModifier_Click(object sender, EventArgs e)
        {
           var CurrentPar = BuildParameter();
            if (btnModifier.Text == MODIFIER)
            {
                parAncien = CurrentPar ;
 
                edit();
                txtDomain.Focus();
                btnAjouter.Enabled  = button3.Enabled =  button4.Enabled =    button5.Enabled =  button6.Enabled = false;
                btnModifier.Text = SAUVEGARDER;
            }
            else
            {
                par = CurrentPar ;                
 
                if(parAncien.Equals(par))
                     MessageBox.Show("Aucune modification apporte");
 
                else if (gp.parametreDifferent(par))
                {
                    gp.updateParameter(parAncien,par);
                    MessageBox.Show("Modification effectue avec succes");
                    btnModifier.Text = MODIFIER;
                }
                else
                {
                    MessageBox.Show("ERREUR");
                }
 
            }
        }

[jbrasselet]

Tu peux aussi faire des .Equals pour la comparaison de tes chaines de caractères.

Code C# :

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

1
2
3
if(MODIFIER.Equals(btnModifier.Text)) {
...
}

[punisher999]

Merci de vos commentaires

C'est avec l'appuit d'expert qu'on apprend à mieux travailler !!!

[Er3van]

Juste pour faire le mec chiant, optimiser du code ce n'est pas ça.
Là on va parler de formatage de code source.

Optimiser ça concernerait plutôt un problème d'efficacité, de performances, d'utilisation de mémoire, consommation de cycles processeurs, etc..., et au niveau du bite code généré.

Par ailleurs, une amélioration pourrait consister à utiliser des fichiers de ressources pour gérer les champs textes que tu affiches

[punisher999]

Encore une fois les amis, je fais du code redondant.

Une solution pour ceci ?

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
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
public DataSet searchParameter(Parameter par)
        {
            int nbParam = 0;
            String sRequete;
 
            sRequete = "SELECT DOMAIN_ID, SUB_DOMAIN_ID,CODE,VAL1,VAL2,VAL3,VAL4,VAL5,VAL6 FROM APPLICATION_PARAMETER_EXT";
            if (par.DomainID != "Select an item" )
            {
                sRequete += " WHERE DOMAIN_ID like '" + par.DomainID + "'";
                nbParam++;
            }
            if (par.SubDomainID != "Select an item")
            {
                if (nbParam == 0)
                    sRequete += " WHERE";
                else
                    sRequete += " AND";
                sRequete += " SUB_DOMAIN_ID like '" + par.SubDomainID + "'";
                nbParam++;
            }
            if (par.Code != "Select an item")
            {
                if (nbParam == 0)
                    sRequete += " WHERE";
                else
                    sRequete += " AND";
                sRequete += " CODE like '" + par.Code + "'";
            }
            if (par.Description != "")
            {
                if (nbParam == 0)
                    sRequete += " WHERE";
                else
                    sRequete += " AND";
                sRequete += " upper(DESCRIPTION) like upper('%" + par.Description + "%')";
 
            }
            ds = db.ExecuteReader(sRequete);
            return ds;
        }
 
        public DataSet searchParameter2(Parameter par)
        {
            int nbParam = 0;
            String sRequete;
 
            sRequete = "SELECT description FROM APPLICATION_PARAMETER_EXT";
            if (par.DomainID != "Select an item")
            {
                sRequete += " WHERE DOMAIN_ID like '" + par.DomainID + "'";
                nbParam++;
            }
            if (par.SubDomainID != "Select an item")
            {
                if (nbParam == 0)
                    sRequete += " WHERE";
                else
                    sRequete += " AND";
                sRequete += " SUB_DOMAIN_ID like '" + par.SubDomainID + "'";
                nbParam++;
            }
            if (par.Code != "Select an item")
            {
                if (nbParam == 0)
                    sRequete += " WHERE";
                else
                    sRequete += " AND";
                sRequete += " CODE like '" + par.Code + "'";
            }
            if (par.Description != "")
            {
                if (nbParam == 0)
                    sRequete += " WHERE";
                else
                    sRequete += " AND";
                sRequete += " upper(DESCRIPTION) like upper('%" + par.Description + "%')";
 
            }
            ds = db.ExecuteReader(sRequete);
            return ds;
        }

Ça fait la même chose sauf pas la même requête....

Pour est-ce je fais cela ? C'est ici, quand je crée mon datagridView, il ne faut pas j'aille le même tableau...

Code :

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

1
2
3
4
5
6
7
8
  private void button1_Click(object sender, EventArgs e)
        {
            par = construireParameter();
            DataSet ds = gp.searchParameter(par);
            DataSet ds2 = gp.searchParameter2(par);
            dataGridView1.DataSource = ds.Tables[0];
            dataGridView2.DataSource = ds2.Tables[0];
        }

Il faudrait que je réussisse à choisir les colonnes en créant mon dataGridview je crois, mais je n'Ai pas trouvé la solution..