Bug report from @Illedan

[troll.git] / src / main / java / com / codingame / game / Referee.java

diff --git a/src/main/java/com/codingame/game/Referee.java b/src/main/java/com/codingame/game/Referee.java
index b907739..67d96e0 100644 (file)
--- a/src/main/java/com/codingame/game/Referee.java
+++ b/src/main/java/com/codingame/game/Referee.java
@@ -9,29 +9,29 @@ import com.codingame.gameengine.core.AbstractPlayer.TimeoutException;
 import com.codingame.gameengine.core.AbstractReferee;
 import com.codingame.gameengine.core.GameManager;
 import com.codingame.gameengine.core.MultiplayerGameManager;
 import com.codingame.gameengine.core.AbstractReferee;
 import com.codingame.gameengine.core.GameManager;
 import com.codingame.gameengine.core.MultiplayerGameManager;

-import com.codingame.gameengine.module.entities.GraphicEntityModule;
-import com.codingame.gameengine.module.entities.Rectangle;
-import com.codingame.gameengine.module.entities.Sprite;
-import com.codingame.gameengine.module.entities.Text;
-import com.codingame.gameengine.module.entities.Curve;
+import com.codingame.game.GodModeManager;

 import com.google.inject.Inject;
 import com.google.inject.Provider;
 
 public class Referee extends AbstractReferee {
     @Inject private MultiplayerGameManager<Player> gameManager;
 import com.google.inject.Inject;
 import com.google.inject.Provider;
 
 public class Referee extends AbstractReferee {
     @Inject private MultiplayerGameManager<Player> gameManager;

-    @Inject private GraphicEntityModule graphicEntityModule;
+    @Inject private GodModeManager gm;

 
     @Inject private View view;
     @Inject private Model model;
 
 
     @Inject private View view;
     @Inject private Model model;
 

+    boolean disqual = false;
+

     @Override
     public void init() {
     @Override
     public void init() {

+        gm.init();

         model.init(gameManager.getSeed());
         gameManager.getPlayer(0).model = model.p0;
         gameManager.getPlayer(1).model = model.p1;
 
         for (Player p: gameManager.getPlayers()) {
         model.init(gameManager.getSeed());
         gameManager.getPlayer(0).model = model.p0;
         gameManager.getPlayer(1).model = model.p1;
 
         for (Player p: gameManager.getPlayers()) {

-            p.gameInit(model.roadLength, model.initialStones);
+            p.gameInit(model.roadLength, model.initialStones,
+                       gameManager.getSeed(), gm.getSalt());

         }
 
         view.init(model);
         }
 
         view.init(model);


@@ -52,6 +52,11 @@ public class Referee extends AbstractReferee {
 
         view.startTurn();
 
 
         view.startTurn();
 

+        // Did I mention I hate Java? It didn't *have* to be this ugly!
+        if (disqual) { endGame(); return; }
+        if (model.exhausted()) { finishStones(); return ;}
+        if (model.haveWinner()) { endGame(); return; }
+

         for (Player player : gameManager.getActivePlayers()) {
             player.sendGameTurn();
         }
         for (Player player : gameManager.getActivePlayers()) {
             player.sendGameTurn();
         }


@@ -62,33 +67,35 @@ public class Referee extends AbstractReferee {
          * is ill-formed it could help them debug.  Or shame them, at
          * least.
          */
          * is ill-formed it could help them debug.  Or shame them, at
          * least.
          */

-        boolean disqual = false;

         for (Player player : gameManager.getActivePlayers()) {
         for (Player player : gameManager.getActivePlayers()) {

-            player.receiveGameTurn();
+            player.receiveGameTurn(); gm.transcend(player);

             switch (player.type) {
             case Timeout:
                 disqualify(player, "T/O", "timed out!");
             switch (player.type) {
             case Timeout:
                 disqualify(player, "T/O", "timed out!");

+                player.view.markTimeout();

                 disqual = true;
                 break;
             case Invalid:
                 disqualify(player, "INVALID", "provided an ill-formed action");
                 disqual = true;
                 break;
             case Invalid:
                 disqualify(player, "INVALID", "provided an ill-formed action");

+                player.view.markIllegal();

                 disqual = true;
                 break;
             case Throw:
                 try { player.model.consumeStones(player.stoneThrow); }
                 catch (Model.Player.ThrewMoreStonesThanHad e) {
                     if (model.random.nextInt(10) > 0) {
                 disqual = true;
                 break;
             case Throw:
                 try { player.model.consumeStones(player.stoneThrow); }
                 catch (Model.Player.ThrewMoreStonesThanHad e) {
                     if (model.random.nextInt(10) > 0) {

-                        gameManager.addToGameSummary(GameManager.formatErrorMessage(player.getNicknameToken() + " tried to throw more stones than they had.  I'll let it slide for this time.  (But not let them throw that much!)"));
+                        player.view.threwMoreStonesThanHad();

                         player.stoneThrow = player.model.consumeMaxStones();
                     }
                     else {
                         disqualify(player, "ILLEGAL", "tried to throw more stones than they had.  They went into debt trying to provide.  The economy tanked, recession and famine ensued; even the troll wouldn't have wanted to bash them anymore.  But that's no victory.");
                         player.stoneThrow = player.model.consumeMaxStones();
                     }
                     else {
                         disqualify(player, "ILLEGAL", "tried to throw more stones than they had.  They went into debt trying to provide.  The economy tanked, recession and famine ensued; even the troll wouldn't have wanted to bash them anymore.  But that's no victory.");

+                        player.view.markIllegal();

                         disqual = true;
                     }
                 }
                 catch (Model.Player.FailedToThrowStonesAndShouldHave e) {
                     if (model.random.nextInt(10) > 0) {
                         disqual = true;
                     }
                 }
                 catch (Model.Player.FailedToThrowStonesAndShouldHave e) {
                     if (model.random.nextInt(10) > 0) {

-                        gameManager.addToGameSummary(GameManager.formatErrorMessage(player.getNicknameToken() + " tried not throwing any stones.  Fixing that for them because I'm in a good mood today."));
+                        player.view.failedToThrowStonesAndShouldHave();

                         player.stoneThrow = player.model.consumeMinStones();
                     }
                     else {
                         player.stoneThrow = player.model.consumeMinStones();
                     }
                     else {


@@ -112,70 +119,82 @@ public class Referee extends AbstractReferee {
          * exhaustion).
          */
         int delta = 0;
          * exhaustion).
          */
         int delta = 0;

-        boolean victory = false;
-        boolean exhausted = false;
-        if (! disqual) {
-            for (Player player : gameManager.getActivePlayers()) {
-                gameManager.addToGameSummary(String.format("%s throws %d stone%s at the troll.", player.getNicknameToken(), player.stoneThrow, player.stoneThrow == 1 ? "" : "s"));
-                delta += player.model.getMultiplier() * player.stoneThrow;
-
-                if (player.stoneThrow < 0) {
-                    disqualify(player, "CHEAT", "cheated.  Banning account.");
-                    disqual = true;
-                }
-                else if (player.stoneThrow > 0) {
-                    player.view.animateStones(player.stoneThrow);
-                    player.view.updateStoneCounter();
-                }
-            }
+        gm.update(gameManager.getPlayers());
+        for (Player player : gameManager.getActivePlayers()) {
+            player.view.throwStones(player.stoneThrow);
+            delta += player.model.getMultiplier() * player.stoneThrow;

 
 

-            /* If a player cheated, delta is unusable as is.
-             * (Consider the case the player on the right sent
-             * INT_MIN.  INT_MIN * (-1) = INT_MIN, so that player
-             * would both glean the stones *and* push the troll away.
-             * It would be unfair to have a cheating player "win"
-             * (earn the opponent castle destruction animation) this
-             * way.
-             */
-            boolean cheat0 = gameManager.getPlayer(0).stoneThrow < 0;
-            boolean cheat1 = gameManager.getPlayer(1).stoneThrow < 0;
-            if (cheat0 && cheat1); // here we can actually keep delta's value
-            else if (cheat0) delta = -1;
-            else if (cheat1) delta =  1;
-
-            if (delta > 0) {
-                gameManager.addToGameSummary("Troll walks right.");
-                model.trollPosition++;
-            }
-            else if (delta < 0) {
-                gameManager.addToGameSummary("Troll walks left.");
-                model.trollPosition--;
+            if (player.stoneThrow < 0) {
+                disqualify(player, "CHEAT", "cheated.  Banning account.");
+                player.view.markCheat();
+                disqual = true;

             }
             }

-            else {
-                gameManager.addToGameSummary("Troll stands still.");
-                // XXX animate
+            if (player.stoneThrow != 0) {
+                player.view.animateStones(player.stoneThrow);
+                player.view.updateStoneCounter();

             }
             }

-            view.moveTroll();
+        }

 
 

-            for (Player player : gameManager.getActivePlayers()) {
-                player.model.adjustScore(model.trollPosition);
-            }
+        /* If a player cheated, delta is unusable as is.
+         * (Consider the case the player on the right sent
+         * INT_MIN.  INT_MIN * (-1) = INT_MIN, so that player
+         * would both glean the stones *and* push the troll away.
+         * It would be unfair to have a cheating player "win"
+         * (earn the opponent castle destruction animation) this
+         * way.
+         */
+        boolean cheat0 = gameManager.getPlayer(0).isActive()
+            && gameManager.getPlayer(0).stoneThrow < 0;
+        boolean cheat1 = gameManager.getPlayer(1).isActive()
+            && gameManager.getPlayer(1).stoneThrow < 0;
+        if (cheat0 && cheat1); // here we can actually keep delta's value
+        else if (cheat0) delta = -1;
+        else if (cheat1) delta =  1;
+
+        if (delta > 0) {
+            model.moveTroll(+1);
+            view.moveTroll(View.Dir.RIGHT);
+        }
+        else if (delta < 0) {
+            model.moveTroll(-1);
+            view.moveTroll(View.Dir.LEFT);
+        }
+        else {
+            view.moveTroll(View.Dir.STILL);
+            // XXX animate
+        }
+    }

 
 

-            if (model.haveWinner()) {
-                int loser = model.getLoser();
-                gameManager.addToGameSummary(GameManager.formatErrorMessage("Troll destroys " + gameManager.getPlayer(loser).getNicknameToken()) + ".");
-                gameManager.getPlayer(loser).view.destroy();
-                victory = true;
+    // XXX very similar to main turn pendant
+    private void finishStones() {
+        boolean noStones = true;
+        int delta = 0;
+        for (Player player : gameManager.getActivePlayers()) {
+            if (model.haveWinner() && player.getIndex() == model.getLoser())
+                continue;
+            player.stoneThrow = player.model.getStones();
+            player.model.setStones(0);
+            delta += player.stoneThrow * player.model.getMultiplier();
+            player.view.throwStones(player.stoneThrow);
+            if (player.stoneThrow != 0) {
+                noStones = false;
+                player.view.animateStones(player.stoneThrow);
+                player.view.updateStoneCounter();

             }
             }

-            else if (model.exhausted()) exhausted = true;

         }
         }

-
-        if (disqual || victory || exhausted) endGame();
+        if (noStones) { endGame(); return; }
+        model.moveTroll(delta);
+        view.moveTroll();

     }
 
     private void endGame() {
         gameManager.endGame();
 
     }
 
     private void endGame() {
         gameManager.endGame();
 

+        if (model.haveWinner()) {
+            int loser = model.getLoser();
+            gameManager.getPlayer(loser).view.defeat();
+        }
+

         Player p0 = gameManager.getPlayer(0);
         Player p1 = gameManager.getPlayer(1);
 
         Player p0 = gameManager.getPlayer(0);
         Player p1 = gameManager.getPlayer(1);
 


@@ -183,20 +202,20 @@ public class Referee extends AbstractReferee {
         int s1 = p1.getScore();
 
         if (s0 > s1) {
         int s1 = p1.getScore();
 
         if (s0 > s1) {

-            gameManager.addToGameSummary(GameManager.formatSuccessMessage(p0.getNicknameToken() + " wins."));
+            p0.view.victory();

             p1.view.markLoser();
         }
         else if (s0 < s1) {
             p1.view.markLoser();
         }
         else if (s0 < s1) {

-            gameManager.addToGameSummary(GameManager.formatSuccessMessage(p1.getNicknameToken() + " wins."));
+            p1.view.victory();

             p0.view.markLoser();
         }
         else if (s0 < 0) {
             p0.view.markLoser();
         }
         else if (s0 < 0) {

-            gameManager.addToGameSummary(GameManager.formatErrorMessage("Everybody loses!"));
+            view.doubleDefeat();

             p0.view.markLoser();
             p1.view.markLoser();
         }
         else {
             p0.view.markLoser();
             p1.view.markLoser();
         }
         else {

-            gameManager.addToGameSummary("Draw.");
+            view.draw();

         }
     }
 }
         }
     }
 }