(Note: pour jira qui a sa boîte de réception pleine >< ... si ça en intéresse d'autres bonne lecture, sinon passez votre chemin )

Alors voilà comment j'ai essayé d'implémenter la méthode DRY (Don't repeat yourself). Déjà il n'y a rien de carré, je ne suis pas webmaster à la base donc j'ai juste tenté d'appliquer du bon sens, pour éviter un maximum de répéter le code.

1. Formulaire d'ajout / d'édition
J'ai fait en sorte de ne jamais dupliquer le code de mes formulaires d'ajout / d'édition car la plupart du temps ce sont quasi les mêmes à quelques variables près. Du coup c'est le même formulaire qui est appelé pour modifier et pour créer un objet (ex: une guilde, un message, etc...). Idem pour coder l'action correspondante dans mon modèle: le même code gère la méthode save() pour enregistrer en base de données un nouveau objet et un objet que l'on modifie.

2. Création de mini templates pour afficher les objets POO
Pour les objets POO que j'affiche souvent (ex: un item du jeu, un message du forum ou de la messagerie), j'ai codé des mini templates en html que j'inclus pour afficher l'objet. Ça m'évite de me répéter et c'est ainsi très facile de réutiliser ce mini template (ex: le mini template afficherItem est appelé quand on liste les items d'une catégorie, sur la page équiper quand on affiche un item, sur la page CaC pour afficher l'arme, à la Forge etc...). Sous Symfony par ex, on appelle ça un "Partial".

3. Création des templates pour gérer les résultats d'un action
Le site étant découpé en modules, eux-mêmes découpés en actions (ajouter, voir, supprimer, modifier, etc), j'écrivais souvent le même code pour afficher le résultat d'une action (ex: "Erreur: item introuvable" ou "Equipement sauvegardé correctement"). Du coup dans mon controleur, je teste toutes les erreurs possibles pour chaque action:

• Si je n'ai pas d'erreur, et que mon action affiche un formulaire, j'affiche la page correspondante à mon action avec son formulaire (supprimer.php, ajouter.php, etc).
• Si je n'ai pas d'erreur et que mon formulaire a été posté, j'appelle la méthode adéquate de mon modèle depuis mon controleur (ex: $managerObjet->save($objet), $managerObjet->delete($objet)) puis j'affiche toujours la même page : ok.php.
• Si j'ai une erreur j'affiche toujours la même page : erreur.php.

Voilà, encore une fois rien d'extraordinaire, juste du bon sens qui m'a permis d'épurer mon code et de gagner en lisibilité =)

Note pour douce : Ta messagerie est pleine à 90% (27 / 30 messages autorisés au maximum) donc c'est une mauvaise excuse :D

Ok merci pour cette petite explication :) J'étais parti dans le même sens ça me rassure ^^'

J'ai a peu près compris le tous mais ma curiosité me démange : Pour ne pas répéter le code? je suppose que c'est juste ( si j'ai bien compris) pour que sa reste clair et pour se retrouver plus facilement c'est sa?

Parce que c'est plus propre, optimisé et plus facile a faire évoluer aussi =)

Je ne sais pas si ça existe pour du PHP et si tu en utilises déjà, mais il y a des "outils" de reporting pour suivre la qualité du code :
- rapport sur la duplication de code
- rapport sur la couverture des tests unitaires
- rapport sur la détection de bug (par exemple, une ouverture de fichier sans sa fermeture, un risque de boucle infinie)
- rapport sur le nb de ligne par "classes" et le nb de ligne des méthodes (fonction)
- rapport sur l'utilisation des variables et import (une variable déclarée non utilisée).

En programmation Java, on utilise souvent (en mode opensource/gratuit) :
- Eclipse (IDE pour programmer)
- Maven 2 (pour compiler, exécuter, lancer des tests unitaires, etc.)
- Sonar (pour le reporting)
- et plein de plugin pour les rapports de qualité (checkstyle, pmd, find bugs; cobertura, etc.)

Voilà, juste des idées, pour trouver les équivalents en PHP.

C'est ce que j'utilise au boulot à un détail près, on utilise hudson pour l'intégration continue