Skip to content
Snippets Groups Projects
Commit 2b08df44 authored by BELHACHEMI Mehdi's avatar BELHACHEMI Mehdi
Browse files

improving code

parent 6fb5b806
No related branches found
No related tags found
No related merge requests found
...@@ -43,9 +43,6 @@ public class FirefighterBoard implements Board<List<ModelElement>> { ...@@ -43,9 +43,6 @@ public class FirefighterBoard implements Board<List<ModelElement>> {
public void initializeElements() { public void initializeElements() {
obstacles.clear();
extinguishers.clear();
fires.clear();
for(ModelElement modelElement : ModelElement.values()){ for(ModelElement modelElement : ModelElement.values()){
for(int elementCount = 0; elementCount< modelElement.getInitialNumber();elementCount++){ for(int elementCount = 0; elementCount< modelElement.getInitialNumber();elementCount++){
Position position = randomPosition(); Position position = randomPosition();
...@@ -94,7 +91,7 @@ public class FirefighterBoard implements Board<List<ModelElement>> { ...@@ -94,7 +91,7 @@ public class FirefighterBoard implements Board<List<ModelElement>> {
public List<Position> updateToNextGeneration() { public List<Position> updateToNextGeneration() {
Fire fire = new Fire(randomPosition()); Fire fire = new Fire(null);
List<Position> modifiedPositions = new ArrayList<>(); List<Position> modifiedPositions = new ArrayList<>();
for(Extinguisher element : extinguishers.values()){ for(Extinguisher element : extinguishers.values()){
...@@ -124,6 +121,9 @@ public class FirefighterBoard implements Board<List<ModelElement>> { ...@@ -124,6 +121,9 @@ public class FirefighterBoard implements Board<List<ModelElement>> {
@Override @Override
public void reset() { public void reset() {
step = 0; step = 0;
obstacles.clear();
extinguishers.clear();
fires.clear();
initializeElements(); initializeElements();
} }
......
...@@ -25,12 +25,12 @@ public abstract class Extinguisher implements model.Elements.element { ...@@ -25,12 +25,12 @@ public abstract class Extinguisher implements model.Elements.element {
public void setPosition(Position position) { public void setPosition(Position position) {
this.position = position; this.position = position;
} }
public ModelElement getElement(){ public ModelElement getElement(){
return this.element; return this.element;
} }
public void extinguish(Board board, Position position) { public void extinguish(Board board, Position position) {
if (board.getFires().containsKey(position)) { if (board.getFires().containsKey(position)) {
board.getFires().remove(position); board.getFires().remove(position);
......
...@@ -15,12 +15,14 @@ import util.Position; ...@@ -15,12 +15,14 @@ import util.Position;
public enum ModelElement { public enum ModelElement {
FIREFIGHTER(5), FIREFIGHTER(5),
FIRE(20), FIRE(25),
CLOUD(10), CLOUD(10),
ROAD(5), ROAD(5),
MOUNTAIN(5), MOUNTAIN(5),
ROCAILLE(3), ROCAILLE(3),
MOTORIZEDFIREFIGHTER(5); MOTORIZEDFIREFIGHTER(3);
private final int initialNumber; private final int initialNumber;
...@@ -34,7 +36,6 @@ public enum ModelElement { ...@@ -34,7 +36,6 @@ public enum ModelElement {
} }
public element getModelElementClass(Position position){ public element getModelElementClass(Position position){
return switch (this){ return switch (this){
case FIRE -> new Fire(position); case FIRE -> new Fire(position);
case ROAD -> new Road(position); case ROAD -> new Road(position);
......
...@@ -6,7 +6,7 @@ public enum ViewElement { ...@@ -6,7 +6,7 @@ public enum ViewElement {
FIREFIGHTER(Color.BLUE), FIREFIGHTER(Color.BLUE),
FIRE(Color.RED), FIRE(Color.RED),
CLOUD(Color.LIGHTSKYBLUE), CLOUD(Color.LIGHTSKYBLUE),
ROAD(Color.GRAY), ROAD(Color.BLACK),
MOUNTAIN(Color.BROWN), MOUNTAIN(Color.BROWN),
ROCAILLE(Color.GREEN), ROCAILLE(Color.GREEN),
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment