Fix the GameStateSerializer (comply with the subject)

4 files changed, 49 insertions, 16 deletions

diff --git a/src/ch/epfl/xblast/RunLengthEncoder.java b/src/ch/epfl/xblast/RunLengthEncoder.java
index ecb9f4e..d7ab106 100644
--- a/src/ch/epfl/xblast/RunLengthEncoder.java
+++ b/src/ch/epfl/xblast/RunLengthEncoder.java
@@ -189,4 +189,4 @@ public final class RunLengthEncoder {
                 .collect(Collectors.toList());
     }
 
-}
+}
\ No newline at end of file
diff --git a/src/ch/epfl/xblast/server/GameStateSerializer.java b/src/ch/epfl/xblast/server/GameStateSerializer.java
index 998ff21..049e993 100644
--- a/src/ch/epfl/xblast/server/GameStateSerializer.java
+++ b/src/ch/epfl/xblast/server/GameStateSerializer.java
@@ -47,13 +47,17 @@ public final class GameStateSerializer {
      * @param bombedCells cells containing a bomb
      * @param blastedCells cells containing a blast
      * @param cell cell hosting the explosion
+     * @param board actual Board
      * @return the serialized explosion
      */
-    private static byte serializeExplosion(Map<Cell, Bomb> bombedCells, Set<Cell> blastedCells, Cell cell) {
+    private static byte serializeExplosion(Map<Cell, Bomb> bombedCells, Set<Cell> blastedCells, Cell cell, Board board) {
         if (bombedCells.containsKey(cell))
             return ExplosionPainter.byteForBomb(bombedCells.get(cell));
-        else
+
+        if (blastedCells.contains(cell) && board.blockAt(cell).isFree())
             return serializeBlast(blastedCells, cell);
+
+        return ExplosionPainter.INVALID_EXPLOSION_IMAGE_ID;
     }
 
     /**
@@ -61,15 +65,29 @@ public final class GameStateSerializer {
      *
      * @param bombedCells cells containing a bomb
      * @param blastedCells cells containing a blast
+     * @param board actual Board
      * @return the serialized explosions
      */
-    private static List<Byte> serializeExplosions(Map<Cell, Bomb> bombedCells, Set<Cell> blastedCells) {
+    private static List<Byte> serializeExplosions(Map<Cell, Bomb> bombedCells, Set<Cell> blastedCells, Board board) {
         return RunLengthEncoder.encode(Cell.ROW_MAJOR_ORDER.stream()
-                .map(c -> serializeExplosion(bombedCells, blastedCells, c))
+                .map(c -> serializeExplosion(bombedCells, blastedCells, c, board))
                 .collect(Collectors.toList()));
     }
 
     /**
+     * Prepend its size to a Byte List.
+     *
+     * @param list Byte List to be prefixed
+     * @return the Byte List prefixed with its size
+     */
+    private static List<Byte> prefixByteListWithSize(List<Byte> list) {
+        List<Byte> prefixedList = new ArrayList<>();
+        prefixedList.add((byte) list.size());
+        prefixedList.addAll(list);
+        return prefixedList;
+    }
+
+    /**
      * Serialize a Player.
      *
      * @param player player to be serialized
@@ -96,17 +114,27 @@ public final class GameStateSerializer {
      */
     public static List<Byte> serialize(BoardPainter boardPainter, GameState gameState) {
         List<Byte> output = new ArrayList<>();
-        List<Byte> data = new ArrayList<>();
 
-        data.addAll(serializeBoard(boardPainter, gameState.board())); // Serialize the Board
-        data.addAll(serializeExplosions(gameState.bombedCells(), gameState.blastedCells())); // Serialize Explosions
+        // Add the serialized Board to the output
+        output.addAll(
+                prefixByteListWithSize(
+                    serializeBoard(boardPainter, gameState.board())
+                )
+        );
+
+        // Add the serialized Explosions to the output
+        output.addAll(
+                prefixByteListWithSize(
+                        serializeExplosions(gameState.bombedCells(), gameState.blastedCells(), gameState.board())
+                )
+        );
+
+        // Add the serialized Players to the output
         for (Player player : gameState.players()) {
-            data.addAll(serializePlayer(player, gameState.ticks())); // Serialize each Player
+            output.addAll(serializePlayer(player, gameState.ticks())); // Serialize each Player
         }
 
-        data.add((byte) (gameState.remainingTime() / 2)); // Add the state Tick to the data
-        output.add((byte) data.size()); // Get the size of the whole data chunk and add it as first element of the output
-        output.addAll(data); // Add all the data to the output
+        output.add((byte) (gameState.remainingTime() / 2)); // Add the state Tick to the output
 
         return output;
     }
diff --git a/src/ch/epfl/xblast/server/Level.java b/src/ch/epfl/xblast/server/Level.java
index 81bc85b..155b99a 100644
--- a/src/ch/epfl/xblast/server/Level.java
+++ b/src/ch/epfl/xblast/server/Level.java
@@ -13,15 +13,15 @@ import java.util.HashMap;
 public final class Level {
 
     /** Players' initial parameters **/
-    private static final int PLAYER_INITIAL_LIVES = 5;
+    private static final int PLAYER_INITIAL_LIVES = 3;
     private static final int PLAYER_INITIAL_BOMB_MAXIMUM = 5;
     private static final int PLAYER_INITIAL_BOMB_RANGE = 5;
 
     /** Players' initial positions (Ugly!) **/
     private static final Cell PLAYER_1_INITIAL_POSITION = new Cell(1,1);
-    private static final Cell PLAYER_2_INITIAL_POSITION = new Cell(16,1);
-    private static final Cell PLAYER_3_INITIAL_POSITION = new Cell(16,14);
-    private static final Cell PLAYER_4_INITIAL_POSITION = new Cell(1,14);
+    private static final Cell PLAYER_2_INITIAL_POSITION = new Cell(13,1);
+    private static final Cell PLAYER_3_INITIAL_POSITION = new Cell(13,11);
+    private static final Cell PLAYER_4_INITIAL_POSITION = new Cell(1,11);
 
     /** Level values **/
     private final BoardPainter painter;
diff --git a/src/ch/epfl/xblast/server/painter/ExplosionPainter.java b/src/ch/epfl/xblast/server/painter/ExplosionPainter.java
index 63181e9..83c6176 100644
--- a/src/ch/epfl/xblast/server/painter/ExplosionPainter.java
+++ b/src/ch/epfl/xblast/server/painter/ExplosionPainter.java
@@ -15,6 +15,11 @@ public final class ExplosionPainter {
     private static final byte WEST_EXPLOSION = 0b0001;
 
     /**
+     * Image ID corresponding to an explosion-free (and bomb-free) cell.
+     */
+    public static final byte INVALID_EXPLOSION_IMAGE_ID = (byte) 16;
+
+    /**
      * Return the image corresponding to a bomb depending on its fuse length.
      *
      * @param bomb given bomb