Refactor bombs evolution flow

1 files changed, 9 insertions, 9 deletions

diff --git a/src/ch/epfl/xblast/server/GameState.java b/src/ch/epfl/xblast/server/GameState.java
index abd07b3..19b16a9 100644
--- a/src/ch/epfl/xblast/server/GameState.java
+++ b/src/ch/epfl/xblast/server/GameState.java
@@ -416,7 +416,7 @@ public final class GameState {
             List<Bomb> ownedBombs = bombOwnerMap.get(p.id());
             if (ownedBombs != null && ownedBombs.size() >= p.maxBombs()) continue;
 
-            Bomb b = new Bomb(p.id(), p.position().containingCell(), Ticks.BOMB_FUSE_TICKS - 1, p.bombRange());
+            Bomb b = p.newBomb();
             bombedCells.add(b.position());
             bombs1.add(b);
         }
@@ -562,16 +562,16 @@ public final class GameState {
         // 3. existing explosions evolution
         List<Sq<Sq<Cell>>> explosions1 = GameState.nextExplosions(this.explosions);
 
-        // 4.1. existing bombs evolution
-        List<Bomb> explodingBombs = GameState.explodingBombs(this.bombs, blastedCells1);
-        List<Bomb> existingBombs = GameState.nextBombs(this.bombs, explodingBombs);
+        // 4.1. newly dropped bombs addition
+        Set<PlayerID> topPriorityBombDropEvents = discardConflictingBombDropEvents(bombDropEvents);
+        List<Bomb> allBombs = GameState.newlyDroppedBombs(this.players, topPriorityBombDropEvents, this.bombs);
 
-        // 4.2. subsequent explosions addition
-        explodingBombs.forEach(b -> explosions1.addAll(b.explosion()));
+        // 4.2. existing bombs evolution
+        List<Bomb> explodingBombs = GameState.explodingBombs(allBombs, blastedCells1);
+        List<Bomb> bombs1 = GameState.nextBombs(allBombs, explodingBombs);
 
-        // 4.3. newly dropped bombs addition
-        Set<PlayerID> topPriorityBombDropEvents = discardConflictingBombDropEvents(bombDropEvents);
-        List<Bomb> bombs1 = GameState.newlyDroppedBombs(this.players, topPriorityBombDropEvents, existingBombs);
+        // 4.3. subsequent explosions addition
+        explodingBombs.forEach(b -> explosions1.addAll(b.explosion()));
 
         // 5. players evolution
         Map<PlayerID, Bonus> playerBonuses = mapPlayersToBonuses(consumedBonuses);