Ecriture

• pas de fonctions non exécutées / appelées / code mort. pas de variable non utilisée
• les noms des méthodes reflètent ce que font les méthodes ( pas de getXXXX qui ne renvoie rien et écrit des choses en base par exemple )
• les noms de variables reflètent ce qu'elles contiennent
• indentation : 2 lignes entre chaque méthode, 4 entre chaque classe, 2 espaces pour l'indentation

Documentation

• fichier VERSION et FEATURES complétés
• page wiki si besoin
• vidéo si besoin

Tests

• présence et exhaustivité des tests
• tests green

Design

• on comprend facilement le code
• pas de duplication de code
• un seul niveau d'indentation (pas de if dans un for dans un if dans un while)
• ne parle qu'à ton voisin: pas de $chose->getXXX()->getXXX()->doThis(). (Loi de Demeter)
• vérifier ce qui est enregistré en base de données (ex: connecteur jumel / getUrl() )
• attention aux variables statiques ( on n'aime pas les variables globales . Ex: Class_FileManager::beOpenBar)