Discussion :

[Nowwis]

Bonjour à tous,

Voilà j'ai un projet à effectuer lors de ma formation, et j'ai une demande à vous faire.

J'ai une multitude de requête dans ce projet, j'aimerais savoir si je peux effectuer une fonction qui me permettrait de ne pas rappeler ces lignes à chaque fois :

Code :

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

1
2
3
OleDbCommand oCmd2 = new OleDbCommand(requete, Program.oConn);
oCmd2.CommandText = requete;
oCmd2.CommandType = CommandType.Text;

Merci d'avance !

[olibara]

Ecrire des methodes pour centraliser des traitement récurents c'est quand meme le baba du développement

Quel est plus précisément ton problème ?

[Nowwis]

J'ai en fait une multitude de requête (ajout, suppression, modification...). De plus, pour l'ajout, je fait 4 select pour remplir des combobox. Donc par exemple ça donne ça :

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
  #region Auteur
            string requete;
            requete = "SELECT NumAuteur FROM AUTEUR WHERE NomAuteur='" + CB_InsertAuteur.SelectedItem + "';";
 
            OleDbCommand oCmd = new OleDbCommand(requete, Program.oConn);
            oCmd.CommandText = requete;
            oCmd.CommandType = CommandType.Text;
            OleDbDataReader oDr;
            oDr = oCmd.ExecuteReader();
            oDr.Read();
            idauteur = (int)oDr[0];
            oDr.Close();
            #endregion
            #region Editeur
            requete = "SELECT NumEditeur FROM EDITEUR WHERE NomEditeur='" + CB_InsertEditeur.SelectedItem + "'";
 
            OleDbCommand oCmd2 = new OleDbCommand(requete, Program.oConn);
            oCmd2.CommandText = requete;
            oCmd2.CommandType = CommandType.Text;
            OleDbDataReader oDr2;
            oDr2 = oCmd2.ExecuteReader();
            oDr2.Read();
            idediteur = (int)oDr2[0];
            oDr2.Close();
            #endregion
            #region Collection
            requete = "SELECT NumCollection FROM COLLECTION WHERE NomCollection='" + CB_InsertCollection.SelectedItem + "'";
 
            OleDbCommand oCmd3 = new OleDbCommand(requete, Program.oConn);
            oCmd3.CommandText = requete;
            oCmd3.CommandType = CommandType.Text;
            OleDbDataReader oDr3;
            oDr3 = oCmd3.ExecuteReader();
            oDr3.Read();
            idcollection = (int)oDr3[0];
            oDr3.Close();
            #endregion
            #region Theme
            requete = "SELECT NumTheme FROM THEME WHERE NomTheme='" + CB_InsertTheme.SelectedItem + "'";
 
            OleDbCommand oCmd4 = new OleDbCommand(requete, Program.oConn);
            oCmd4.CommandText = requete;
            oCmd4.CommandType = CommandType.Text;
            OleDbDataReader oDr4;
            oDr4 = oCmd4.ExecuteReader();
            oDr4.Read();
            idtheme = (int)oDr4[0];
            oDr4.Close();
            #endregion

ça me parait bien lourd... Donc j'aurais voulu effectuer une sorte de fonction qui serait appelé à chaque fois...

[StormimOn]

... ça me parait bien lourd... Donc j'aurais voulu effectuer une sorte de fonction qui serait appelé à chaque fois...

Regarde attentivement ton code et tu verras que tu peux le factoriser, comme l'a indiqué Olibara. Tu centralises la partie en question dans une méthode et le tour est joué. C'est le b.a.-ba comme l'a dit Olibara ^^