Optimisation du code

Version imprimable

Salut à tous

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

Merci

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

19/05/2011, 17h16
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(); ".

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:

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

20/05/2011, 08h13
jbrasselet
Tu peux aussi faire des .Equals pour la comparaison de tes chaines de caractères.
Code:

1 2 3 if(MODIFIER.Equals(btnModifier.Text)) { ... }
20/05/2011, 15h07
punisher999

Merci de vos commentaires

C'est avec l'appuit d'expert qu'on apprend à mieux travailler !!!
20/05/2011, 15h39
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

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

Une solution pour ceci ?

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

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