Discussion :

[pascal_70]

Bonjour, j'ais un code que je pense possible d'optimiser mais je ne vois pas comment, le code est en fait composer de deux partie , updatecommand et insertcommand il s'agit de la meme chose, mais je vois pas comment créer un seuk blok voir moins.
Merci pour votre aide.

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
 if (id_pays.Equals("4"))
            {
                insertCommand.Parameters.Add("@tva", SqlDbType.VarChar, 250, "tva");
                insertCommand.Parameters.Add("@" + Dest_Prov, SqlDbType.NVarChar, 250, Dest_Prov);
            }
 
 
            else if (id_pays.Equals("5"))
            {
                insertCommand.Parameters.Add("@Provenance", SqlDbType.VarChar, 250, "Provenance");
                insertCommand.Parameters.Add("@Destination", SqlDbType.VarChar, 250, "Destination");
            }
            else if (id_pays.Equals("1"))
            {
                               insertCommand.Parameters.Add("@regime_fr", SqlDbType.VarChar, 250, "Régime");
                insertCommand.Parameters.Add("@nature_fr", SqlDbType.VarChar, 250, "Nature");
            }
            else if (id_pays != "5")
                insertCommand.Parameters.Add("@transport", SqlDbType.NVarChar, 1, "transport");
            else
            {
                                insertCommand.Parameters.Add("@" + Dest_Prov, SqlDbType.NVarChar, 250, Dest_Prov);
                insertCommand.Parameters.Add("@regime", SqlDbType.VarChar, 250, "Régime");
                insertCommand.Parameters.Add("@nature", SqlDbType.VarChar, 250, "Nature");
                                insertCommand.Parameters.Add("@dept", SqlDbType.VarChar, 250, "dept");
                insertCommand.Parameters.Add("@origine", SqlDbType.VarChar, 250, "origine");
            }
			// Partie insert qui ressemble à l'identique au updatecommand
 
			 if (id_pays.Equals("4"))
            {
                updateCommand.Parameters.Add("@tva", SqlDbType.VarChar, 250, "tva");
                updateCommand.Parameters.Add("@" + Dest_Prov, SqlDbType.NVarChar, 250, Dest_Prov);
            }
 
                       else if (id_pays.Equals("5"))
            {
                updateCommand.Parameters.Add("@Provenance", SqlDbType.VarChar, 250, "Provenance");
                updateCommand.Parameters.Add("@Destination", SqlDbType.VarChar, 250, "Destination");
            }
            else if (id_pays.Equals("1"))
            {
                               updateCommand.Parameters.Add("@regime_fr", SqlDbType.VarChar, 250, "Régime");
                updateCommand.Parameters.Add("@nature_fr", SqlDbType.VarChar, 250, "Nature");
            }
            else if (id_pays != "5")
                updateCommand.Parameters.Add("@transport", SqlDbType.NVarChar, 1, "transport");
            else
            {
                // Pour les autres pays on met à jour soit la colonne "Provenance" ou "Destination".
                updateCommand.Parameters.Add("@" + Dest_Prov, SqlDbType.NVarChar, 250, Dest_Prov);
                updateCommand.Parameters.Add("@regime", SqlDbType.VarChar, 250, "Régime");
                updateCommand.Parameters.Add("@nature", SqlDbType.VarChar, 250, "Nature");
                               updateCommand.Parameters.Add("@dept", SqlDbType.VarChar, 250, "dept");
                updateCommand.Parameters.Add("@origine", SqlDbType.VarChar, 250, "origine");
            }

[pascal_70]

Devrais-je creer un objet pour chaque colonne?

[olsimare]

Bonjour.

Pourquoi ne pas créer un procédure qui prend en paramétre une commande et id_pays et qui se charge d'ajouter les paramétres de la requête à cette commande ?

Cdt.

[pascal_70]

Bonjour,
Merci pour ta réponse, tu veux dire que je devrais créer une procédure stockée pour effectuer ce traitement, j'aurais donc la meme chose mais du coté de la base de donnée.
Certe on y verrait plus claire.
Si c'est bien ce que tu voulais dire, biensure.

[olsimare]

Bonjour.

Si c'est bien ce que tu voulais dire, biensure.

Non, je parlais d'une void et pas d'une proc stockée !

Cdt.

[Philippe Vialatte]

Juste en passant, ton code:

Code :

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

1
2
3
4
5
6
7
8
9
10
if (id_pays.Equals("4"))
// cas 1 
            else if (id_pays.Equals("5"))
// cas 2
            else if (id_pays.Equals("1"))
// cas 3
            else if (id_pays != "5")
// cas 4
            else
// cas 5

je le verrais pas mal mieux comme ca :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
switch(id_pays)
case "4":
// cas 1 
    break;
case "5":
// cas 2
    break;
case "1":
// cas 3
    break;
default:
// cas 4 et 5 (on a deja teste id_pays != "5") 
     break;

Deuxieme conseil, si tu fais des tests sur 1, 4, et 5, tu vqs te compliquer la maintenance, remplace tes valeurs magiques par une constante qui a du sens pour ton appli

je vois pas comment créer un seuk blok voir moins.

voire moins ? 0 blocs de code ?

Sinon, si la logique de tes commande est identique, tu peux diminuer ton nombre de ligne facilement...

tu ajoutes une variable :

Code :

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

List<DbParameter> param = new List<DbParameter>();

tu reprends ton code, mais tu fais un Add de ton parametre dans params a chaque fois...

et a la fin de ta fonction, tu fais

Code :

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

1
2
insertCommand.Parameters.AddRange(param.ToArray());
updateCommand.Parameters.AddRange(param.ToArray());