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

5 files changed, 30 insertions, 38 deletions

diff --git a/src/main/java/fr/umlv/java/wallj/context/Game.java b/src/main/java/fr/umlv/java/wallj/context/Game.java
index 422e9b7..46de147 100644
--- a/src/main/java/fr/umlv/java/wallj/context/Game.java
+++ b/src/main/java/fr/umlv/java/wallj/context/Game.java
@@ -1,10 +1,7 @@
 package fr.umlv.java.wallj.context;
 
 import fr.umlv.java.wallj.board.Board;
-import fr.umlv.java.wallj.event.ConfirmOrder;
-import fr.umlv.java.wallj.event.Event;
-import fr.umlv.java.wallj.event.Events;
-import fr.umlv.java.wallj.event.GameOverEvent;
+import fr.umlv.java.wallj.event.*;
 
 import java.util.*;
 
@@ -76,34 +73,38 @@ public final class Game implements Updateable {
     currentStage = new Stage(currentStage.getBoard());
   }
 
+
+
+  private void goToNext(){
+    if (hasNextBoard()) { //continue
+      nextStage();
+      return;
+    }
+    setOver();
+  }
+  private void handleEvents(Context context) {
+    if (Events.findFirst(context.getEvents(), QuitGameOrder.class).isPresent()) {
+      context.getGame().setOver();
+      return;
+    }
+    if (Events.findFirst(context.getEvents(), ConfirmOrder.class).isPresent()) {
+      if (currentStage.isCleared()) {
+        goToNext();
+        return;
+      }
+      retryStage();
+    }
+  }
+
+
   /**
    * @param context the current context
    * @return a list of new events
    */
   @Override
   public List<Event> update(Context context) {
-    boolean isConfirmOrder = Events.findFirst(context.getEvents(),ConfirmOrder.class).isPresent();
-    boolean isGameOverEvent = Events.findFirst(context.getEvents(),GameOverEvent.class).isPresent();
-    Game currentGame = context.getGame();
-    LinkedList<Event> events = new LinkedList<>();
-    if (isGameOverEvent) {
-      currentGame.setOver();
-    } else {
-      if (isConfirmOrder) {
-        if (false /*currentGame.getCurrentStage().isCleared()*/) { // FIXME: use StageClearedEvent
-          if (currentGame.hasNextBoard()) { //continue
-            currentGame.nextStage();
-          } else { //no more board so game over => exiting
-            currentGame.setOver();
-          }
-        } else {//retry
-          currentGame.retryStage();
-        }
-      }
-    }
-
-    // FIXME: update underlying stage and merge generated events
-    return events;
+    handleEvents(context);
+    return currentStage.update(context);
   }
 
 
diff --git a/src/main/java/fr/umlv/java/wallj/event/GameOverEvent.java b/src/main/java/fr/umlv/java/wallj/event/QuitGameOrder.java
index 4365dc4..2cabac1 100644
--- a/src/main/java/fr/umlv/java/wallj/event/GameOverEvent.java
+++ b/src/main/java/fr/umlv/java/wallj/event/QuitGameOrder.java
@@ -5,6 +5,5 @@ package fr.umlv.java.wallj.event;
  *
  * @author Pacien TRAN-GIRARD
  */
-public final class GameOverEvent implements GameEvent {
-  // void
+public final class QuitGameOrder implements InputEvent {
 }
diff --git a/src/main/java/fr/umlv/java/wallj/viewer/InputHandler.java b/src/main/java/fr/umlv/java/wallj/viewer/InputHandler.java
index a8f53d3..9eea443 100644
--- a/src/main/java/fr/umlv/java/wallj/viewer/InputHandler.java
+++ b/src/main/java/fr/umlv/java/wallj/viewer/InputHandler.java
@@ -3,7 +3,7 @@ package fr.umlv.java.wallj.viewer;
 import fr.umlv.java.wallj.board.TileVec2;
 import fr.umlv.java.wallj.event.BombSetupOrder;
 import fr.umlv.java.wallj.event.ConfirmOrder;
-import fr.umlv.java.wallj.event.GameOverEvent;
+import fr.umlv.java.wallj.event.QuitGameOrder;
 import fr.umlv.java.wallj.event.MoveRobotOrder;
 import fr.umlv.zen5.ApplicationContext;
 import fr.umlv.zen5.Event;
@@ -58,7 +58,7 @@ public final class InputHandler {
               events.add(new ConfirmOrder());
               break;
             case Q:
-              events.add(new GameOverEvent());
+              events.add(new QuitGameOrder());
               break;
           }
         }
diff --git a/src/main/java/fr/umlv/java/wallj/viewer/Main.java b/src/main/java/fr/umlv/java/wallj/viewer/Main.java
index ac1a088..01c6211 100644
--- a/src/main/java/fr/umlv/java/wallj/viewer/Main.java
+++ b/src/main/java/fr/umlv/java/wallj/viewer/Main.java
@@ -34,7 +34,6 @@ public class Main {
     return paths;
   }
 
-  // TODO: use System.getProperty("levels") to get the path to the level directory
   public static void main(String[] args) {
     List<Board> boards = new LinkedList<>();
     try {
diff --git a/src/test/java/fr/umlv/java/wallj/viewer/ViewerTest.java b/src/test/java/fr/umlv/java/wallj/viewer/ViewerTest.java
deleted file mode 100644
index ca9ee0c..0000000
--- a/src/test/java/fr/umlv/java/wallj/viewer/ViewerTest.java
+++ /dev/null
@@ -1,7 +0,0 @@
-package fr.umlv.java.wallj.viewer;
-
-import org.junit.jupiter.api.Test;
-
-public class ViewerTest {
-
-}