Optimiser insertcommand et updatecommand

Version imprimable

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:

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

06/06/2008, 11h10
pascal_70

Devrais-je creer un objet pour chaque colonne?
07/06/2008, 01h10
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.
09/06/2008, 13h27
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.
10/06/2008, 00h07
olsimare

Bonjour.

Citation:

Envoyé par pascal_70

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

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

Cdt.

Juste en passant, ton code:
Code:

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:

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

Citation:

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

voire moins ? 0 blocs de code :mouarf:?

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

tu ajoutes une variable :

Code:

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:

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

Merci pour vos precieux conseils
Voila comment j'ais procédé , qu'en penser vous?

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
 
        private string Pays_Varchar;
        private string Pays_Commun;
 
        public Parametre_Command(string id_pays, string Dest_Prov, SqlCommand command)
        {
            switch (id_pays)
            {
                case "4": //  Belgique
                    Pays_Varchar = "Valfis,transport," + Dest_Prov + ",tva";
                    break;
 
                case "5": //  Luxembourg
                    Pays_Varchar = "Valfis,Provenance,Destination";
                    break;
 
                case "1": // France
                    Pays_Varchar = "Valfis_fr,Flux_fr,regime_fr,nature_fr,CondLiv,dept,origine,transport," + Dest_Prov + "";
                    break;
 
                default:
                    // Requete SELECT pour la structure standard (autres pays).
                    Pays_Varchar = "Valfis,Regime,nature,CondLiv,dept,origine,transport," + Dest_Prov + "";
                    break;
            }
 
            // Parametre commun à tous les pays.
            Pays_Commun = ",Flux,Nomenclature,ligne,poids,unité";
 
            Pays_Varchar += Pays_Commun;
 
            string sep = ",";
 
            string[] Pays_ToArray = Pays_Varchar.Split(sep.ToCharArray());
 
            foreach (string val in Pays_ToArray)
            {
                // Cas particulier où le type est different de Varchar ou les colonnes du DGv et de la BD sont differents .
                if (val == "nature_fr")
                    command.Parameters.Add("@" + val, SqlDbType.VarChar, 250, "nature");
                else if(val == "regime_fr")
                    command.Parameters.Add("@" + val, SqlDbType.VarChar, 250, "Regime");
                else if (val == "Valfis_fr")
                    command.Parameters.Add("@" + val, SqlDbType.Float, 250, "Valfis");
                else if (val == "Valfis")
                    command.Parameters.Add("@" + val, SqlDbType.Float, 250, "Valfis");
                else
                    // La colonne du DGV et de la BD identique et de type varchar.
                    command.Parameters.Add("@" + val, SqlDbType.VarChar, 250, val);
            }
        }

10/06/2008, 23h02
olsimare

Bonjour.

C'est déjà bien mieux !

Citation:

Envoyé par pvialatte

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

Entiérement d'accord, par une enum ou mieux via un paramétrage en base que tu charges au lancement de l'appli et via lequel tu pilotes la construction de la requête... mais là je vais un peu loin pour pas grand chose :lol: !

Cdt.