Centralisation de requête d'insertion a travers plusieurs forms

**ammar.dev** · 08/11/2016, 23h13

bonsoir, j'ai un projet avec plusieurs formulaire d'ajout dans la base de donnée mysql. mais quand j’écris la partie d'insertion dans la base de donnée dans le dossier DAL je me trouve que je recopie presque le même code source pour insérer à la base de donnée. par exemple pour l'objet personne j’écris ce partie de code source:

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
class DALPersonne
    {
        public static void AddPersonne(string DALcin, string DALnom, string DALprenom, string DALnomarabe, string DALprenomarabe, string DALadresse, string DALadressearabe, string DALcivilite, string DALvillenaissance, string DALvillenaissancearabe, string DALnationnalite, string DALgouvernerat, string DALsexe, string DALville, string DALvillearabe, DateTime DALdatenaissance, int DALcodepostal, int DALtelephonne, string DALEmail,
string DALciviliteAr, string DALsexAr, string DALnationaliteAr, string DALgovernoratAr)
        {
  MySqlCommand requeteadd = new MySqlCommand("insert into Personne (CIN,Nom,Prenom,Nomarabe,Prenomarabe,Adresse,Adressearabe,Civilite,Villenaissance,Villenaissancearabe,Nationnalite,Gouvernerat,Sexe,Ville,Villearabe,Datenaissance,Codepostal,Telephonne,Email,CiviliteAr,SexAr,NationaliteAr,GovernoratAr) values('" +
                DALcin + "','" + 
                DALnom + "','"+
                DALprenom +"','"+
                 DALnomarabe.ToString() + "','" +
                 DALprenomarabe.ToString() + "','" +
                 DALadresse+ "','"+
                 DALadressearabe+ "','"+
                      DALcivilite+ "','"+
                      DALvillenaissance+ "','"+
                          DALvillenaissancearabe+ "','"+
                          DALnationnalite+ "','"+
                              DALgouvernerat+ "','"+
                              DALsexe+ "','"+
                                  DALville+ "','"+
                                  DALvillearabe+ "','"+
                                      DALdatenaissance.ToString("yyyy-MM-dd") + "'," +
 
                DALcodepostal+","+
            DALtelephonne+",'"+
            DALEmail + "','"+
             DALciviliteAr+ "','"+
                 DALsexAr+ "','"+
                 DALnationaliteAr+ "','"+
                     DALgovernoratAr+ "')", Program.Connection);
 
            requeteadd.ExecuteNonQuery();
 
 
 
        }
    }

et quand j’écris un autre requête d'insertion je répété presque le même code source sauf que le paramètre d'entrée au méthode se change.
comment faire pour écrire une seule méthode sans écrire à chaque form ce partie de code source.

merci d'avance

**jopopmk** · 09/11/2016, 07h46

Salut,

en gros il arrive que tu insères un tuple "personne" avec plus ou moins de paramètres ?
Si oui ce que tu peux faire c'est une méthode qui prend tous les champs possibles. A l'appel, si un paramètre est null/nothing alors tu ne le considères pas, si il est valorisé alors tu montes ta requête en conséquence.

Bon dev.

NB : utilise les MySqlParameters à la place de faire un simple concat', ça t'éviteras des surprises/contraintes de validation sur les saisies des utilisateurs.

**ammar.dev** · 09/11/2016, 12h49

Envoyé par jopopmk

Salut,

en gros il arrive que tu insères un tuple "personne" avec plus ou moins de paramètres ?
Si oui ce que tu peux faire c'est une méthode qui prend tous les champs possibles. A l'appel, si un paramètre est null/nothing alors tu ne le considères pas, si il est valorisé alors tu montes ta requête en conséquence.

Bon dev.

NB : utilise les MySqlParameters à la place de faire un simple concat', ça t'éviteras des surprises/contraintes de validation sur les saisies des utilisateurs.

