Merge branch 'master' of https://github.com/pacien/upem-java-wallj

4 files changed, 36 insertions, 4 deletions

diff --git a/src/main/java/fr/umlv/java/wallj/block/RobotBlock.java b/src/main/java/fr/umlv/java/wallj/block/RobotBlock.java
index da7f3cf..f81f423 100644
--- a/src/main/java/fr/umlv/java/wallj/block/RobotBlock.java
+++ b/src/main/java/fr/umlv/java/wallj/block/RobotBlock.java
@@ -27,6 +27,7 @@ public class RobotBlock extends Block {
   private Vec2 pos;
   private PathFinder pathFinder;
   private Deque<TileVec2> path = new LinkedList<>();
+  private int droppedBombCount = 0;
 
   RobotBlock(Vec2 pos) {
     super(BlockType.ROBOT);
@@ -62,10 +63,17 @@ public class RobotBlock extends Block {
     return Events.findFirst(events, BombSetupOrder.class)
            .map(event -> isOnBomb(stage) ?
                          Collections.<Event>singletonList(new BombTimerIncrEvent(getTile())) :
-                         Collections.<Event>singletonList(new BlockCreateEvent(BlockType.BOMB, getTile())))
+                         dropBomb(event))
            .orElse(Collections.emptyList());
   }
 
+  private List<Event> dropBomb(BombSetupOrder order) {
+    if (droppedBombCount >= Stage.BOMB_PLACEMENTS) return Collections.emptyList();
+
+    droppedBombCount++;
+    return Collections.singletonList(new BlockCreateEvent(BlockType.BOMB, getTile()));
+  }
+
   private void updatePath(Board board, TileVec2 target) {
     if (!board.inside(target) || !board.getBlockTypeAt(target).isTraversable()) return;
     if (pathFinder == null) pathFinder = new PathFinder(board);
@@ -85,6 +93,9 @@ public class RobotBlock extends Block {
     graphicsContext.paintCircle(Color.BLUE, getPos(), TileVec2.TILE_DIM / 2);
   }
 
+  /**
+   * @implNote TODO: profile this and consider a mapping (pos: block) for faster lookup in Stage
+   */
   private boolean isOnBomb(Stage stage) {
     return stage.getBlocks().stream()
            .anyMatch(block -> Objects.equals(block.getType(), BlockType.BOMB) &&
diff --git a/src/main/java/fr/umlv/java/wallj/board/PathFinder.java b/src/main/java/fr/umlv/java/wallj/board/PathFinder.java
index c530d83..dfd1fa6 100644
--- a/src/main/java/fr/umlv/java/wallj/board/PathFinder.java
+++ b/src/main/java/fr/umlv/java/wallj/board/PathFinder.java
@@ -57,7 +57,7 @@ public class PathFinder {
 
   private static <T> List<T> findPath(Node<T> start, T target, BiFunction<T, T, Double> heuristic) {
     Map<Node<T>, NodeSearchData<T>> searchData = new HashMap<>();
-    TreeSet<Node<T>> discovered = new TreeSet<>(Comparator.comparingDouble(n -> searchData.get(n).estimatedCost));
+    Queue<Node<T>> discovered = new PriorityQueue<>(Comparator.comparingDouble(n -> searchData.get(n).estimatedCost));
     Set<Node<T>> visited = new HashSet<>();
 
     searchData.put(start, new NodeSearchData<>(null, 0, heuristic.apply(start.val, target)));
@@ -65,7 +65,7 @@ public class PathFinder {
 
     Node<T> current;
     while (!discovered.isEmpty()) {
-      current = discovered.pollFirst();
+      current = discovered.poll();
       if (target.equals(current.val)) return buildPath(searchData, current);
 
       for (Map.Entry<Node<T>, Integer> neighborEntry : current.neighbors.entrySet()) {
diff --git a/src/main/java/fr/umlv/java/wallj/context/Stage.java b/src/main/java/fr/umlv/java/wallj/context/Stage.java
index 49aa196..9737583 100644
--- a/src/main/java/fr/umlv/java/wallj/context/Stage.java
+++ b/src/main/java/fr/umlv/java/wallj/context/Stage.java
@@ -17,6 +17,7 @@ import java.util.*;
  * @author Pacien TRAN-GIRARD
  */
 public class Stage implements Updateable {
+  public static final int BOMB_PLACEMENTS = 3;
   private static final int VELOCITY_TICK_PER_MS = 6;
   private static final int POSITION_TICK_PER_MS = 2;
 
@@ -60,7 +61,17 @@ public class Stage implements Updateable {
    * @implNote TODO: profile this and consider a garbage block counter
    */
   public boolean isCleared() {
-    return blocks.stream().noneMatch(block -> block.getType() == BlockType.GARBAGE);
+    return blocks.stream()
+           .noneMatch(block -> block.getType() == BlockType.GARBAGE);
+  }
+
+  /**
+   * @return T(the physics simulation can start, i.e. the player has placed all their bombs)
+   */
+  public boolean isReady() {
+    return blocks.stream()
+           .filter(block -> block.getType() == BlockType.BOMB)
+           .count() == BOMB_PLACEMENTS;
   }
 
   /**
diff --git a/src/main/java/fr/umlv/java/wallj/event/SimulationStartOrder.java b/src/main/java/fr/umlv/java/wallj/event/SimulationStartOrder.java
new file mode 100644
index 0000000..a268ed0
--- /dev/null
+++ b/src/main/java/fr/umlv/java/wallj/event/SimulationStartOrder.java
@@ -0,0 +1,10 @@
+package fr.umlv.java.wallj.event;
+
+/**
+ * Signals the beginning of the physics simulation.
+ *
+ * @author Pacien TRAN-GIRARD
+ */
+public final class SimulationStartOrder implements InputEvent {
+  // void
+}