Discussion :

[Hanoni]

Bonjour à tous,

je voudrais avoir votre avis sur mon implémentation du DP MVC pour mon appli, et plus particulièrement à propos de ma couche DAO.

Alors voilà, j'ai 1 classe et 1 seule qui gère les connexions à la BDD.

Et à côté de cette classe, j'ai 1 classe par objet/bean qui se sert de la classe de connexion pour établir et fermer les connexions.

Chacune de ces classes gère ses propres Statement et Requêtes et formate les données dans le type de bean associé pour être utilisé dans la couche métier.

Ce qui fait qu'au final, on a dans la couche métier des instructions du genre :

Code :

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

1
2
3
4
 
User_DAO userdao=new User_DAO();
User user=userdao.formatUser(userdao.selectUser(codeUser));
userdao.close();

Est-ce que cette implémentation vous paraît correcte ?

J'ai lu pas mal de documentation à ce sujet, mais ça fait vraiment débat et les avis divergent tellement que...

Je veux juste savoir si vous voyez des failles dans mon raisonnement.

Merci d'avance de vos réponses

[Patriarch24]

User user=userdao.formatUser(userdao.selectUser(codeUser));

C'est pas terrible, tu donnes à la couche services des détails sur l'implémentation de ton DAO (selectUser).
Pourquoi au lieu de cette ligne affreuse, tu n'as pas un truc du genre :

Code :

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

User user = userDao.findByCode(userCode);

[Hanoni]

Bonjour et merci de ta réponse ,

En fait, je pense que ton findByCode() fait ce que fait mon selectUser() sauf que tu y inclues le formatage des données, c'est bien ça ?

[verbose]

Je suis d'accord avec Patriarche, le dao doit fournir des données directement exploitables. Le "formatage" est une bidouille qui ne concerne pas le client.

[Hanoni]

Ok, très bien, je fais les modifs nécessaires dans ce sens.

Merci de vos réponses