Simplifier : Revue et Nettoyage de Code
Examiner tous les fichiers modifiés pour la réutilisation, la qualité et l'efficacité. Corriger les problèmes rencontrés.
Phase 1 : Identifier les Changements
Exécuter git diff (ou git diff HEAD s'il y a des changements en staging) pour voir ce qui a changé.
Phase 2 : Revue
Réutilisation du Code
- Rechercher des utilitaires existants qui pourraient remplacer le nouveau code.
- Signaler les fonctionnalités en double — suggérer la fonction existante.
- Signaler la logique inline qui pourrait utiliser un utilitaire existant.
Qualité du Code
- État redondant ou valeurs mises en cache qui pourraient être dérivées.
- Copier-coller avec de légères variations — unifier avec une abstraction partagée.
- Commentaires inutiles expliquant CE QUE fait le code.
- Chaînes brutes alors que des constantes ou des énumérations existent déjà.
Efficacité
- Calculs redondants, lectures répétées de fichiers.
- Opérations indépendantes exécutées séquentiellement alors qu'elles pourraient être parallélisées.
- Vérifications d'existence inutiles avant d'opérer (TOCTOU).
Phase 3 : Correction
Corriger chaque problème directement. Si un constat est un faux positif, le noter et passer à autre chose. Quand terminé, résumer brièvement ce qui a été corrigé (ou confirmer que le code était déjà propre).
Préserver le comportement. Préférer les petites refactorisations mécaniques. Éviter les réécritures non liées.