From 7aa7b4950b38e88f3f7932b522652f17c4e93cce Mon Sep 17 00:00:00 2001
From: arnaudlabourel <arnaud.labourel@univ-amu.fr>
Date: Wed, 25 Oct 2023 15:48:53 +0200
Subject: [PATCH] minor renaming

---
 src/main/java/model/FirefighterBoard.java |  8 +++---
 src/main/java/view/FirefighterGrid.java   | 35 ++++++++++++++---------
 2 files changed, 26 insertions(+), 17 deletions(-)

diff --git a/src/main/java/model/FirefighterBoard.java b/src/main/java/model/FirefighterBoard.java
index 1e383a9..3251952 100644
--- a/src/main/java/model/FirefighterBoard.java
+++ b/src/main/java/model/FirefighterBoard.java
@@ -58,13 +58,13 @@ public class FirefighterBoard implements Board<List<ModelElement>> {
   }
 
   public List<Position> updateToNextGeneration() {
-    List<Position> result = activateFirefighters();
-    result.addAll(activateFires());
+    List<Position> result = updateFirefighters();
+    result.addAll(updateFires());
     step++;
     return result;
   }
 
-  private List<Position> activateFires() {
+  private List<Position> updateFires() {
     List<Position> result = new ArrayList<>();
     if (step % 2 == 0) {
       List<Position> newFirePositions = new ArrayList<>();
@@ -83,7 +83,7 @@ public class FirefighterBoard implements Board<List<ModelElement>> {
     return step;
   }
 
-  private List<Position> activateFirefighters() {
+  private List<Position> updateFirefighters() {
     List<Position> result = new ArrayList<>();
     List<Position> firefighterNewPositions = new ArrayList<>();
     for (Position firefighterPosition : firefighterPositions) {
diff --git a/src/main/java/view/FirefighterGrid.java b/src/main/java/view/FirefighterGrid.java
index fdb7cda..c4df123 100644
--- a/src/main/java/view/FirefighterGrid.java
+++ b/src/main/java/view/FirefighterGrid.java
@@ -12,7 +12,6 @@ public class FirefighterGrid extends Canvas implements Grid<ViewElement>{
     private void paintElementAtPosition(ViewElement element, Position position) {
         paintSquare(position.row(), position.column(), element.color);
     }
-
     private int squareWidth;
     private int squareHeight;
     private int columnCount;
@@ -20,19 +19,27 @@ public class FirefighterGrid extends Canvas implements Grid<ViewElement>{
 
     @Override
     public void repaint(List<Pair<Position, ViewElement>> positionedElements) {
+        paint(positionedElements);
+        paintLines();
+    }
+
+    private void paint(List<Pair<Position, ViewElement>> positionedElements) {
         for(Pair<Position, ViewElement> pair : positionedElements){
             paintElementAtPosition(pair.getValue(), pair.getKey());
         }
-        paintLines();
     }
 
     @Override
     public void repaint(ViewElement[][] elements) {
+        paint(elements);
+        paintLines();
+    }
+
+    private void paint(ViewElement[][] elements) {
         for(int column = 0; column < columnCount; column++)
             for(int row = 0; row < rowCount; row++){
                 paintElementAtPosition(elements[row][column], new Position(row, column));
             }
-        paintLines();
     }
 
     public int columnCount() {
@@ -43,9 +50,6 @@ public class FirefighterGrid extends Canvas implements Grid<ViewElement>{
         return rowCount;
     }
 
-    public FirefighterGrid(){
-    }
-
     @Override
     public void setDimensions(int columnCount, int rowCount, int squareWidth, int squareHeight) {
         this.squareWidth = squareWidth;
@@ -55,17 +59,22 @@ public class FirefighterGrid extends Canvas implements Grid<ViewElement>{
     }
 
     private void paintLines(){
-        for(int column=0; column<columnCount; column++)
-            getGraphicsContext2D().strokeLine(0, column*squareHeight, getWidth(), column*squareWidth);
-        for(int row=0; row<rowCount;row++)
-            getGraphicsContext2D().strokeLine(row*squareHeight, 0,row*squareHeight, getHeight());
+        paintHorizontalLines();
+        paintVerticalLines();
+    }
+
+    private void paintVerticalLines() {
+        for(int column = 0; column < columnCount; column++)
+            getGraphicsContext2D().strokeLine(column*squareWidth, 0,column*squareWidth, getHeight());
+    }
+
+    private void paintHorizontalLines() {
+        for(int row = 0; row < rowCount; row++)
+            getGraphicsContext2D().strokeLine(0, row*squareHeight, getWidth(), row*squareHeight);
     }
 
     private void paintSquare(int row, int column, Color color){
         getGraphicsContext2D().setFill(color);
         getGraphicsContext2D().fillRect(row*squareHeight,column*squareWidth,squareHeight,squareWidth);
     }
-
-
-
 }
\ No newline at end of file
-- 
GitLab