Correction

src/main/java/model/BoardFireFighterBehavior.java

@@ -54,6 +54,13 @@ public class BoardFireFighterBehavior implements BoardBehavior{
 
     public static void extinguish(Position position) {
         fireMovements.getPositions().remove(position);
+
+        // Si la position est une montagne, elle cesse d'être une montagne
+        if (rocky.contains(position)) {
+            rocky.remove(position);
+            terrainMap.remove(position); // Optionnel, si vous utilisez terrainMap pour gérer les types de terrain
+            System.out.println("Une montagne a brûlé à la position : " + position);
+        }
     }
 
 

src/main/java/model/FireFighterMovements.java

@@ -25,38 +25,39 @@ public class FireFighterMovements implements Movements{
         List<Position> newFirefighterPositions = new ArrayList<>();
 
         for (Position firefighterPosition : firefighterPositions) {
-            // Trouver la position cible en excluant les montagnes et les positions non valides
-            Position newFirefighterPosition = getNeighbors().get(firefighterPosition).stream()
-                    .filter(pos -> !getRocky().contains(pos)) // Exclure montagnes
-                    .filter(pos -> canMoveTo(pos, (Set<Position>) fireMovements.getPositions(), firefighterPositions)) // Valider mouvements
-                    .findFirst()
-                    .orElse(firefighterPosition); // Reste sur place si aucun mouvement valide
-
-            if (!newFirefighterPosition.equals(firefighterPosition)) {
-                // Déplacer et éteindre feu
+            // Calcul de la position vers laquelle le pompier devrait se déplacer
+            Position newFirefighterPosition =getTargetStrategy().neighborClosestToTarget(firefighterPosition, fireMovements.getPositions(), getNeighbors());
+
+            // Vérification si la position cible est valide pour le mouvement
+            if (canMoveTo(newFirefighterPosition, (Set<Position>) fireMovements.getPositions(), firefighterPositions)) {
+                // Si le déplacement est valide, on met à jour la position du pompier
                 newFirefighterPositions.add(newFirefighterPosition);
+
+                // Éteindre le feu à la nouvelle position
                 extinguish(newFirefighterPosition);
                 modifiedPositions.add(firefighterPosition);
                 modifiedPositions.add(newFirefighterPosition);
 
-                // Éteindre les feux voisins accessibles
+                // Vérification des voisins et extinction des feux voisins
                 List<Position> neighborFirePositions = getNeighbors().get(newFirefighterPosition).stream()
-                        .filter(fireMovements.getPositions()::contains)
-                        .filter(pos -> !getRocky().contains(pos))
-                        .toList();
+                        .filter(fireMovements.getPositions()::contains).toList();
+
+                // Log pour débogage
+                System.out.println("Pompiers se déplacent de " + firefighterPosition + " vers " + newFirefighterPosition);
 
+                // Éteindre les feux voisins
                 for (Position firePosition : neighborFirePositions) {
                     extinguish(firePosition);
-                    modifiedPositions.add(firePosition);
+                    modifiedPositions.add(firePosition);  // Ajout des feux éteints dans la liste des positions modifiées
                 }
             } else {
-                // Pompier reste sur place
+                // Si la position n'est pas valide, le pompier reste sur place
                 newFirefighterPositions.add(firefighterPosition);
-                System.out.println("Pompier bloqué à " + firefighterPosition);
+                System.out.println("Pompier ne peut pas se déplacer à " + newFirefighterPosition + ", il reste à " + firefighterPosition);
             }
         }
 
-        // Mise à jour des positions des pompiers
+        // Mettre à jour la liste des positions des pompiers
         firefighterPositions = newFirefighterPositions;
         return modifiedPositions;
     }