Points à vérifier pour une RT
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)