Fix the Bomb & Player secondary builder

2 files changed, 5 insertions, 6 deletions

diff --git a/src/ch/epfl/xblast/server/Bomb.java b/src/ch/epfl/xblast/server/Bomb.java
index 3f49f64..38bd4c5 100644
--- a/src/ch/epfl/xblast/server/Bomb.java
+++ b/src/ch/epfl/xblast/server/Bomb.java
@@ -58,14 +58,12 @@ public final class Bomb {
      * @throws NullPointerException if ownerId, position or fuseLengths is null
      */
     public Bomb(PlayerID ownerId, Cell position, int fuseLength, int range) {
-        this.ownerId = Objects.requireNonNull(ownerId);
-        this.position = Objects.requireNonNull(position);
         if (fuseLength == 0) {
             throw new IllegalArgumentException();
         } else {
             fuseLengths = Sq.iterate(fuseLength, fuseLengths -> fuseLength - 1 );
         }
-        this.range = ArgumentChecker.requireNonNegative(range);
+        this.Bomb(ownerId, position, fuseLengths, range);
     }
 
     /**
diff --git a/src/ch/epfl/xblast/server/Player.java b/src/ch/epfl/xblast/server/Player.java
index b3ae851..57a280a 100644
--- a/src/ch/epfl/xblast/server/Player.java
+++ b/src/ch/epfl/xblast/server/Player.java
@@ -166,13 +166,14 @@ public final class Player {
      */
     public Player(PlayerID id, int lives, Cell position, int maxBombs, int bombRange) {
         this.id = Objects.requireNonNull(id);
-        DirectedPosition newDirectedPos = new DirectedPosition(SubCell.centralSubCellOf(Objects.requireNonNull(position)), Direction.S);
-        this.directedPos = DirectedPosition.stopped(newDirectedPos);
+        DirectedPosition newDirectedPosition = new DirectedPosition(SubCell.centralSubCellOf(Objects.requireNonNull(position)), Direction.S);
+        DirectedPosition directedPositionSequence = DirectedPosition.stopped(newDirectedPos);
         LifeState invulnerability = new LifeState(ArgumentChecker.requireNonNegative(lives), LifeState.State.INVULNERABLE);
         LifeState vulnerability = new LifeState(ArgumentChecker.requireNonNegative(lives), LifeState.State.VULNERABLE);
-        this.lifeStates = Sq.repeat(Ticks.PLAYER_INVULNERABLE_TICKS,invulnerability).concat(Sq.constant(vulnerability));
+        Sq<LifeState> lifeStateSequence = Sq.repeat(Ticks.PLAYER_INVULNERABLE_TICKS,invulnerability).concat(Sq.constant(vulnerability));
         this.maxBombs = ArgumentChecker.requireNonNegative(maxBombs);
         this.bombRange = ArgumentChecker.requireNonNegative(bombRange);
+        this.Player(id, lifeStateSequence,directedPositionSequence, maxBombs, bombRange);
     }
 
     /**