Discussion :

[lapanne]

Bonjour à tous,

Je commence à développer l'architecture du nouvel intranet de l'entreprise et vu que je n'ai pas (encore) eu de cours sur l'architecture n-tier, je viens ici vous demander des feedback sur le code que j'ai pondu.

Aprés de nombreuses lectures (codes sources & autres arcticles...), j'ai réussi à pondre ma couche d'accès aux données et ma couche métiers. Je trouve ça super bancal et je n'utilise pas d'héritages ni d'interface ni de design pattern. Alors je me dis que c'est pas cool au vu de tout ce que je lis partout, y'a des IMachin partout .

Le but est de créer dans un premier temps les couches d'accés aux données et les couches métiers.

Bref, je vous joins juste quelques fichiers et un Modèle physique des données pondu par quelqu'un d'autre et dont je dépends :
Mon fichier de connection à la base de données
2 classes de la BLL
Une classe d'accés aux données
Le modèle physique des données

Voilà.
1er point : Je suis un débutant en développement donc n'hésitez à laisser tous les commentaires que vous voulez même si c'est trés méchant/condescendant/moqueur tant que ça m'aide à avancer.
2ème : un gros merci à ceux qui prendront le temps de regarder tout ça .

[BCmDev]

Quelques remarques rapides :

Il y a un problème de gestion de l'état dans la classe DataConnect.cs : la connection est fermée uniquement quand elle est... fermée !

Pour tes classes d'accès aux données, il pourrait être intéressant de gérer un cache des paramètres des procédures stockées (regarde du côté du Data Access Application Block).

Evite de nommer tes procédures stockées avec un préfixe "sp_" (http://www.sql-server-performance.co...procedures.asp)

[akecoocoo]

Ne t'arrêtes pas aux exemples que tu peux trouver ici ou là. Ce n'est pas parce qu'un projet implémente de nombreux motifs de conceptions, de frameworks réputés et se décompose en des dizaines de sous-projets qu'il est forcément bien réalisé.

Tu peux souvent lire que si le code source d'un projet ne contient pas ou très peu d'interfaces ("imachin"), cela signifie qu'il est très pauvrement programmé, ceci est bien évidement faux. Dans le cas des interfaces, elles servent à architecturer un projet en plusieurs briques logicielles. Si un jour, une particularité technique du projet est amenée à être changée, l'utilisation d'interfaces, ou de frameworks de découplages, garantie des modifications à moindre impact.

Je ne peux voir ton code source car free m'est inaccessible depuis mon poste de travail Une chose est certaine, il faut beaucoup de temps avant de prendre de bons réflexes en programmation. Faire du n-tiers, n'est pas une obligation ! Mais avec de l'expérience, on se rends compte qu'on est beaucoup plus productif/efficace en travaillant ainsi.

Je laisse aux autres le soin de commenter ton code

[Thomas Lebrun]

Tu as été voir ici: http://morpheus.developpez.com/architecture/ ?

[lapanne]

Merci à tous pour vos réponses je suis en train de mater le Data Access Application Block. Je ne sais pas encore si je vais m'en servir mais merci pour le tuyau !

Tu as été voir ici: http://morpheus.developpez.com/architecture/ ?

J'avais déjà maté. J'ai vu, j'ai lu, j'ai imprimu ! D'ailleurs j'ai une petite question :

Pourquoi utiliser un singleton dans la DAL et pas une classe entièrement statique (comme je le fais, je dois surement me planter ) ?

Tu peux souvent lire que si le code source d'un projet ne contient pas ou très peu d'interfaces ("imachin"), cela signifie qu'il est très pauvrement programmé, ceci est bien évidement faux. Dans le cas des interfaces, elles servent à architecturer un projet en plusieurs briques logicielles. Si un jour, une particularité technique du projet est amenée à être changée, l'utilisation d'interfaces, ou de frameworks de découplages, garantie des modifications à moindre impact.

D'ailleurs, j'ai du mal à voir l'utilité d'une interface. Qu'appeles-tu brique logicielle ?

[Thomas Lebrun]

Pourquoi utiliser un singleton dans la DAL et pas une classe entièrement statique (comme je le fais, je dois surement me planter ) ?

Il s'agit d'un choix fait par le développeur. J'ai opté pour un Singleton mais une classe statique aurait aussi pût faire l'affaire