Discussion :

[mayekeul]

Bonjour,

alors voici mon soucis

Je récolte dans mon code une série de parametre pour lancer une procedure stockée dans mon serveur sql.

Suivant les cas, la procedure à lancer sera différente mais les nombreux parametres eux restent identiques

je me suis dis que je devrais faire appel à un délégué pour cela, pour m'éviter de retaper la longue série de parametre à envoyer à la procedure.

Alors deux questions:
- mon raisonnement est-il bon?
- Comment faire car j'essaye mais ça ne fonctionne pas du tout.

merci de votre aide

[Guulh]

Hello,
ça doit pas être la peine.
En gros, t'as un truc comme ça:
- des proc stocks: nommées Machin, Bidule, Truc, Chose, Chouette
- des paramètres tout pareils: 1, "toto", 3.14, 21-FEB-2048
Je sais pas comment tu utilises ado.net pour appeler tes procs. Mais si tu le fais à la main, la seule chose qui changera, ce sera la IdbCommand.CommandText, qui contient le nom de la proc stock.

Donc ta couche DAO ressemblerait à ça:

Code :

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

1
2
3
4
5
6
7
8
9
10
enum Command { Machin, Bidule, Truc, Chose, Chouette }
private string GetProcStockName(Command cmd)
{
  // renvoie le nom de la proc stock
}
public void Execute(Command cmd, int param1, string param2, double param3, ...)
{
  string procStockName = GetProcStockName(cmd);
  //... ton code usuel d'exécution de la proc stock
}

Et si par exemple t'as une GUI avec 5 boutons, chacune correspondant à une proc stock, t'aurais ça dans la méthode abonnée au clic de chaque bouton:

Code :

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

1
2
3
4
5
6
7
8
9
private void OnClick1(object sender, EventArgs e)
{
  coucheDAO.Execute(Commands.Machin, 1, "toto", 3.14, ...);
}
 
private void OnClick2(object sender, EventArgs e)
{
  coucheDAO.Execute(Commands.Bidule, 1, "toto", 3.14, ...);
}

Pas besoin de délégué, donc.

[mayekeul]

hello,

merci de ta réponse

je passe par le clr mais en gros mon code, pour l'instant, ressemble à ton exemple.

C'est juste que il y a +/- 40 parametres à passer quoi
donc a la lecture, ça fait un peu brouillon quand il apparait plusieurs fois

[Guulh]

je passe par le clr mais en gros mon code, pour l'instant, ressemble à ton exemple.

C'est à dire ?

C'est juste que il y a +/- 40 parametres à passer quoi
donc a la lecture, ça fait un peu brouillon quand il apparait plusieurs fois

Ah, c'est cette partie là que tu veux regrouper?
Bon, je suppose que tu as un bouton par méthode.
Tu te fais un dico qui associe à un bouton à une valeur d'enum:

Code :

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

1
2
3
4
5
6
var mappings = new Dictionary<Button, Command>
{
   { this.button1, Commands.Machin },
   { this.button2, Commands.Truc },
...
}

Tu abonnes les Clic de tous les boutons à cette méthode:

Code :

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

1
2
3
4
5
private void OnClick(object sender, EventArgs e)
{
   var command = this.mappings[(Button)sender)];
   coucheDAO.Execute(command, 1, "toto", 3.14, ...);
}

Et d'ailleurs, vu que les méthodes à 40 paramètres sont pas très lisibles, tu peux stocker tout ça dans un seul object (disons SuperParamètre), ca deviendra

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
private SuperParamètre GetParamètre()
{
  return new SuperParamètre(1, "toto", ...);
}
private void OnClick(object sender, EventArgs e)
{
   var command = this.mappings[(Button)sender)];
   var param = GetParamètre();
   coucheDAO.Execute(command, param);
}

[mayekeul]

bon finalement je réponds et sorry du délai

Citation:
Envoyé par mayekeul
je passe par le clr mais en gros mon code, pour l'instant, ressemble à ton exemple.

C'est à dire ?

Code :

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

1
2
3
4
5
6
7
8
9
private void OnClick1(object sender, EventArgs e)
{
  Dataset.Procedure1( 1, "toto", 3.14, ...);
}
 
private void OnClick2(object sender, EventArgs e)
{
  Dataset.Procedure2( 1, "toto", 3.14, ...);
}

et il faudra que j'essaye cette methode super parametre alors!