Discussion :

[Oberown]

Bonjour,

J'ai une classe qui a un constructeurs avec 19 paramètres de types différents. (des ints, des strings, d'autres classes, des bools..) Tous ces paramètres sont obligatoires.

Je voulais savoir si c'était crade ou non, et comment bien faire ?

Ex:

Code :

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

1
2
 
public MyClass(int a, string b, bool c, object d, string d)

Merci beaucoup pour vos conseils.

[Thomas Lebrun]

C'est pas très joli-joli mais si tu ne peux pas faire autrement...

[GoN49]

Pourquoi pas rassembler tes parametre pas groupe que tu mettra dans une structure ou une classe qui servira uniquement a transporter tes données et a les manipuler.

exemple de constructeur avec pleine de parametre :

Form1(string labelEntete,string labelPiedPage,string label centre ....)

tu stocke tout sa dans une structure ou classe (selon tes gouts)

public struct DecscriptifForm
{
string labelEntete;
string labelPiedPage;
string label centre;
...
}

Ca l'avantage de mieux regroupé par idées tes informations (donc plus facile pour l'utilisateur de ta classe) et ca évite les constructeur a rallonge .

Voila en ésperant que ca ta aidé

[SaumonAgile]

Pourquoi pas rassembler tes parametre pas groupe que tu mettra dans une structure ou une classe qui servira uniquement a transporter tes données et a les manipuler.

J'ai l'impression que ça ne ferait que déplacer le problème vers le constructeur de l'objet paramètre en question.
Car je pense que Oberown préfère garder la syntaxe la plus pratique possible et ne pas avoir à affecter à la suite 19 propriétés. Me trompes-je ?

Oberown : Sinon tu peux, en dérivant de l'idée de GoN49, créer des méthodes SetQuelqueChose en essayant de regrouper les arguments par domaines fonctionnels.
Par exemple :

Code :

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

1
2
3
Client myClient = new Client(Name, DateOfBirth, etc);
myClient.SetBillingInformation(Account, Bank, etc);
myClient.SetMainAddress(...);