|
|
* pas de fonctions non exécutées / appelées / code mort
|
|
|
# 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 |
|
|
\ No newline at end of file |
|
|
* 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)
|
|
|
|
|
|
|
|
|
|
|
|
|