♻ Refactorisation Commune

Author: MathieuSoysal

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

@@ -18,7 +18,9 @@
  * @see Departement
  * 
  * @author MathieuSoysal
- * @version 1.0.0
+ * @author fabien-zoccola
+ * 
+ * @version 1.1.0
  */
 public class Commune {
     @CsvBindByPosition(position = 0)
@@ -58,21 +60,14 @@ public int getCodePostal() {
     }
 
     /**
-     * @return la latitude de la commune
-     */
-    public double getLatitude() {
-        return latitude;
-    }
-
-    /**
-     * @return la longitude de la commune
+     * @return la localisation de la caumune
+     * 
+     * @see Localisation
+     * 
+     * @since 1.1.0
      */
-    public double getLongitude() {
-        return longitude;
-    }
-
-    public double[] getCoordonnees() {
-        return new double[] {latitude, longitude};
+    public Localisation getLocalisation() {
+        return new Localisation(latitude, longitude);
     }
 
     /**
@@ -142,7 +137,8 @@ public boolean equals(Object obj) {
             return false;
         if (nom == null) {
             return other.nom == null;
-        } else return nom.equals(other.nom);
+        } else
+            return nom.equals(other.nom);
     }
 
 }

src/main/java/fr/france/Localisation.java

@@ -0,0 +1,37 @@
+package fr.france;
+
+/**
+ * <b>Localisation est la classe représentant une localisation gps.</b>
+ * <p>
+ * Un objet Localisation est caractérisé par les informations suivantes :
+ * <ul>
+ * <li>Une latitude.</li>
+ * <li>Une longitude.</li>
+ * </ul>
+ * 
+ * @author MathieuSoysal
+ * @version 1.0.0
+ */
+public class Localisation {
+    private final double latitude;
+    private final double longitude;
+
+    public Localisation(double lat, double lon) {
+        latitude = lat;
+        longitude = lon;
+    }
+
+    /**
+     * @return la latitude de la localisation
+     */
+    public double getLatitude() {
+        return latitude;
+    }
+
+    /**
+     * @return la longitude de la localisation
+     */
+    public double getLongitude() {
+        return longitude;
+    }
+}

src/test/java/fr/france/CommuneTest.java

@@ -39,20 +39,10 @@ void test_getCodePostal() {
     }
 
     @Test
-    void test_getCoordonnees() {
-        double[] coord = new double[] {43.6134409138d, 3.86851657896d};
-        assertEquals(coord[0], Commune.getCoordonnees()[0]);
-        assertEquals(coord[1], Commune.getCoordonnees()[1]);
-    }
-
-    @Test
-    void test_getLatitude() {
-        assertEquals(43.6134409138d, Commune.getLatitude());
-    }
-
-    @Test
-    void test_getLongitude() {
-        assertEquals(3.86851657896d, Commune.getLongitude());
+    void test_getLocalisation() {
+        Localisation localisation = new Localisation(43.6134409138d, 3.86851657896d);
+        assertEquals(localisation.getLatitude(), Commune.getLocalisation().getLatitude());
+        assertEquals(localisation.getLongitude(), Commune.getLocalisation().getLongitude());
     }
 
     @Test

src/test/java/fr/france/LocalisationTest.java

@@ -0,0 +1,20 @@
+package fr.france;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+import org.junit.jupiter.api.Test;
+
+class LocalisationTest {
+
+    private Localisation localisation = new Localisation(43.6134409138d, 3.86851657896d);
+
+    @Test
+    void test_getLatitude() {
+        assertEquals(43.6134409138d, localisation.getLatitude());
+    }
+
+    @Test
+    void test_getLongitude() {
+        assertEquals(3.86851657896d, localisation.getLongitude());
+    }
+}