fix(polling): Improve polling mechanism and delay handling (#60)

Reviewed-on: #60

knockoutwhistweb/app/components/WebApplicationConfiguration.scala

@@ -2,11 +2,12 @@ package components
 
 import de.knockoutwhist.components.DefaultConfiguration
 import de.knockoutwhist.ui.UI
+import de.knockoutwhist.utils.DelayHandler
 import de.knockoutwhist.utils.events.EventListener
 
 class WebApplicationConfiguration extends DefaultConfiguration {
 
   override def uis: Set[UI] = Set()
-  override def listener: Set[EventListener] = Set()
+  override def listener: Set[EventListener] = Set(DelayHandler)
 
 }

knockoutwhistweb/app/controllers/IngameController.scala

@@ -59,7 +59,6 @@ class IngameController @Inject() (
       case Some(g) =>
         val results = Try {
           returnInnerHTML(g, request.user)
-
         }
         if (results.isSuccess) {
           Ok(views.html.main("In-Game - Knockout Whist")(results.get))
@@ -67,7 +66,7 @@ class IngameController @Inject() (
           InternalServerError(results.failed.get.getMessage)
         }
       case None =>
-        NotFound("Game not found")
+        Redirect(routes.MainMenuController.mainMenu())
     }
   }
   def startGame(gameId: String): Action[AnyContent] = authAction { implicit request: AuthenticatedRequest[AnyContent] =>

knockoutwhistweb/app/controllers/PollingController.scala

@@ -78,7 +78,8 @@ class PollingController @Inject() (
         "name" -> u.name,
         "id" -> u.id,
         "self" -> (u.id == userSession.id)
-      ))
+      )),
+      "maxPlayers" -> game.maxPlayers
     )
   }
 

knockoutwhistweb/app/logic/game/GameLobby.scala

@@ -119,15 +119,6 @@ class GameLobby private(
         }
       }
     }
-
-    waitingPromises.keys.foreach { playerId =>
-      val queue =  eventsPerPlayer(playerId)
-      if (queue.nonEmpty) {
-        val promise = waitingPromises(playerId)
-        promise.success(queue.dequeue())
-        waitingPromises.remove(playerId)
-      }
-    }
   }
 
   /**

knockoutwhistweb/app/views/lobby/lobby.scala.html

@@ -14,7 +14,7 @@
         </div>
         <div class="row">
             <div class="col">
-                <div class="p-3 text-center fs-4">Playeramount: @gamelobby.getPlayers.size / @gamelobby.maxPlayers</div>
+                <div class="p-3 text-center fs-4" id="playerAmount">Playeramount: @gamelobby.getPlayers.size / @gamelobby.maxPlayers</div>
             </div>
         </div>
         <div class="row justify-content-center align-items-center flex-grow-1">

knockoutwhistweb/public/javascripts/main.js

@@ -79,7 +79,13 @@
     })
 })()
 
+let polling = false;
 function pollForUpdates(gameId) {
+    if (polling) {
+        console.log("[DEBUG] Polling already in progress. Skipping this cycle.");
+        return;
+    }
+    polling = true;
     console.log(`[DEBUG] Starting poll cycle for Game ID: ${gameId} at ${new Date().toISOString()}`);
     if (!gameId) {
         console.error("[DEBUG] Game ID is missing. Stopping poll.");
@@ -90,7 +96,7 @@ function pollForUpdates(gameId) {
     const $mainmenuElement = $('#main-menu-screen')
     const $mainbody = $('#main-body')
     if (!$handElement.length && !$lobbyElement.length && !$mainmenuElement.length && !$mainbody.length) {
-        setTimeout(() => pollForUpdates(gameId), 1000);
+        setTimeout(() => { polling = false; pollForUpdates(gameId) }, 1000);
         return;
     }
     const route = jsRoutes.controllers.PollingController.polling(gameId);
@@ -254,6 +260,7 @@ function pollForUpdates(gameId) {
                     })
                 }
                 $("#players").html(newHtml);
+                $('#playerAmount').text(`Playeramount: ${data.users.length} / ${data.maxPlayers}`);
             } else {
                 console.warn(`[DEBUG] Received unknown status: ${data.status}`);
             }
@@ -267,11 +274,7 @@ function pollForUpdates(gameId) {
             }
         }),
         complete: (() => {
-            if (!window.location.href.includes("game")) {
-                console.log("[DEBUG] Page URL changed. Stopping poll restart.");
-                return;
-            }
-            setTimeout(() => pollForUpdates(gameId), 200);
+            setTimeout(() => { polling = false; pollForUpdates(gameId) }, 200);
         })
     })
 }