Coding rules Java - TecostNetwork/Network GitHub Wiki

Comparaison d'enums

Les enums doivent être comparées via == et non via equals() car c'est type-safe, null-safe, en cas d'erreur c'est détecté à la compilation et c'est plus pratique à écrire et à lire.

If-else et accolades

De manière générale, on préfère avoir trop d'accolades que pas assez.

Les accolades rendent le code plus lisible et facilitent la maintenance. Il est arrivé d'ajouter une ligne dans un if sans accolades et de ne pas se rendre compte que cette ligne ne fait donc pas partie du if. L'effet n'est pas toujours vu immédiatement, en l'occurence ici c'était en production.

Si un if a une accolade, il faut mettre une accolade au else également. ça évitera les erreurs dues à la lisibilité.

Idem pour les if-else imbriqués, lorsqu'un if contient un seul bloc mais qu'il est composé de plusieurs lignes (une boucle ou un autre if) il vaut mieux mettre les accolades. La ligne rouge ci-desous indique là où les accolades manquent :