Use preferably immutable interfaces

1 files changed, 6 insertions, 6 deletions

diff --git a/src/ch/epfl/xblast/Cell.java b/src/ch/epfl/xblast/Cell.java
index 8214066..9720281 100644
--- a/src/ch/epfl/xblast/Cell.java
+++ b/src/ch/epfl/xblast/Cell.java
@@ -30,12 +30,12 @@ public final class Cell {
     /**
      * The list of the board's Cell-s, major-ordered.
      */
-    public static final List<Cell> ROW_MAJOR_ORDER = Collections.unmodifiableList(rowMajorOrder());
+    public static final List<Cell> ROW_MAJOR_ORDER = rowMajorOrder();
 
     /**
      * The list of the board's Cell-s, spiral-ordered.
      */
-    public static final List<Cell> SPIRAL_ORDER = Collections.unmodifiableList(spiralOrder());
+    public static final List<Cell> SPIRAL_ORDER = spiralOrder();
 
     /**
      * The coordinates of the Cell.
@@ -69,14 +69,14 @@ public final class Cell {
      *
      * @return the list of Cell-s
      */
-    private static ArrayList<Cell> rowMajorOrder() {
+    private static List<Cell> rowMajorOrder() {
         ArrayList<Cell> list = new ArrayList<>(COUNT);
 
         for (int row = 0; row < ROWS; ++row)
             for (int col = 0; col < COLUMNS; ++col)
                 list.add(new Cell(col, row));
 
-        return list;
+        return Collections.unmodifiableList(list);
     }
 
     /**
@@ -84,7 +84,7 @@ public final class Cell {
      *
      * @return the list of Cell-s
      */
-    private static ArrayList<Cell> spiralOrder() {
+    private static List<Cell> spiralOrder() {
         ArrayList<Cell> list = new ArrayList<>(COUNT);
         ArrayList<Integer> ix = range(0, COLUMNS, 1);
         ArrayList<Integer> iy = range(0, ROWS, 1);
@@ -101,7 +101,7 @@ public final class Cell {
             horizontal = !horizontal;
         }
 
-        return list;
+        return Collections.unmodifiableList(list);
     }
 
     /**