diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/client/ui/gameui/CasinoGameUI.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/client/ui/gameui/CasinoGameUI.java index 2970e70..d6908c1 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/client/ui/gameui/CasinoGameUI.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/client/ui/gameui/CasinoGameUI.java @@ -1,5 +1,6 @@ package ch.unibas.dmi.dbis.cs108.casono.client.ui.gameui; +import ch.unibas.dmi.dbis.cs108.casono.client.ClientApp; import ch.unibas.dmi.dbis.cs108.casono.client.game.GameService; import ch.unibas.dmi.dbis.cs108.casono.client.game.PlayerId; import ch.unibas.dmi.dbis.cs108.casono.client.network.ClientService; @@ -77,8 +78,7 @@ public class CasinoGameUI extends Application { public void start(Stage stage) throws IOException { if (clientService == null) { - clientService = - ch.unibas.dmi.dbis.cs108.casono.client.ClientApp.getSharedClientService(); + clientService = ClientApp.getSharedClientService(); } if (clientService == null) { throw new IllegalStateException( @@ -90,8 +90,7 @@ public class CasinoGameUI extends Application { String effectiveUsername = normalize(username); if (effectiveUsername == null) { - effectiveUsername = - normalize(ch.unibas.dmi.dbis.cs108.casono.client.ClientApp.getSharedUsername()); + effectiveUsername = normalize(ClientApp.getSharedUsername()); } if (effectiveUsername == null) { @@ -105,7 +104,7 @@ public class CasinoGameUI extends Application { + "', injectedUsername='" + username + "', sharedUsername='" - + ch.unibas.dmi.dbis.cs108.casono.client.ClientApp.getSharedUsername() + + ClientApp.getSharedUsername() + "', hasClientService=" + (clientService != null)); diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/client/ui/lobbyui/CasinomainuiController.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/client/ui/lobbyui/CasinomainuiController.java index db644f4..b3c2297 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/client/ui/lobbyui/CasinomainuiController.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/client/ui/lobbyui/CasinomainuiController.java @@ -55,9 +55,9 @@ public class CasinomainuiController { } /** - * Initializes the UI components and sets default values. If a shared {@link - * ch.unibas.dmi.dbis.cs108.casono.client.network.ClientService} exists (created at application - * start), the controller reuses it so the connection remains open and already-logged-in. + * Initializes the UI components and sets default values. If a shared {@link ClientService} + * exists (created at application start), the controller reuses it so the connection remains + * open and already-logged-in. */ @FXML public void initialize() { diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/client/ui/lobbyui/LobbyButtonGridManager.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/client/ui/lobbyui/LobbyButtonGridManager.java index 116686f..e3f1d01 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/client/ui/lobbyui/LobbyButtonGridManager.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/client/ui/lobbyui/LobbyButtonGridManager.java @@ -1,7 +1,9 @@ package ch.unibas.dmi.dbis.cs108.casono.client.ui.lobbyui; +import ch.unibas.dmi.dbis.cs108.casono.client.ClientApp; import ch.unibas.dmi.dbis.cs108.casono.client.network.ClientService; import ch.unibas.dmi.dbis.cs108.casono.client.network.LobbyClient; +import ch.unibas.dmi.dbis.cs108.casono.client.ui.gameui.CasinoGameUI; import ch.unibas.dmi.dbis.cs108.casono.server.network.command.parsing.RequestParameter; import java.util.ArrayList; import java.util.Collections; @@ -553,14 +555,10 @@ public class LobbyButtonGridManager { try { var cs = lobbyClient.getClientService(); - ch.unibas.dmi.dbis.cs108.casono.client.ui.gameui.CasinoGameUI - .setClientService(cs); - ch.unibas.dmi.dbis.cs108.casono.client.ui.gameui.CasinoGameUI.setLobbyId( - lobbyId); + CasinoGameUI.setClientService(cs); + CasinoGameUI.setLobbyId(lobbyId); - String username = - ch.unibas.dmi.dbis.cs108.casono.client.ClientApp - .getSharedUsername(); + String username = ClientApp.getSharedUsername(); if (username == null || username.isBlank()) { username = "Guest-" @@ -570,11 +568,9 @@ public class LobbyButtonGridManager { .toString() .substring(0, GUEST_ID_LENGTH); } - ch.unibas.dmi.dbis.cs108.casono.client.ui.gameui.CasinoGameUI.setUsername( - username); + CasinoGameUI.setUsername(username); - new ch.unibas.dmi.dbis.cs108.casono.client.ui.gameui.CasinoGameUI() - .start(gameStage); + new CasinoGameUI().start(gameStage); } catch (Exception e) { LOGGER.error("Game UI failed: {}", e.getMessage()); diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/game/bet/PlayerBetHandler.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/game/bet/PlayerBetHandler.java index ea39326..a37ebce 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/game/bet/PlayerBetHandler.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/game/bet/PlayerBetHandler.java @@ -4,6 +4,7 @@ import ch.unibas.dmi.dbis.cs108.casono.server.domain.game.GameController; import ch.unibas.dmi.dbis.cs108.casono.server.domain.game.player.PlayerId; import ch.unibas.dmi.dbis.cs108.casono.server.domain.lobby.LobbyId; import ch.unibas.dmi.dbis.cs108.casono.server.domain.lobby.LobbyManager; +import ch.unibas.dmi.dbis.cs108.casono.server.domain.user.User; import ch.unibas.dmi.dbis.cs108.casono.server.domain.user.UserRegistry; import ch.unibas.dmi.dbis.cs108.casono.server.network.command.execution.CommandHandler; import ch.unibas.dmi.dbis.cs108.casono.server.network.protocol.response.ErrorResponse; @@ -58,8 +59,7 @@ public class PlayerBetHandler extends CommandHandler { return; } - Optional opt = - userRegistry.getBySessionId(request.getSessionId()); + Optional opt = userRegistry.getBySessionId(request.getSessionId()); if (opt.isEmpty()) { responseDispatcher.dispatch( new ErrorResponse(request.getContext(), "NOT_LOGGED_IN", "User not logged in")); diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/game/call/PlayerCallHandler.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/game/call/PlayerCallHandler.java index cdaffc9..b1c3618 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/game/call/PlayerCallHandler.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/game/call/PlayerCallHandler.java @@ -4,6 +4,7 @@ import ch.unibas.dmi.dbis.cs108.casono.server.domain.game.GameController; import ch.unibas.dmi.dbis.cs108.casono.server.domain.game.player.PlayerId; import ch.unibas.dmi.dbis.cs108.casono.server.domain.lobby.LobbyId; import ch.unibas.dmi.dbis.cs108.casono.server.domain.lobby.LobbyManager; +import ch.unibas.dmi.dbis.cs108.casono.server.domain.user.User; import ch.unibas.dmi.dbis.cs108.casono.server.domain.user.UserRegistry; import ch.unibas.dmi.dbis.cs108.casono.server.network.command.execution.CommandHandler; import ch.unibas.dmi.dbis.cs108.casono.server.network.protocol.response.ErrorResponse; @@ -46,8 +47,7 @@ public class PlayerCallHandler extends CommandHandler { */ @Override public void execute(PlayerCallRequest request) { - Optional opt = - userRegistry.getBySessionId(request.getSessionId()); + Optional opt = userRegistry.getBySessionId(request.getSessionId()); if (opt.isEmpty()) { responseDispatcher.dispatch( new ErrorResponse(request.getContext(), "NOT_LOGGED_IN", "User not logged in")); diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/game/fold/PlayerFoldHandler.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/game/fold/PlayerFoldHandler.java index 0db3628..7adca85 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/game/fold/PlayerFoldHandler.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/game/fold/PlayerFoldHandler.java @@ -4,6 +4,7 @@ import ch.unibas.dmi.dbis.cs108.casono.server.domain.game.GameController; import ch.unibas.dmi.dbis.cs108.casono.server.domain.game.player.PlayerId; import ch.unibas.dmi.dbis.cs108.casono.server.domain.lobby.LobbyId; import ch.unibas.dmi.dbis.cs108.casono.server.domain.lobby.LobbyManager; +import ch.unibas.dmi.dbis.cs108.casono.server.domain.user.User; import ch.unibas.dmi.dbis.cs108.casono.server.domain.user.UserRegistry; import ch.unibas.dmi.dbis.cs108.casono.server.network.command.execution.CommandHandler; import ch.unibas.dmi.dbis.cs108.casono.server.network.protocol.response.ErrorResponse; @@ -46,8 +47,7 @@ public class PlayerFoldHandler extends CommandHandler { */ @Override public void execute(PlayerFoldRequest request) { - Optional opt = - userRegistry.getBySessionId(request.getSessionId()); + Optional opt = userRegistry.getBySessionId(request.getSessionId()); if (opt.isEmpty()) { responseDispatcher.dispatch( new ErrorResponse(request.getContext(), "NOT_LOGGED_IN", "User not logged in")); diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/game/raise/PlayerRaiseHandler.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/game/raise/PlayerRaiseHandler.java index 6fbaaca..c17b11c 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/game/raise/PlayerRaiseHandler.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/game/raise/PlayerRaiseHandler.java @@ -4,6 +4,7 @@ import ch.unibas.dmi.dbis.cs108.casono.server.domain.game.GameController; import ch.unibas.dmi.dbis.cs108.casono.server.domain.game.player.PlayerId; import ch.unibas.dmi.dbis.cs108.casono.server.domain.lobby.LobbyId; import ch.unibas.dmi.dbis.cs108.casono.server.domain.lobby.LobbyManager; +import ch.unibas.dmi.dbis.cs108.casono.server.domain.user.User; import ch.unibas.dmi.dbis.cs108.casono.server.domain.user.UserRegistry; import ch.unibas.dmi.dbis.cs108.casono.server.network.command.execution.CommandHandler; import ch.unibas.dmi.dbis.cs108.casono.server.network.protocol.response.ErrorResponse; @@ -60,8 +61,7 @@ public class PlayerRaiseHandler extends CommandHandler { return; } - Optional opt = - userRegistry.getBySessionId(request.getSessionId()); + Optional opt = userRegistry.getBySessionId(request.getSessionId()); if (opt.isEmpty()) { responseDispatcher.dispatch( new ErrorResponse(request.getContext(), "NOT_LOGGED_IN", "User not logged in")); diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/lobby/get_lobby_list/GetLobbyListHandler.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/lobby/get_lobby_list/GetLobbyListHandler.java index 49e49d8..8cc9759 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/lobby/get_lobby_list/GetLobbyListHandler.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/lobby/get_lobby_list/GetLobbyListHandler.java @@ -1,5 +1,6 @@ package ch.unibas.dmi.dbis.cs108.casono.server.app.commands.lobby.get_lobby_list; +import ch.unibas.dmi.dbis.cs108.casono.server.domain.lobby.Lobby; import ch.unibas.dmi.dbis.cs108.casono.server.domain.lobby.LobbyManager; import ch.unibas.dmi.dbis.cs108.casono.server.network.command.execution.CommandHandler; import ch.unibas.dmi.dbis.cs108.casono.server.network.protocol.response.dispatcher.ResponseDispatcher; @@ -21,8 +22,7 @@ public class GetLobbyListHandler extends CommandHandler { @Override public void execute(GetLobbyListRequest request) { - Collection l = - lobbyManager.getAllLobbies(); + Collection l = lobbyManager.getAllLobbies(); responseDispatcher.dispatch(new GetLobbyListResponse(request.getContext(), l)); } } diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/lobby/start_game/StartGameHandler.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/lobby/start_game/StartGameHandler.java index eb1f335..7c24449 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/lobby/start_game/StartGameHandler.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/lobby/start_game/StartGameHandler.java @@ -11,6 +11,7 @@ import ch.unibas.dmi.dbis.cs108.casono.server.domain.game.state.GameState; import ch.unibas.dmi.dbis.cs108.casono.server.domain.lobby.Lobby; import ch.unibas.dmi.dbis.cs108.casono.server.domain.lobby.LobbyId; import ch.unibas.dmi.dbis.cs108.casono.server.domain.lobby.LobbyManager; +import ch.unibas.dmi.dbis.cs108.casono.server.domain.user.User; import ch.unibas.dmi.dbis.cs108.casono.server.domain.user.UserRegistry; import ch.unibas.dmi.dbis.cs108.casono.server.network.command.execution.CommandHandler; import ch.unibas.dmi.dbis.cs108.casono.server.network.protocol.response.ErrorResponse; @@ -38,8 +39,7 @@ public class StartGameHandler extends CommandHandler { @Override public void execute(StartGameRequest request) { - Optional opt = - userRegistry.getBySessionId(request.getSessionId()); + Optional opt = userRegistry.getBySessionId(request.getSessionId()); if (opt.isEmpty()) { // Guard: UserLoggedInCheck should normally handle this responseDispatcher.dispatch( diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/send_message/SendMessageHandler.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/send_message/SendMessageHandler.java index 680819a..915ded1 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/send_message/SendMessageHandler.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/send_message/SendMessageHandler.java @@ -5,6 +5,7 @@ import ch.unibas.dmi.dbis.cs108.casono.client.chat.Message; import ch.unibas.dmi.dbis.cs108.casono.server.domain.lobby.Lobby; import ch.unibas.dmi.dbis.cs108.casono.server.domain.lobby.LobbyId; import ch.unibas.dmi.dbis.cs108.casono.server.domain.lobby.LobbyManager; +import ch.unibas.dmi.dbis.cs108.casono.server.domain.user.User; import ch.unibas.dmi.dbis.cs108.casono.server.domain.user.UserRegistry; import ch.unibas.dmi.dbis.cs108.casono.server.network.command.execution.CommandHandler; import ch.unibas.dmi.dbis.cs108.casono.server.network.protocol.response.OkResponse; @@ -40,8 +41,7 @@ public class SendMessageHandler extends CommandHandler { @Override public void execute(SendMessageRequest request) { Message message = request.getMessage(); - Optional senderUser = - userRegistry.getBySessionId(request.getSessionId()); + Optional senderUser = userRegistry.getBySessionId(request.getSessionId()); if (senderUser.isPresent()) { message.sender = senderUser.get().getName(); } diff --git a/src/main/resources/ui-structure/Casinogameui.fxml b/src/main/resources/ui-structure/Casinogameui.fxml index 1baf1ae..a5d5514 100644 --- a/src/main/resources/ui-structure/Casinogameui.fxml +++ b/src/main/resources/ui-structure/Casinogameui.fxml @@ -5,6 +5,7 @@ + + diff --git a/src/main/resources/ui-structure/gameuicomponents/Taskbar.fxml b/src/main/resources/ui-structure/gameuicomponents/Taskbar.fxml index 86064a3..476c584 100644 --- a/src/main/resources/ui-structure/gameuicomponents/Taskbar.fxml +++ b/src/main/resources/ui-structure/gameuicomponents/Taskbar.fxml @@ -3,11 +3,10 @@ - +