diff --git a/src/main/java/inf112/fiasko/roborally/networking/containers/GameStartInfo.java b/src/main/java/inf112/fiasko/roborally/networking/containers/GameStartInfo.java index b10ada8..de98d3b 100644 --- a/src/main/java/inf112/fiasko/roborally/networking/containers/GameStartInfo.java +++ b/src/main/java/inf112/fiasko/roborally/networking/containers/GameStartInfo.java @@ -13,14 +13,22 @@ public class GameStartInfo { private String playerName; /** - * Empty initialization method used by kryo + * Empty initialization method required by kryo */ public GameStartInfo() {} + /** + * Sets the player name of the current player + * @param playerName The player name of the current player + */ public void setPlayerName(String playerName) { this.playerName = playerName; } + /** + * Gets the player name of the current player + * @return The player name of the current player + */ public String getPlayerName() { return playerName; } diff --git a/src/main/java/inf112/fiasko/roborally/networking/containers/PowerdownContainer.java b/src/main/java/inf112/fiasko/roborally/networking/containers/PowerdownContainer.java index 2b0413b..0bd6880 100644 --- a/src/main/java/inf112/fiasko/roborally/networking/containers/PowerdownContainer.java +++ b/src/main/java/inf112/fiasko/roborally/networking/containers/PowerdownContainer.java @@ -2,14 +2,25 @@ package inf112.fiasko.roborally.networking.containers; import java.util.Map; +/** + * This class is used to contain power down status for all players + */ public class PowerdownContainer { - private Map powerdown; + private final Map powerDown; - public PowerdownContainer(Map powerdown) { - this.powerdown = powerdown; + /** + * Instantiates a new power down container + * @param powerDown A map between player names and whether they should remain in power down + */ + public PowerdownContainer(Map powerDown) { + this.powerDown = powerDown; } - public Map getPowerdown() { - return powerdown; + /** + * Gets the power down map stored in the container + * @return A map between player name and stay in power down + */ + public Map getPowerDown() { + return powerDown; } } diff --git a/src/main/java/inf112/fiasko/roborally/objects/Phase.java b/src/main/java/inf112/fiasko/roborally/objects/Phase.java index f581739..648b9ff 100644 --- a/src/main/java/inf112/fiasko/roborally/objects/Phase.java +++ b/src/main/java/inf112/fiasko/roborally/objects/Phase.java @@ -9,9 +9,9 @@ import java.util.concurrent.TimeUnit; * This class handles everything that should happen during a phase */ public class Phase { - private Board gameBoard; + private final Board gameBoard; private final List playerList; - private int cycleDelay; + private final int cycleDelay; private List> cogwheels; private List> conveyorBelts; private List> fastConveyorBelts; diff --git a/src/main/java/inf112/fiasko/roborally/objects/RoboRallyGame.java b/src/main/java/inf112/fiasko/roborally/objects/RoboRallyGame.java index 19ff2b7..3c5b593 100644 --- a/src/main/java/inf112/fiasko/roborally/objects/RoboRallyGame.java +++ b/src/main/java/inf112/fiasko/roborally/objects/RoboRallyGame.java @@ -183,7 +183,7 @@ public class RoboRallyGame implements IRoboRallyGame { @Override public void receiveStayInPowerDown(PowerdownContainer powerDowns) { for (Player player : playerList) { - player.setPowerDownNextRound(powerDowns.getPowerdown().get(player.getName())); + player.setPowerDownNextRound(powerDowns.getPowerDown().get(player.getName())); } respawnRobots(); resetHasTouchedFlagThisTurnForAllRobots();