15 lines
865 B
Text
15 lines
865 B
Text
Problèmes de qualité de code:
|
|
- Nombres magiques un peu partout (20, 80, 30, 10, ...)
|
|
- la méthode demarrerVoiture n'utilie pas son paramètre voiture
|
|
- idem pour arreterVoiture
|
|
- La méthode changerVitesse déclare une variable locale vitesseActuelle qu'elle n'utilise pas
|
|
- Les méthodes accelerer et ralentir
|
|
- Mélanges anglais / français
|
|
- méthodes à déplacer
|
|
- tests à revoir
|
|
|
|
|
|
Checkstyle mentionne surtout le manque de commentaire Javadoc, et demande d'utiliser 2 espaces au lieu de 4 par indentation (ce qui est purement arbitraire et subjectif).
|
|
Checkstyle parle également des manques d'espaces après certaines virgules, notamment dans Application.java:10.
|
|
Il y a aussi des mentions du nombre de charactères par ligne qui excèdent 80, et certaines variables qui devraient être finales.
|
|
Les nombres magiques sont aussi mentionnés.
|