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

40 files changed, 487 insertions, 502 deletions

diff --git a/src/main/java/fr/umlv/java/wallj/block/Block.java b/src/main/java/fr/umlv/java/wallj/block/Block.java
index d3c349e..7ae9c06 100644
--- a/src/main/java/fr/umlv/java/wallj/block/Block.java
+++ b/src/main/java/fr/umlv/java/wallj/block/Block.java
@@ -1,44 +1,25 @@
 package fr.umlv.java.wallj.block;
 
 import fr.umlv.java.wallj.board.TileVec2;
-import fr.umlv.java.wallj.context.Context;
-import fr.umlv.java.wallj.controller.BlockController;
-import fr.umlv.java.wallj.controller.Controller;
-import fr.umlv.java.wallj.event.Event;
+import fr.umlv.java.wallj.context.Updateable;
 import org.jbox2d.common.Vec2;
+import org.jbox2d.dynamics.World;
 
 import java.util.*;
-import java.util.stream.Collectors;
 
 /**
  * A block.
  *
  * @author Pacien TRAN-GIRARD
  */
-public abstract class Block {
-
+public abstract class Block implements Updateable {
   private final BlockType type;
-  private List<Controller> controllers;
-  private Vec2 pos;
 
   /**
    * @param type type of block
-   * @param pos  initial position
    */
-  Block(BlockType type, Vec2 pos) {
+  Block(BlockType type) {
     this.type = Objects.requireNonNull(type);
-    this.pos = Objects.requireNonNull(pos).clone();
-  }
-
-  void setControllers(List<BlockController> l) {
-    controllers = Collections.unmodifiableList(new LinkedList<>(Objects.requireNonNull(l)));
-  }
-
-  /**
-   * @param pos new position to set
-   */
-  public void setPos(Vec2 pos) {
-    this.pos = Objects.requireNonNull(pos).clone();
   }
 
   /**
@@ -49,27 +30,19 @@ public abstract class Block {
   }
 
   /**
-   * @return the current position
-   */
-  public Vec2 getPos() {
-    return pos.clone();
-  }
-
-  /**
    * @return the current tile position
    */
   public TileVec2 getTile() {
-    return TileVec2.of(pos);
+    return TileVec2.of(getPos());
   }
 
   /**
-   * @param ctx execution context
-   * @return list of generated events
+   * @return the current position
    */
-  public List<Event> update(Context ctx) {
-    return controllers.stream()
-           .flatMap(controller -> controller.update(ctx).stream())
-           .collect(Collectors.toList());
-  }
+  public abstract Vec2 getPos();
 
+  /**
+   * @param world a JBox2D world
+   */
+  public abstract void link(World world);
 }
diff --git a/src/main/java/fr/umlv/java/wallj/block/BlockFactory.java b/src/main/java/fr/umlv/java/wallj/block/BlockFactory.java
index 951d182..acd280f 100644
--- a/src/main/java/fr/umlv/java/wallj/block/BlockFactory.java
+++ b/src/main/java/fr/umlv/java/wallj/block/BlockFactory.java
@@ -1,7 +1,6 @@
 package fr.umlv.java.wallj.block;
 
 import fr.umlv.java.wallj.board.TileVec2;
-import fr.umlv.java.wallj.controller.BlockControllerFactory;
 import org.jbox2d.common.Vec2;
 
 /**
@@ -30,9 +29,7 @@ public final class BlockFactory {
   }
 
   public static Block build(BlockType type, TileVec2 pos) {
-    Block block = forType(type, pos.toVec2());
-    if (block != null) block.setControllers(BlockControllerFactory.build(block));
-    return block;
+    return forType(type, pos.toVec2());
   }
 
   private BlockFactory() {
diff --git a/src/main/java/fr/umlv/java/wallj/block/BombBlock.java b/src/main/java/fr/umlv/java/wallj/block/BombBlock.java
index de7a3b9..f58f1cc 100644
--- a/src/main/java/fr/umlv/java/wallj/block/BombBlock.java
+++ b/src/main/java/fr/umlv/java/wallj/block/BombBlock.java
@@ -1,28 +1,69 @@
 package fr.umlv.java.wallj.block;
 
+import fr.umlv.java.wallj.board.TileVec2;
+import fr.umlv.java.wallj.context.Context;
+import fr.umlv.java.wallj.context.GraphicsContext;
+import fr.umlv.java.wallj.event.Event;
 import org.jbox2d.common.Vec2;
+import org.jbox2d.dynamics.BodyType;
+
+import java.awt.*;
+import java.time.Duration;
+import java.util.Arrays;
+import java.util.Collections;
+import java.util.List;
 
 /**
  * A bomb block.
  *
  * @author Pacien TRAN-GIRARD
+ * @author Adam NAILI