c'est une classe personne avec leurs propriétés et méthode qui appel cette methode d'insertion :

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
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
class Personne
    {
 
        private string _cin;
 
        public string CIN
        {
            get { return _cin; }
            set { _cin = value; }
        }
 
        private string _nom;
 
        public string Nom
        {
            get { return _nom; }
            set { _nom = value; }
        }
        private string _prenom;
 
        public string Prenom
        {
            get { return _prenom; }
            set { _prenom = value; }
        }
        private string _nomarabe;
 
        public string Nomarabe
        {
            get { return _nomarabe; }
            set { _nomarabe = value; }
        }
        private string _prenomarabe;
 
        public string Prenomarabe
        {
            get { return _prenomarabe; }
            set { _prenomarabe = value; }
        }
        private string _adresse;
 
        public string Adresse
        {
            get { return _adresse; }
            set { _adresse = value; }
        }
        private string _adressearabe;
 
        public string Adressearabe
        {
            get { return _adressearabe; }
            set { _adressearabe = value; }
        }
        private string _civilite;
 
        public string Civilite
        {
            get { return _civilite; }
            set { _civilite = value; }
        }
        private string _villenaissance;
 
        public string Villenaissance
        {
            get { return _villenaissance; }
            set { _villenaissance = value; }
        }
        private string _villenaissancearabe;
 
        public string Villenaissancearabe
        {
            get { return _villenaissancearabe; }
            set { _villenaissancearabe = value; }
        }
        private string _nationnalite;
 
        public string Nationnalite
        {
            get { return _nationnalite; }
            set { _nationnalite = value; }
        }
        private string _gouvernerat;
 
        public string Gouvernerat
        {
            get { return _gouvernerat; }
            set { _gouvernerat = value; }
        }
        private string _sexe;
 
        public string Sexe
        {
            get { return _sexe; }
            set { _sexe = value; }
        }
        private string _ville;
 
        public string Ville
        {
            get { return _ville; }
            set { _ville = value; }
        }
        private string _villearabe;
 
        public string Villearabe
        {
            get { return _villearabe; }
            set { _villearabe = value; }
        }
        private DateTime _datenaissance;
 
        public DateTime Datenaissance
        {
            get { return _datenaissance; }
            set { _datenaissance = value; }
        }
        private int _codepostal;
 
        public int Codepostal
        {
            get { return _codepostal; }
            set { _codepostal = value; }
        }
        private int _telephonne;
 
        public int Telephonne
        {
            get { return _telephonne; }
            set { _telephonne = value; }
        }
        private string _Email;
 
        public string Email
        {
            get { return _Email; }
            set { _Email = value; }
        }
 
        private string _civiliteAr;
 
        public string CiviliteAr
        {
            get { return _civiliteAr; }
            set { _civiliteAr = value; }
        }
        private string _sexAr;
 
        public string SexAr
        {
            get { return _sexAr; }
            set { _sexAr = value; }
        }
        private string _nationaliteAr;
 
        public string NationaliteAr
        {
            get { return _nationaliteAr; }
            set { _nationaliteAr = value; }
        }
        private string _governoratAr;
 
        public string GovernoratAr
        {
            get { return _governoratAr; }
            set { _governoratAr = value; }
        }
 
       public Personne()
       {
 
       }
       public Personne(string cin, string nom, string prenom, string nomarabe, string prenomarabe, DateTime datenaissance, string adresse, string adressearabe, int codepostal, string civilite, string villenaissance, string villenaissancearabe, string nationnalite, string gouvernerat, string sexe, string ville, string villearabe, int telephonne, string email, string civiliteAr, string sexAr, string nationaliteAr, string governoratAr)
       {
           this._cin = cin;
           this._nom = nom;
           this._prenom = prenom;
           this._nomarabe                     = nomarabe;
           this._prenomarabe                    = prenomarabe;
           this._adresse                     = adresse;
           this._adressearabe                    = adressearabe;
           this._civilite                   = civilite;
           this._villenaissance                  = villenaissance;
           this._villenaissancearabe                 = villenaissancearabe;
           this._nationnalite                   = nationnalite;
           this._gouvernerat                     = gouvernerat;
           this._sexe                            = sexe;
           this._ville                       = ville;
           this._villearabe                  = villearabe;
           this._datenaissance               = datenaissance;
           this._codepostal                 = codepostal;
           this._telephonne                 = telephonne;
           this._Email                       = email;
           this._civiliteAr = civilite;
           this._sexAr = sexAr;
           this._nationaliteAr = nationaliteAr;
           this._governoratAr = gouvernerat;
 
 
       }
 
       public void addpersonne()
       {
           DALPersonne inscription = new DALPersonne();
 
           inscription.AddPersonne(this.CIN, this.Nom, this.Prenom, this.Nomarabe, this.Prenomarabe, this.Adresse, this.Adressearabe, this.Civilite, this.Villenaissance, this.Villenaissancearabe, this.Nationnalite, this.Gouvernerat, this.Sexe, this.Ville, this.Villearabe, this.Datenaissance, this.Codepostal, this.Telephonne, this.Email,this.CiviliteAr,this.SexAr,this.NationaliteAr,this.GovernoratAr);
       }
 
    }

