Discussion :

[WebTeamMtl]

Bonjour à tous, voilà je bute sur ce problème et je n'arrive pas à comprendre le pourquoi (certainement dû à un manque de connaissance).

La situation :

- Un site ASP.NET lié à une DLL.

Le probléme :

J'ai une classe User dans ma DLL:

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
 
public class User {
 
public User() {...}
 
public User GetbyID(int myUid){...}
 
public bool Create(){...}
 
virtual public bool Update(){...}
 
virtual public bool Delete(){...}
 
public static List<User> GetAllUser(){...}
}

Je veux que la commande GetAllUser() soit static pour ne pas devoir instancier un objet User. Cela ne me paraît pas logique d'instancier un objet User alors que je veux une liste de User.

Lorsque je vais dans une autre classe de ma dll, par exemple Session et que je fais User.GetAllUser(); pas de probléme.

Par contre lorsque je suis dans mon page load de mon default.aspx. J'ai bien mis le using madll.data;

Je veux réappeler ma méthode statique et la rien... Impossible de refaire un User.GetAllUser();

La seule solution que j'ai est de rendre la méthode publique et ensuite faire un :

Code :

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

1
2
3
 
var user = new User();
var y = user.GetAllUser();

Pourquoi dans ma dll, cela marche et que dans le site ASP cela ne fonctionne pas ? J'ai bien ajouté ma référence au site, j'ai fait un rebuild de ma dll. Rien n'y fait ?

QQun a-t'il une solution ou une piste ?

Merci d'avance à vous tous...

[Philippe Vialatte]

Bonjour,

J'ai jamais rencontre ce pb...par contre, le cas de ton appel dans la page asp me semble absolument normal, vu que ta classe User n'est pas definie comme statique...

Si tu veux pouvoir appeler User.GetAllUser() comme ca, User devrait etre statique. Personnelement, je bougerais ton GetallUsers dans une classe UsersFactory, comme ca, tu as une factory statique, qui instatie tes classes normales

[WebTeamMtl]

Merci de ta réponse.

Mais il me semble que l'on déconseille de créer des class rien que pour ce genre d'appel.

D'aprés C# 3.0 O'Reilly

Je cite :

Resist the temptation to create a single class in your program in which you stash all your miscellaneous methods. It is possible, but not desirablem and it undermines the encapsulation of an oject-oriented design.

La solution de créer une classe statique UserFactory avec mes méthodes statiques n'est pas contre ce qui est écrit ci-dessous ?

De plus je pensais que c'était l'une des utilisations dans laquelle on devait rendre une méthode statique malgré que la classe ne le soit pas.

De plus pourquoi lorsque je suis au niveau de la DLL, le User.GetAll fonctionne et lorsque je suis au niveau de mon site ASP, cela ne fonctionne plus.

C'est surtout cette partie qui m'échappe totalement....

De plus, si je vais dans ton sens, je vais avoir une Factory pour chacune de mes classes (Projects, Sessions, Page,...). Il est vrai que le GetAll ne sert que dans quelques cas mais j'aimerais avoir une explication sur ce comportement car qqchose m'échappe et ça ne me plaît pas...

[WebTeamMtl]

J'ai trouvé pourquoi

En fait dans using System.web, il y a déjà un objet User.
Ce qui fait qu'il prenait l'objet du framework et non pas l'objet de la dll.

En nommant ma classe User2, j'ai maintenant accès à ma méthode statique.

Je ne sais pas si c'est vraiment le bon fonctionnement d'utiliser une méthode statique à cette endroit mais cela simplifie mon code...

Des idées sur quel est la meilleur solution

[Philippe Vialatte]

re.

En fait dans using System.web, il y a déjà un objet User.

Tiens, oui, c'est vrai, j'y avais pas pensé...meuh

Resist the temptation to create a single class in your program in which you stash all your miscellaneous methods. It is possible, but not desirablem and it undermines the encapsulation of an oject-oriented design.

La solution de créer une classe statique UserFactory avec mes méthodes statiques n'est pas contre ce qui est écrit ci-dessous ?

Non, l'idée, c'est que, pour l'instant, tu as une classe user avec n attributs representant ton utilisateur.

Pour l'instant, tu veux ajouter une methode, qui va rapatrier tes n utilisateurs. D'ici peu, tu vas ajouter un GetOneUser(id), GetOneUserByFirstName, probablement un update, un insert et un delete, et quelques autres fonctions annexes.

Ces fonctions, je les vois bien dans une autre classe-> factory, ce qui te permets de separer les responsabilites de tes objets.