style: renamed constants to match pascal case, as they should

Author: Fabien Zoccola

src/main/java/fr/france/Commune.java

@@ -117,11 +117,8 @@ public boolean equals(Object obj) {
         if (codePostal != other.codePostal)
             return false;
         if (nom == null) {
-            if (other.nom != null)
-                return false;
-        } else if (!nom.equals(other.nom))
-            return false;
-        return true;
+            return other.nom == null;
+        } else return nom.equals(other.nom);
     }
 
 }

src/main/java/fr/france/Departement.java

@@ -478,20 +478,20 @@ public enum Departement {
     ILE_DE_CLIPPERTON(989, StatutDepartement.COM, "Île de Clipperton", "Clipperton",
             Region.DEPARTEMENT_NON_RATTACHE_A_UNE_REGION);
 
-    private int code;
-    private StatutDepartement statut;
-    private String nom;
-    private String villeDePrefecture;
-    private Region region;
+    private final int code;
+    private final StatutDepartement statut;
+    private final String nom;
+    private final String villeDePrefecture;
+    private final Region region;
 
     /**
-     * @param code
-     * @param statut
-     * @param nom
-     * @param villeDePrefecture
-     * @param region
+     * @param code le code du département
+     * @param statut son statut
+     * @param nom son nom
+     * @param villeDePrefecture la ville de sa préfecture
+     * @param region sa région
      */
-    private Departement(int code, StatutDepartement statut, String nom, String villeDePrefecture, Region region) {
+    Departement(int code, StatutDepartement statut, String nom, String villeDePrefecture, Region region) {
         this.code = code;
         this.statut = statut;
         this.nom = nom;

src/main/java/fr/france/Region.java

@@ -104,12 +104,12 @@ public enum Region {
     DEPARTEMENT_NON_RATTACHE_A_UNE_REGION("Département non rattaché à une région.");
 
     private List<Departement> departements;
-    private String nom;
+    private final String nom;
 
     /**
      * @param nom Nom de la région
      */
-    private Region(String nom) {
+    Region(String nom) {
         this.nom = nom;
     }
 

src/main/java/fr/france/RepertoireCommune.java

@@ -35,7 +35,7 @@ private RepertoireCommune() {
         throw new IllegalStateException("Class utilitaire");
     }
 
-    private static final List<Commune> CommuneS = initCommunes();
+    private static final List<Commune> COMMUNES = initCommunes();
 
     private static List<Commune> initCommunes() {
         List<Commune> result = null;
@@ -56,7 +56,7 @@ private static List<Commune> initCommunes() {
         Map<Integer, List<Commune>> codePostaux = new HashMap<>();
         Map<String, List<Commune>> noms = new HashMap<>();
         Map<String, Commune> nomEtCodePostal = new HashMap<>();
-        for (Commune Commune : CommuneS) {
+        for (Commune Commune : COMMUNES) {
             if (Commune.getNom() == null)
                 continue;
             nomEtCodePostal.put(OutilsString.formater(Commune.getNom()) + Commune.getCodePostal(), Commune);
@@ -78,7 +78,7 @@ private static List<Commune> initCommunes() {
      * @see Commune
      */
     public static List<Commune> getCommunes() {
-        return new ArrayList<>(CommuneS);
+        return new ArrayList<>(COMMUNES);
     }
 
     /**

src/main/java/fr/france/outils/OutilsDepartement.java

@@ -2,7 +2,6 @@
 
 import java.util.ArrayList;
 import java.util.Arrays;
-import java.util.Collection;
 import java.util.EnumMap;
 import java.util.List;
 import java.util.Map;
@@ -33,7 +32,7 @@ private OutilsDepartement() {
 
     private static final Map<Integer, Departement> REPERTOIRE_CODE = initialiserRepertoireCode();
     private static final RepertoireGenerique<Departement> REPERTOIRE_NOM = new RepertoireGenerique<>(Departement.class);
-    private static final Map<Departement, List<Commune>> REPERTOIRE_Commune = initialiserRepertoireCommune();
+    private static final Map<Departement, List<Commune>> REPERTOIRE_COMMUNE = initialiserRepertoireCommune();
 
     public static List<Departement> filtrerDepartementsParRegion(Region region) {
         List<Departement> result = new ArrayList<>();
@@ -60,22 +59,22 @@ public static Departement rechercherParCode(int code) {
     }
 
     public static List<Commune> getCommunes(Departement departement){
-        return REPERTOIRE_Commune.getOrDefault(departement, new ArrayList<>());
+        return REPERTOIRE_COMMUNE.getOrDefault(departement, new ArrayList<>());
     }
 
     private static EnumMap<Departement, List<Commune>> initialiserRepertoireCommune() {
         EnumMap<Departement, List<Commune>> result = new EnumMap<>(Departement.class);
         for (Commune Commune : RepertoireCommune.getCommunes()) {
             Departement departement = Commune.getDepartement();
-            List<Commune> Communes = result.computeIfAbsent(departement, k -> new ArrayList<>());
-            Communes.add(Commune);
+            List<Commune> communes = result.computeIfAbsent(departement, k -> new ArrayList<>());
+            communes.add(Commune);
         }
         return result;
     }
 
     private static Map<Integer, Departement> initialiserRepertoireCode() {
         return OutilsRepertoireGenerique.genererMap(
-                Arrays.asList(Departement.values()).stream().map(Departement::getCode).collect(Collectors.toList()),
+                Arrays.stream(Departement.values()).map(Departement::getCode).collect(Collectors.toList()),
                 Departement.class);
     }
 }

src/main/java/fr/france/outils/OutilsRegion.java

@@ -26,24 +26,24 @@ private OutilsRegion() {
     }
 
     private static final RepertoireGenerique<Region> REPERTOIRE_REGION = new RepertoireGenerique<>(Region.class);
-    private static final Map<Region, List<Commune>> REPERTOIRE_Commune = initialiserRepertoireCommune();
+    private static final Map<Region, List<Commune>> REPERTOIRE_COMMUNE = initialiserRepertoireCommune();
 
     public static Region rechercherParNom(String nom) {
         return REPERTOIRE_REGION.rechercherParNom(nom);
     }
 
     public static List<Commune> getCommunes(Region region) {
-        return REPERTOIRE_Commune.get(region);
+        return REPERTOIRE_COMMUNE.get(region);
     }
 
     private static Map<Region, List<Commune>> initialiserRepertoireCommune() {
         EnumMap<Region, List<Commune>> result = new EnumMap<>(Region.class);
         for (Region region : Region.values()) {
-            List<Commune> Communes = new ArrayList<>();
+            List<Commune> communes = new ArrayList<>();
             for (Departement departement : region.getDepartements()) {
-                Communes.addAll(departement.getCommunes());
+                communes.addAll(departement.getCommunes());
             }
-            result.put(region, Communes);
+            result.put(region, communes);
         }
         return result;
     }

src/main/java/fr/france/outils/RepertoireGenerique.java

@@ -9,7 +9,7 @@ class RepertoireGenerique<T extends Enum<T>> {
     private final Map<String, T> repertoire;
 
     /**
-     * @param repertoire
+     * @param enumType the enum
      */
     public RepertoireGenerique(Class<T> enumType) {
         Map<String, T> temp = new HashMap<>();