cette classe à été appeler dans le form et assigner leur propriété:

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
Personne personne = new Personne();
            personne.CIN = TxtCin.Text;
 
 
                    personne.Nom = TxtNom.Text;
                    personne.Prenom = TxtPrenom.Text;
                    personne.Nomarabe = "";
                    personne.Prenomarabe = "";
                    personne.Adresse = TxtAdresse.Text;
                    personne.Adressearabe = "";
                    personne.Civilite = CmbCivilite.Text;
                    personne.Villenaissance = TxtVilleNaissance.Text;
                    personne.Villenaissancearabe = "";
                    personne.Nationnalite = cmbnationalite.Text;
                    personne.Gouvernerat = CmbGouvernerat.Text;
                    personne.Sexe = cmbsexe.Text;
                    personne.Ville = TxtVille.Text;
 
                    personne.Villearabe = "";
 
                    personne.CiviliteAr ="";
                    personne.SexAr = "";
                    personne.NationaliteAr = "";
                    personne.GovernoratAr = "";
 
                personne.Datenaissance = DateTime.Parse(DTPDateNaissance.Value.ToString());
                personne.Codepostal = int.Parse(TxtCodePostal.Text);
                personne.Telephonne = int.Parse(TxtTelephone.Text);
                personne.Email = TxtEmail.Text;
 
                    personne.addpersonne();

**jopopmk** · 09/11/2016, 13h00

Ben dans ton premier code c'est une classe DALPersonne avec une méthode AddPersonne static ...

Quoi qu'il en soit ça change pas grand chose à mon propos : lorsque tu fais un INSERT tu composes ta requête selon les attributs (au lieu de paramètres) qui sont valorisés. Et encore une fois : n'oublie pas d'utiliser des MySqlParameters, la concat' c'est cracra.

**ammar.dev** · 10/11/2016, 10h55

Envoyé par jopopmk

Ben dans ton premier code c'est une classe DALPersonne avec une méthode AddPersonne static ...

Quoi qu'il en soit ça change pas grand chose à mon propos : lorsque tu fais un INSERT tu composes ta requête selon les attributs (au lieu de paramètres) qui sont valorisés. Et encore une fois : n'oublie pas d'utiliser des MySqlParameters, la concat' c'est cracra.

oui mais mon problème dans ce cas consiste que je ne sais pas exactement les nombres des paramètres d'entrées. comme il peut être:

Code :

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

1
2
public static void AddPersonne(string DALcin, string DALnom, string DALprenom, string DALnomarabe, string DALprenomarabe, string DALadresse, string DALadressearabe, string DALcivilite, string DALvillenaissance, string DALvillenaissancearabe, string DALnationnalite, string DALgouvernerat, string DALsexe, string DALville, string DALvillearabe, DateTime DALdatenaissance, int DALcodepostal, int DALtelephonne, string DALEmail,
string DALciviliteAr, string DALsexAr, string DALnationaliteAr, string DALgovernoratAr)

il peut être aussi:

Code :

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

public static void AddEtudiant(string DALcin, string DALnom, string DALprenom, string DALadresse,int note, int niveau)

**Noxen** · 10/11/2016, 11h11

C'est annexe à la question mais si tes propriétés sont des simples accesseurs à des champs internes tu peux alléger le code de ta classe avec les propriétés automatique :

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
 
class Personne {
  private string _nom;
  public string Nom {
    get { return _nom; }
    set { _nom = value; }
  }
 
  private string _prenom;
  public string Prenom {
    get { return _prenom; }
    set { _prenom = value; }
  }
 
  private string _ville;
  public string Ville{
    get { return _ville; }
    set { _ville = value; }
  }
}

devient simplement :

Code :

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

1
2
3
4
5
6
 
class Personne {
  public string Nom { get; set; }
  public string Prenom { get; set; }
  public string Ville { get; set; }
}

Pareil pour le constructeur paramétré, avec des setters public tu peux utiliser l'initialisation à la volée :

Code :

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

1
2
3
 
var personne = new Personne { Nom = "Darme", Prenom = "Jean" }; 
// je peux n'initialiser que les champs dont j'ai besoin ou pour lesquels j'ai l'information

Il sera toujours temps plus tard de les enrichir si tu as d'autres besoins (implémenter INotifyPropertyChanged, par exemple).

**ZenZiTone** · 10/11/2016, 11h12

Petite remarque sur tes paramètres. Pour un AddPersonne tu envois en paramètres tous les éléments définissant une Personne. Pour simplifier cela tu pourrais ne demander qu'un paramètre : une Personne (qui est composée de tous les attributs nécessaires). Ça rendra le code plus facile à comprendre et à modifier.

Centralisation de requête d'insertion a travers plusieurs forms

C#

Vue hybride

Discussions similaires

Partager

Partager