MusicAssetTests

Applied template method pattern to abstract player

.idea/inspectionProfiles/Project_Default.xml

@@ -4,6 +4,9 @@
     <inspection_tool class="AutoCloseableResource" enabled="true" level="WARNING" enabled_by_default="true">
       <option name="METHOD_MATCHER_CONFIG" value="java.util.Formatter,format,java.io.Writer,append,com.google.common.base.Preconditions,checkNotNull,org.hibernate.Session,close,java.io.PrintWriter,printf,java.io.PrintStream,printf,java.lang.foreign.Arena,ofAuto,java.lang.foreign.Arena,global,org.toop.framework.audio.AudioPlayer,play,java.util.Map,remove,java.util.concurrent.Executors,newSingleThreadScheduledExecutor" />
     </inspection_tool>
+    <inspection_tool class="ThrowablePrintStackTrace" enabled="true" level="WARNING" enabled_by_default="true">
+      <scope name="Tests" level="WARNING" enabled="false" />
+    </inspection_tool>
     <inspection_tool class="WriteOnlyObject" enabled="false" level="WARNING" enabled_by_default="false" />
   </profile>
 </component>

app/src/main/java/org/toop/app/App.java

@@ -1,14 +1,10 @@
 package org.toop.app;
 
-import javafx.application.Application;
 import javafx.application.Platform;
-import javafx.geometry.Pos;
-import javafx.scene.Scene;
 import javafx.scene.input.KeyCode;
 import javafx.scene.input.KeyCodeCombination;
 import javafx.scene.input.KeyEvent;
-import javafx.scene.layout.StackPane;
-import javafx.stage.Stage;
+
 import org.toop.app.widget.Primitive;
 import org.toop.app.widget.WidgetContainer;
 import org.toop.app.widget.complex.LoadingWidget;
@@ -20,8 +16,8 @@ import org.toop.framework.audio.*;
 import org.toop.framework.audio.events.AudioEvents;
 import org.toop.framework.eventbus.EventFlow;
 import org.toop.framework.eventbus.GlobalEventBus;
-import org.toop.framework.networking.connection.NetworkingClientEventListener;
-import org.toop.framework.networking.connection.NetworkingClientManager;
+import org.toop.framework.networking.NetworkingClientEventListener;
+import org.toop.framework.networking.NetworkingClientManager;
 import org.toop.framework.resource.ResourceLoader;
 import org.toop.framework.resource.ResourceManager;
 import org.toop.framework.resource.events.AssetLoaderEvents;
@@ -31,6 +27,12 @@ import org.toop.framework.resource.resources.SoundEffectAsset;
 import org.toop.local.AppContext;
 import org.toop.local.AppSettings;
 
+import javafx.application.Application;
+import javafx.geometry.Pos;
+import javafx.scene.Scene;
+import javafx.scene.layout.StackPane;
+import javafx.stage.Stage;
+
 import java.util.Objects;
 import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.ExecutorService;
@@ -84,7 +86,7 @@ public final class App extends Application {
 
 		AppSettings.applySettings();
 
-        setKeybinds(root);
+		setKeybinds(root);
 
         LoadingWidget loading = new LoadingWidget(Primitive.text(
                 "Loading...", false), 0, 0, Integer.MAX_VALUE, false, false // Just set a high default
@@ -92,7 +94,7 @@ public final class App extends Application {
 
 		WidgetContainer.setCurrentView(loading);
 
-        setOnLoadingSuccess(loading);
+		setOnLoadingSuccess(loading);
 
         EventFlow loadingFlow = new EventFlow();
 
@@ -135,12 +137,13 @@ public final class App extends Application {
 		}
 
 		stage.show();
-    }
+
+	}
 
 	private void setKeybinds(StackPane root) {
 		root.addEventHandler(KeyEvent.KEY_PRESSED,event -> {
 			if (event.getCode() == KeyCode.ESCAPE) {
-				escapePopup();	
+				escapePopup();
 			}
 		});
         stage.setFullScreenExitKeyCombination(

app/src/main/java/org/toop/app/Server.java

@@ -14,14 +14,14 @@ import org.toop.framework.eventbus.EventFlow;
 import org.toop.framework.gameFramework.controller.GameController;
 import org.toop.framework.eventbus.GlobalEventBus;
 import org.toop.framework.gameFramework.model.player.Player;
-import org.toop.framework.networking.connection.clients.TournamentNetworkingClient;
-import org.toop.framework.networking.connection.events.NetworkEvents;
-import org.toop.framework.networking.connection.types.NetworkingConnector;
-import org.toop.framework.game.games.reversi.BitboardReversi;
-import org.toop.framework.game.games.tictactoe.BitboardTicTacToe;
-import org.toop.framework.game.players.ArtificialPlayer;
-import org.toop.framework.game.players.OnlinePlayer;
-import org.toop.framework.game.players.RandomAI;
+import org.toop.framework.networking.clients.TournamentNetworkingClient;
+import org.toop.framework.networking.events.NetworkEvents;
+import org.toop.framework.networking.types.NetworkingConnector;
+import org.toop.game.games.reversi.BitboardReversi;
+import org.toop.game.games.tictactoe.BitboardTicTacToe;
+import org.toop.game.players.ArtificialPlayer;
+import org.toop.game.players.OnlinePlayer;
+import org.toop.game.players.ai.RandomAI;
 import org.toop.local.AppContext;
 
 import java.util.List;
@@ -214,16 +214,16 @@ public final class Server {
 
             switch (type) {
                 case TICTACTOE ->{
-                        Player[] players = new Player[2];
-                        players[(myTurn + 1) % 2] = new OnlinePlayer(response.opponent());
-                        players[myTurn] = new ArtificialPlayer(new RandomAI(), user);
+                        Player<BitboardTicTacToe>[] players = new Player[2];
+                        players[(myTurn + 1) % 2] = new OnlinePlayer<>(response.opponent());
+                        players[myTurn] = new ArtificialPlayer<>(new RandomAI<BitboardTicTacToe>(), user);
                         gameController = new TicTacToeBitController(players);
                 }
                 case REVERSI -> {
-                    Player[] players = new Player[2];
-                    players[(myTurn + 1) % 2] = new OnlinePlayer(response.opponent());
-                    players[myTurn] = new ArtificialPlayer(new RandomAI(), user);
-                    gameController = new ReversiBitController(players);}
+                    Player<BitboardReversi>[] players = new Player[2];
+                    players[(myTurn + 1) % 2] = new OnlinePlayer<>(response.opponent());
+                    players[myTurn] = new ArtificialPlayer<>(new RandomAI<BitboardReversi>(), user);
+                    gameController = new ReversiBitController(players, false);}
                 default -> new ErrorPopup("Unsupported game type.");
 
             }
@@ -263,7 +263,7 @@ public final class Server {
 		String gameType = extractQuotedValue(response.gameType());
 		final String finalGameType = gameType;
 		var a = new ChallengePopup(challengerName, gameType, (playerInformation) -> {
-			final long challengeId = Long.parseLong(response.challengeId().replaceAll("\\D", ""));
+			final int challengeId = Integer.parseInt(response.challengeId().replaceAll("\\D", ""));
 			new EventFlow().addPostEvent(new NetworkEvents.SendAcceptChallenge(clientId, challengeId)).postEvent();
             isSingleGame.set(true);
 		});

app/src/main/java/org/toop/app/canvas/BitGameCanvas.java

@@ -14,7 +14,7 @@ import org.toop.framework.gameFramework.view.GUIEvents;
 
 import java.util.function.Consumer;
 
-public abstract class BitGameCanvas implements GameCanvas {
+public abstract class BitGameCanvas<T extends TurnBasedGame<T>> implements GameCanvas<T> {
 	protected record Cell(float x, float y, float width, float height) {
 		public boolean isInside(double x, double y) {
 			return x >= this.x && x <= this.x + width &&
@@ -108,6 +108,7 @@ public abstract class BitGameCanvas implements GameCanvas {
         }
     }
 
+
 	private void render() {
 		graphics.setFill(backgroundColor);
 		graphics.fillRect(0, 0, width, height);

app/src/main/java/org/toop/app/canvas/BitLegalMoveDrawer.java

@@ -0,0 +1,5 @@
+package org.toop.app.canvas;
+
+public interface BitLegalMoveDrawer {
+    void showLegalMove(long move, int currentPlayerIndex);
+}

app/src/main/java/org/toop/app/canvas/GameCanvas.java

@@ -1,7 +1,8 @@
 package org.toop.app.canvas;
 
 import javafx.scene.canvas.Canvas;
+import org.toop.framework.gameFramework.model.game.TurnBasedGame;
 
-public interface GameCanvas extends GameDrawer{
+public interface GameCanvas<T extends TurnBasedGame<T>> extends GameDrawer<T>{
     Canvas getCanvas();
 }

app/src/main/java/org/toop/app/canvas/GameDrawer.java

@@ -2,6 +2,6 @@ package org.toop.app.canvas;
 
 import org.toop.framework.gameFramework.model.game.TurnBasedGame;
 
-public interface GameDrawer {
-    void redraw(TurnBasedGame gameCopy);
+public interface GameDrawer<T extends TurnBasedGame<T>> {
+    void redraw(T gameCopy);
 }

app/src/main/java/org/toop/app/canvas/ReversiBitCanvas.java

@@ -2,11 +2,17 @@ package org.toop.app.canvas;
 
 import javafx.scene.paint.Color;
 import org.toop.app.App;
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
+import org.toop.game.games.reversi.BitboardReversi;
 
-public class ReversiBitCanvas extends BitGameCanvas {
-    public ReversiBitCanvas() {
+public class ReversiBitCanvas extends BitGameCanvas<BitboardReversi> implements BitLegalMoveDrawer {
+
+    private final boolean switchColors;
+    private final boolean local;
+
+    public ReversiBitCanvas(boolean switchColors, boolean local) {
         super(Color.GRAY, new Color(0f, 0.4f, 0.2f, 1f), (App.getHeight() / 4) * 3, (App.getHeight() / 4) * 3, 8, 8, 5, true);
+        this.switchColors = switchColors;
+        this.local = local;
         canvas.setOnMouseMoved(event -> {
             double mouseX = event.getX();
             double mouseY = event.getY();
@@ -30,10 +36,42 @@ public class ReversiBitCanvas extends BitGameCanvas {
     }
 
     @Override
-    public void redraw(TurnBasedGame gameCopy) {
+    public void redraw(BitboardReversi gameCopy) {
         clearAll();
         long[] board = gameCopy.getBoard();
-        loopOverBoard(board[0], (i) -> drawDot(Color.WHITE, i));
-        loopOverBoard(board[1], (i) -> drawDot(Color.BLACK, i));
+        if (switchColors && local) {
+            loopOverBoard(board[0], (i) -> drawDot(Color.BLACK, i));
+            loopOverBoard(board[1], (i) -> drawDot(Color.WHITE, i));
+        } else {
+            loopOverBoard(board[0], (i) -> drawDot(Color.WHITE, i));
+            loopOverBoard(board[1], (i) -> drawDot(Color.BLACK, i));
+        }
+    }
+
+    @Override
+    public void showLegalMove(long move, int currentPlayerIndex) {
+        int idx = Long.numberOfTrailingZeros(move);
+        drawLegalMove(idx, currentPlayerIndex);
+    }
+
+    public void drawLegalMove(int cell, int player) {
+        Color innerColor;
+        if (switchColors) {
+            if (player == 0) {
+                innerColor = new Color(0.0f, 0.0f, 0.0f, 0.6f);
+            }
+            else {
+                innerColor = new Color(1.0f, 1.0f, 1.0f, 0.75f);
+            }
+            this.drawInnerDot(innerColor, cell, false);
+        } else {
+        if (player == 1) {
+            innerColor = new Color(0.0f, 0.0f, 0.0f, 0.6f);
+        }
+        else {
+            innerColor = new Color(1.0f, 1.0f, 1.0f, 0.75f);
+        }
+        this.drawInnerDot(innerColor, cell, false);
+        }
     }
 }

app/src/main/java/org/toop/app/canvas/TicTacToeBitCanvas.java

@@ -2,9 +2,12 @@ package org.toop.app.canvas;
 
 import javafx.scene.paint.Color;
 import org.toop.app.App;
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
+import org.toop.game.games.tictactoe.BitboardTicTacToe;
 
-public class TicTacToeBitCanvas extends BitGameCanvas{
+import java.util.Arrays;
+import java.util.function.Consumer;
+
+public class TicTacToeBitCanvas extends BitGameCanvas<BitboardTicTacToe>{
     public TicTacToeBitCanvas() {
         super(
                 Color.GRAY,
@@ -19,7 +22,7 @@ public class TicTacToeBitCanvas extends BitGameCanvas{
     }
 
     @Override
-    public void redraw(TurnBasedGame gameCopy) {
+    public void redraw(BitboardTicTacToe gameCopy) {
         clearAll();
         drawMoves(gameCopy.getBoard());
     }
@@ -27,7 +30,6 @@ public class TicTacToeBitCanvas extends BitGameCanvas{
     private void drawMoves(long[] gameBoard){
         loopOverBoard(gameBoard[0], (i) -> drawX(Color.RED, i));
         loopOverBoard(gameBoard[1], (i) -> drawO(Color.BLUE, i));
-
     }
 
 

app/src/main/java/org/toop/app/gameControllers/GenericGameController.java

@@ -5,6 +5,7 @@ import javafx.geometry.Pos;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 import org.toop.app.canvas.GameCanvas;
+import org.toop.app.canvas.BitLegalMoveDrawer;
 import org.toop.app.widget.WidgetContainer;
 import org.toop.app.widget.view.GameView;
 import org.toop.framework.eventbus.EventFlow;
@@ -15,10 +16,10 @@ import org.toop.framework.gameFramework.model.game.TurnBasedGame;
 import org.toop.framework.gameFramework.model.game.threadBehaviour.ThreadBehaviour;
 import org.toop.framework.gameFramework.model.player.Player;
 import org.toop.framework.gameFramework.view.GUIEvents;
-import org.toop.framework.networking.connection.events.NetworkEvents;
-import org.toop.framework.game.players.LocalPlayer;
+import org.toop.framework.networking.events.NetworkEvents;
+import org.toop.game.players.LocalPlayer;
 
-public class GenericGameController implements GameController {
+public class GenericGameController<T extends TurnBasedGame<T>, C extends GameCanvas<T>> implements GameController {
     protected final EventFlow eventFlow = new EventFlow();
 
     // Logger for logging
@@ -27,18 +28,22 @@ public class GenericGameController implements GameController {
     // Reference to gameView view
     protected final GameView gameView;
 
-    // Reference to game canvas
-    protected final GameCanvas canvas;
+    // Reference to String gametype
+    protected final String gameType;
 
-    protected final TurnBasedGame game;       // Reference to game instance
+    // Reference to game canvas
+    protected final C canvas;
+
+    protected final TurnBasedGame<T> game;       // Reference to game instance
     private final ThreadBehaviour gameThreadBehaviour;
 
     // TODO: Change gameType to automatically happen with either dependency injection or something else.
-    public GenericGameController(GameCanvas canvas, TurnBasedGame game, ThreadBehaviour gameThreadBehaviour, String gameType) {
+    public GenericGameController(C canvas, T game, ThreadBehaviour gameThreadBehaviour, String gameType) {
         logger.info("Creating: " + this.getClass());
 
         this.canvas = canvas;
         this.game = game;
+        this.gameType = gameType;
         this.gameThreadBehaviour = gameThreadBehaviour;
 
         // Tell thread how to send moves
@@ -55,9 +60,7 @@ public class GenericGameController implements GameController {
         // Listen to updates
         eventFlow
                 .listen(GUIEvents.GameEnded.class, this::onGameFinish, false)
-                .listen(GUIEvents.PlayerAttemptedMove.class, event -> {
-                    if (getCurrentPlayer() instanceof LocalPlayer lp){lp.setMove(event.move());}
-                    }, false);
+                .listen(GUIEvents.PlayerAttemptedMove.class, event -> {if (getCurrentPlayer() instanceof LocalPlayer<T> lp){lp.setLastMove(event.move());}}, false);
     }
 
     public void start(){
@@ -72,7 +75,7 @@ public class GenericGameController implements GameController {
         gameThreadBehaviour.stop();
     }
 
-    public Player getCurrentPlayer(){
+    public Player<T> getCurrentPlayer(){
         return game.getPlayer(getCurrentPlayerIndex());
     }
 
@@ -99,7 +102,7 @@ public class GenericGameController implements GameController {
         stop();
     }
 
-    public Player getPlayer(int player){
+    public Player<T> getPlayer(int player){
         if (player < 0 || player >= 2){ // TODO: Make game turn player count
             logger.error("Invalid player index");
             throw new IllegalArgumentException("player out of range");
@@ -138,5 +141,21 @@ public class GenericGameController implements GameController {
     @Override
     public void updateUI() {
         canvas.redraw(game.deepCopy());
+        gameView.updatePlayerInfo(
+               true,
+               getCurrentPlayer().getName(),
+               game.getCurrentTurn() == 0 ? "X" : "O",
+               getPlayer((game.getCurrentTurn() + 1 ) % 2).getName(),
+               this.gameType
+       );
+        // draw legal moves for bit canvasses
+        if (canvas instanceof BitLegalMoveDrawer) {
+            long movesLoop = game.getLegalMoves();
+            while (movesLoop != 0) {
+                long move = 1L << Long.numberOfTrailingZeros(movesLoop);
+                ((BitLegalMoveDrawer) canvas).showLegalMove(move, getCurrentPlayerIndex());
+                movesLoop &= movesLoop - 1;
+            }
+        }
     }
 }

app/src/main/java/org/toop/app/gameControllers/ReversiBitController.java

@@ -1,23 +1,24 @@
 package org.toop.app.gameControllers;
 
+import javafx.scene.paint.Color;
 import org.toop.app.canvas.ReversiBitCanvas;
 import org.toop.framework.gameFramework.model.game.threadBehaviour.ThreadBehaviour;
 import org.toop.framework.gameFramework.model.player.Player;
-import org.toop.framework.game.gameThreads.LocalThreadBehaviour;
-import org.toop.framework.game.gameThreads.OnlineThreadBehaviour;
-import org.toop.framework.game.games.reversi.BitboardReversi;
-import org.toop.framework.game.players.OnlinePlayer;
+import org.toop.game.gameThreads.LocalThreadBehaviour;
+import org.toop.game.gameThreads.OnlineThreadBehaviour;
+import org.toop.game.games.reversi.BitboardReversi;
+import org.toop.game.players.LocalPlayer;
+import org.toop.game.players.OnlinePlayer;
 
-public class ReversiBitController extends GenericGameController {
-    public ReversiBitController(Player[] players) {
-        BitboardReversi game = new BitboardReversi();
-        game.init(players);
-        ThreadBehaviour thread = new LocalThreadBehaviour(game);
-        for (Player player : players) {
-            if (player instanceof OnlinePlayer){
-                thread = new OnlineThreadBehaviour(game);
+public class ReversiBitController extends GenericGameController<BitboardReversi, ReversiBitCanvas> {
+    public ReversiBitController(Player<BitboardReversi>[] players, boolean switchColors) {
+        BitboardReversi game = new BitboardReversi(players);
+        ThreadBehaviour thread = new LocalThreadBehaviour<>(game);
+        for (Player<BitboardReversi> player : players) {
+            if (player instanceof OnlinePlayer<BitboardReversi>) {
+                thread = new OnlineThreadBehaviour<>(game);
             }
         }
-        super(new ReversiBitCanvas(), game, thread, "Reversi");
+        super(new ReversiBitCanvas(switchColors, players[0] instanceof LocalPlayer<BitboardReversi> || players[1] instanceof LocalPlayer<BitboardReversi>), game, thread, "Reversi");
     }
 }

app/src/main/java/org/toop/app/gameControllers/TicTacToeBitController.java

@@ -3,19 +3,19 @@ package org.toop.app.gameControllers;
 import org.toop.app.canvas.TicTacToeBitCanvas;
 import org.toop.framework.gameFramework.model.game.threadBehaviour.ThreadBehaviour;
 import org.toop.framework.gameFramework.model.player.Player;
-import org.toop.framework.game.gameThreads.LocalThreadBehaviour;
-import org.toop.framework.game.gameThreads.OnlineThreadBehaviour;
-import org.toop.framework.game.games.tictactoe.BitboardTicTacToe;
-import org.toop.framework.game.players.OnlinePlayer;
+import org.toop.game.gameThreads.LocalFixedRateThreadBehaviour;
+import org.toop.game.gameThreads.LocalThreadBehaviour;
+import org.toop.game.gameThreads.OnlineThreadBehaviour;
+import org.toop.game.games.tictactoe.BitboardTicTacToe;
+import org.toop.game.players.OnlinePlayer;
 
-public class TicTacToeBitController extends GenericGameController {
-    public TicTacToeBitController(Player[] players) {
-        BitboardTicTacToe game = new BitboardTicTacToe();
-        game.init(players);
-        ThreadBehaviour thread = new LocalThreadBehaviour(game);
-        for (Player player : players) {
-            if (player instanceof OnlinePlayer){
-                thread = new OnlineThreadBehaviour(game);
+public class TicTacToeBitController extends GenericGameController<BitboardTicTacToe, TicTacToeBitCanvas > {
+    public TicTacToeBitController(Player<BitboardTicTacToe>[] players) {
+        BitboardTicTacToe game = new BitboardTicTacToe(players);
+        ThreadBehaviour thread = new LocalThreadBehaviour<>(game);
+        for (Player<BitboardTicTacToe> player : players) {
+            if (player instanceof OnlinePlayer<BitboardTicTacToe>){
+                thread = new OnlineThreadBehaviour<>(game);
             }
         }
         super(new TicTacToeBitCanvas(), game, thread , "TicTacToe");

app/src/main/java/org/toop/app/widget/view/GameView.java

@@ -16,14 +16,15 @@ import org.toop.app.widget.tutorial.Connect4TutorialWidget;
 import org.toop.app.widget.tutorial.ReversiTutorialWidget;
 import org.toop.app.widget.tutorial.TicTacToeTutorialWidget;
 import org.toop.local.AppContext;
+import org.toop.local.AppSettings;
 
 public final class GameView extends ViewWidget {
 	private final Text playerHeader;
 	private final Text turnHeader;
     private final Text player1Header;
     private final Text player2Header;
-    private Circle player1Icon;
-    private Circle player2Icon;
+    private final Circle player1Icon;
+    private final Circle player2Icon;
 	private final Button forfeitButton;
 	private final Button exitButton;
     private final TextField chatInput;
@@ -94,7 +95,7 @@ public final class GameView extends ViewWidget {
 		}
 	}
 
-	public void nextPlayer(boolean isMe, String currentPlayer, String currentMove, String nextPlayer, char GameType) {
+	public void updatePlayerInfo(boolean isMe, String currentPlayer, String currentMove, String nextPlayer, String GameType) {
 		Platform.runLater(() -> {
             if (!(hasSet)) {
                 playerHeader.setText(currentPlayer + " vs. " + nextPlayer);
@@ -112,8 +113,8 @@ public final class GameView extends ViewWidget {
 		new GameOverPopup(iWon, winner).show(Pos.CENTER);
 	}
 
-    private void setPlayerHeaders(boolean isMe, String currentPlayer, String nextPlayer, char GameType) {
-        if (GameType == 'T') {
+    private void setPlayerHeaders(boolean isMe, String currentPlayer, String nextPlayer, String GameType) {
+        if (GameType.equals("TicTacToe")) {
             if (isMe) {
                 player1Header.setText("X: " + currentPlayer);
                 player2Header.setText("O: " + nextPlayer);
@@ -124,15 +125,16 @@ public final class GameView extends ViewWidget {
             }
             setPlayerInfoTTT();
         }
-        else if (GameType == 'R') {
-            if (isMe) {
-                player1Header.setText(currentPlayer);
-                player2Header.setText(nextPlayer);
-            }
-            else {
+        else if (GameType.equals("Reversi")) {
+            boolean swap = AppSettings.getSettings().getSwitchReversi();
+            if (!swap) {
                 player1Header.setText(nextPlayer);
                 player2Header.setText(currentPlayer);
             }
+            else {
+                player1Header.setText(currentPlayer);
+                player2Header.setText(nextPlayer);
+            }
             setPlayerInfoReversi();
         }
     }
@@ -166,8 +168,8 @@ public final class GameView extends ViewWidget {
         var playerInfo = Primitive.vbox(
                 playerHeader,
                 Primitive.separator(),
-                player1box,
-                player2box
+                player2box,
+                player1box
         );
 
         player1Icon.setRadius(player1Header.fontProperty().map(Font::getSize).getValue());

app/src/main/java/org/toop/app/widget/view/LocalMultiplayerView.java

@@ -6,17 +6,17 @@ import org.toop.app.gameControllers.ReversiBitController;
 import org.toop.app.gameControllers.TicTacToeBitController;
 import org.toop.framework.gameFramework.controller.GameController;
 import org.toop.framework.gameFramework.model.player.Player;
-import org.toop.framework.game.games.reversi.BitboardReversi;
-import org.toop.framework.game.games.tictactoe.BitboardTicTacToe;
-import org.toop.framework.game.players.ArtificialPlayer;
-import org.toop.framework.game.players.LocalPlayer;
+import org.toop.game.games.reversi.BitboardReversi;
+import org.toop.game.games.tictactoe.BitboardTicTacToe;
+import org.toop.game.players.ArtificialPlayer;
+import org.toop.game.players.LocalPlayer;
 import org.toop.app.widget.Primitive;
 import org.toop.app.widget.complex.PlayerInfoWidget;
 import org.toop.app.widget.complex.ViewWidget;
 import org.toop.app.widget.popup.ErrorPopup;
 import org.toop.app.widget.tutorial.*;
-import org.toop.framework.game.players.MiniMaxAI;
-import org.toop.framework.game.players.RandomAI;
+import org.toop.game.players.ai.MiniMaxAI;
+import org.toop.game.players.ai.RandomAI;
 import org.toop.local.AppContext;
 
 import javafx.geometry.Pos;
@@ -52,14 +52,14 @@ public class LocalMultiplayerView extends ViewWidget {
 			switch (information.type) {
                 case TICTACTOE:
                     if (information.players[0].isHuman) {
-                        players[0] = new LocalPlayer(information.players[0].name);
+                        players[0] = new LocalPlayer<>(information.players[0].name);
                     } else {
-                        players[0] = new ArtificialPlayer(new RandomAI(), "Random AI");
+                        players[0] = new ArtificialPlayer<>(new RandomAI<BitboardTicTacToe>(), "Random AI");
                     }
                     if (information.players[1].isHuman) {
-                        players[1] = new LocalPlayer(information.players[1].name);
+                        players[1] = new LocalPlayer<>(information.players[1].name);
                     } else {
-                        players[1] = new ArtificialPlayer(new MiniMaxAI(9), "MiniMax AI");
+                        players[1] = new ArtificialPlayer<>(new MiniMaxAI<BitboardTicTacToe>(9), "MiniMax AI");
                     }
                     if (AppSettings.getSettings().getTutorialFlag() && AppSettings.getSettings().getFirstTTT()) {
                         new ShowEnableTutorialWidget(
@@ -79,30 +79,31 @@ public class LocalMultiplayerView extends ViewWidget {
                     }
                     break;
                 case REVERSI:
+                    boolean swap = AppSettings.getSettings().getSwitchReversi();
                     if (information.players[0].isHuman) {
-                        players[0] = new LocalPlayer(information.players[0].name);
+                        players[0] = new LocalPlayer<>(information.players[0].name);
                     } else {
-                        players[0] = new ArtificialPlayer(new RandomAI(), "Random AI");
+                        players[0] = new ArtificialPlayer<>(new RandomAI<BitboardReversi>(), "Random AI");
                     }
                     if (information.players[1].isHuman) {
-                        players[1] = new LocalPlayer(information.players[1].name);
+                        players[1] = new LocalPlayer<>(information.players[1].name);
                     } else {
-                        players[1] = new ArtificialPlayer(new MiniMaxAI(6), "MiniMax");
+                        players[1] = new ArtificialPlayer<>(new MiniMaxAI<BitboardReversi>(6), "MiniMax");
                     }
                     if (AppSettings.getSettings().getTutorialFlag() && AppSettings.getSettings().getFirstReversi()) {
                         new ShowEnableTutorialWidget(
                                 () -> new ReversiTutorialWidget(() -> {
-                                    gameController = new ReversiBitController(players);
+                                    gameController = new ReversiBitController(players, swap);
                                     gameController.start();
                                 }),
                                 () -> Platform.runLater(() -> {
-                                    gameController = new ReversiBitController(players);
+                                    gameController = new ReversiBitController(players, swap);
                                     gameController.start();
                                 }),
                                 () -> AppSettings.getSettings().setFirstReversi(false)
                         );
                     } else {
-                        gameController = new ReversiBitController(players);
+                        gameController = new ReversiBitController(players, swap);
                         gameController.start();
                     }
                     break;

app/src/main/java/org/toop/app/widget/view/OnlineView.java

@@ -6,13 +6,6 @@ import org.toop.app.widget.complex.LabeledInputWidget;
 import org.toop.app.widget.complex.ViewWidget;
 
 import javafx.geometry.Pos;
-import org.toop.framework.game.games.reversi.BitboardReversi;
-import org.toop.framework.game.games.tictactoe.BitboardTicTacToe;
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-import org.toop.framework.networking.server.MasterServer;
-
-import java.time.Duration;
-import java.util.concurrent.ConcurrentHashMap;
 
 public class OnlineView extends ViewWidget {
 	public OnlineView() {
@@ -30,28 +23,6 @@ public class OnlineView extends ViewWidget {
 			);
 		});
 
-		var localHostButton = Primitive.button("host!", () -> {
-			var games = new ConcurrentHashMap<String, Class<? extends TurnBasedGame>>();
-			games.put("tic-tac-toe", BitboardTicTacToe.class);
-			games.put("reversi", BitboardReversi.class);
-
-			var a = new MasterServer(6666, games, Duration.ofSeconds(10));
-
-			new Thread(() -> {
-				try {
-					a.start();
-				} catch (InterruptedException e) {
-					throw new RuntimeException(e);
-				}
-			}).start();
-
-			new Server(
-					"127.0.0.1",
-					"6666",
-					"host"
-			);
-		}, false);
-
 		add(Pos.CENTER, Primitive.vbox(
 			serverInformationHeader,
 			Primitive.separator(),
@@ -61,9 +32,7 @@ public class OnlineView extends ViewWidget {
 			playerNameInput.getNode(),
 			Primitive.separator(),
 
-			connectButton,
-			Primitive.separator(),
-			localHostButton
+			connectButton
 		));
 	}
 }

app/src/main/java/org/toop/app/widget/view/OptionsView.java

@@ -149,13 +149,22 @@ public class OptionsView extends ViewWidget {
 			"small", "medium", "large"
 		);
 
+        var swapReversiColors = new ToggleWidget(
+                "reversinotswapped", "reversiswapped",
+                AppSettings.getSettings().getSwitchReversi(),
+                swap -> {
+                    AppSettings.getSettings().setSwitchReversi(swap);
+                }
+        );
 
 		return Primitive.vbox(
 			Primitive.header("style"),
 			Primitive.separator(),
 
 			themeWidget.getNode(),
-			layoutWidget.getNode()
+			layoutWidget.getNode(),
+            Primitive.text("reversiswap"),
+            swapReversiColors.getNode()
 		);
 	}
 }

app/src/main/java/org/toop/local/AppSettings.java

@@ -5,18 +5,16 @@ import org.toop.framework.audio.VolumeControl;
 import org.toop.framework.audio.events.AudioEvents;
 import org.toop.framework.eventbus.EventFlow;
 import org.toop.framework.resource.ResourceManager;
-import org.toop.framework.resource.ResourceMeta;
 import org.toop.framework.resource.resources.SettingsAsset;
 import org.toop.framework.settings.Settings;
 
-import java.io.File;
 import java.util.Locale;
 
 public class AppSettings {
 	private static SettingsAsset settingsAsset;
 
 	public static void applySettings() {
-		settingsAsset = getPath();
+		settingsAsset = getSettingsAsset();
 		if (!settingsAsset.isLoaded()) {
 			settingsAsset.load();
 		}
@@ -45,26 +43,9 @@ public class AppSettings {
 				.postEvent();
 	}
 
-	public static SettingsAsset getPath() {
+	public static SettingsAsset getSettingsAsset() {
 		if (settingsAsset == null) {
-			String os = System.getProperty("os.name").toLowerCase();
-			String basePath;
-
-			if (os.contains("win")) {
-				basePath = System.getenv("APPDATA");
-				if (basePath == null) {
-					basePath = System.getProperty("user.home");
-				}
-			} else if (os.contains("mac")) {
-				basePath = System.getProperty("user.home") + "/Library/Application Support";
-			} else {
-				basePath = System.getProperty("user.home") + "/.config";
-			}
-
-			File settingsFile =
-				new File(basePath + File.separator + "ISY1" + File.separator + "settings.json");
-//            this.settingsAsset = new SettingsAsset(settingsFile);
-			ResourceManager.addAsset(new ResourceMeta<>("settings.json", new SettingsAsset(settingsFile)));
+			settingsAsset = SettingsAsset.getPath();
 		}
 		return ResourceManager.get("settings.json");
 	}
@@ -110,5 +91,6 @@ public class AppSettings {
         settingsAsset.setMusicVolume(15);
         settingsAsset.setTutorialFlag(true);
         settingsAsset.setLayoutSize("medium");
+        settingsAsset.setSwitchReversi(false);
     }
 }

app/src/main/resources/localization/localization_ar.properties

@@ -89,6 +89,9 @@ turnof=\u062F\u0648\u0631\u0647
 zwartepiet=\u0633\u0647\u0644: Zwarte Piet
 sinterklaas=\u0645\u062a\u0648\u0633\u0637: Sint R. Klaas
 santa=\u0635\u0639\u0628: Santa
+reversiswapped=\u0645\u0628\u062F\u0651\u064E\u0644
+reversinotswapped=\u063A\u064A\u0631\u0020\u0645\u0628\u062F\u0651\u064E\u0644
+reversiswap=\u0647\u0644\u0020\u062A\u0631\u064A\u062F\u0020\u062A\u0628\u062F\u064A\u0644\u0020\u0623\u0644\u0648\u0627\u0646\u0020\u0627\u0644\u0631\u064A\u0641\u064E\u0631\u0633\u064A\u061F\u0020\u0645\u062D\u0644\u064A\u0020\u0641\u0642\u0637\u002E
 
 
 arabic=\u0627\u0644\u0639\u0631\u0628\u064a\u0629

app/src/main/resources/localization/localization_de.properties

@@ -91,6 +91,9 @@ turnof=ist dran
 zwartepiet=Leicht: Zwarte Piet
 sinterklaas=Mittel: Sint R. Klaas
 santa=Schwer: Santa
+reversiswapped=Getauscht
+reversinotswapped=Nicht getauscht
+reversiswap=M\u00F6chten Sie die Reversi-Farben tauschen? Nur lokal.
 
 arabic=\u0627\u0644\u0639\u0631\u0628\u064a\u0629 (Arabisch)
 chinese=\u4e2d\u6587 (Chinesisch)

app/src/main/resources/localization/localization_en.properties

@@ -93,6 +93,9 @@ turnof='s turn
 zwartepiet=Easy: Zwarte Piet
 sinterklaas=Medium: Sint R. Klaas
 santa=Hard:Santa
+reversiswapped=Swapped
+reversinotswapped=Not Swapped
+reversiswap=Do you want to swap the Reversi colors? Local only.
 
 arabic=\u0627\u0644\u0639\u0631\u0628\u064a\u0629 (Arabic)
 chinese=\u4e2d\u6587 (Chinese)

app/src/main/resources/localization/localization_es.properties

@@ -90,6 +90,9 @@ turnof=le toca
 zwartepiet=F\u00e1cil: Zwarte Piet
 sinterklaas=Medio: Sint R. Klaas
 santa=Dif\u00edcil: Santa
+reversiswapped=Cambiado
+reversinotswapped=No cambiado
+reversiswap=\u00BFDeseas intercambiar los colores de Reversi? Solo local.
 
 arabic=\u0627\u0644\u0639\u0631\u0628\u064a\u0629 (Ar\u00e1bigo)
 chinese=\u4e2d\u6587 (Chino)

app/src/main/resources/localization/localization_fr.properties

@@ -90,6 +90,9 @@ turnof=\u00E0 son tour
 zwartepiet=Facile: Zwarte Piet
 sinterklaas=Moyen : Sint R. Klaas
 santa=Difficile: Santa
+reversiswapped=\u00C9chang\u00E9
+reversinotswapped=Non \u00E9chang\u00E9
+reversiswap=Voulez-vous \u00E9changer les couleurs de Reversi ? Local uniquement.
 
 arabic=\u0627\u0644\u0639\u0631\u0628\u064a\u0629 (Arabe)
 chinese=\u4e2d\u6587 (Chinois)

app/src/main/resources/localization/localization_hi.properties

@@ -90,6 +90,9 @@ turnof=\u0915\u0940 \u092C\u093E\u0930\u0940
 zwartepiet=\u0905\u0938\u093e\u0928: Zwarte Piet
 sinterklaas=\u092e\u0927\u094d\u092f\u092e: Sint R. Klaas
 santa=\u0915\u0924\u093f\u0928: Santa
+reversiswapped=\u092C\u0926\u0932\u093E\u0020\u0939\u0941\u0906
+reversinotswapped=\u0928\u0939\u0940\u0902\u0020\u092C\u0926\u0932\u093E\u0020\u0939\u0941\u0906
+reversiswap=\u0915\u094D\u092F\u093E\u0020\u0906\u092A\u0020\u0930\u093F\u0935\u0930\u094D\u0938\u0940\u0020\u0915\u0947\u0020\u0930\u0902\u0917\u0020\u092C\u0926\u0932\u0928\u093E\u0020\u091A\u093E\u0939\u0924\u0947\u0020\u0939\u0948\u0902?\u0020\u0915\u0947\u0935\u0932\u0020\u0938\u094D\u0925\u093E\u0928\u0940\u092F\u002E
 
 arabic=\u0627\u0644\u0639\u0631\u0628\u064a\u0629 (\u0905\u0930\u092c\u0940)
 chinese=\u4e2d\u6587 (\u091a\u0940\u0928\u0940)

app/src/main/resources/localization/localization_it.properties

@@ -89,6 +89,9 @@ turnof=\u00E8 il suo turno
 zwartepiet=Facile: Zwarte Piet
 sinterklaas=Medio: Sint R. Klaas
 santa=Difficile: Santa
+reversiswapped=Scambiato
+reversinotswapped=Non scambiato
+reversiswap=Vuoi scambiare i colori di Reversi? Solo locale.
 
 arabic=\u0627\u0644\u0639\u0631\u0628\u064a\u0629 (Arabo)
 chinese=\u4e2d\u6587 (Cinese)

app/src/main/resources/localization/localization_ja.properties

@@ -89,6 +89,10 @@ turnof=\u306E\u756A
 zwartepiet=\u7c21\u5358: Zwarte Piet
 sinterklaas=\u4e2d\u7d1a: Sint R. Klaas
 santa=\u96e3\u3057\u3044: Santa
+reversiswapped=\u5165\u308C\u66FF\u3048\u6E08\u307F
+reversinotswapped=\u672A\u5165\u308C\u66FF\u3048
+reversiswap=\u30EA\u30D0\u30FC\u30B7\u306E\u8272\u3092\u5165\u308C\u66FF\u3048\u307E\u3059\u304B?\u0020\u30ED\u30FC\u30AB\u30EB\u306E\u307F\u3067\u3059\u002E
+
 
 arabic=\u0627\u0644\u0639\u0631\u0628\u064a\u0629 (\u30a2\u30e9\u30d3\u30a2\u8a9e)
 chinese=\u4e2d\u6587 (\u4e2d\u6587)

app/src/main/resources/localization/localization_ko.properties

@@ -89,6 +89,9 @@ turnof=\uC758 \uCC28\uB840
 zwartepiet=\uc218\uc601: Zwarte Piet
 sinterklaas=\ubcf4\ud1b5: Sint R. Klaas
 santa=\uc5d0\uc18c: Santa
+reversiswapped=\uAD50\uCCB4\uB428
+reversinotswapped=\uAD50\uCCB4\uB418\uC9C0 \uC54A\uC74C
+reversiswap=\uB9AC\uBC84\uC2DC \uC0C9\uC0C1\uC744 \uBC14\uAFB8\uC2DC\uACA0\uC2B5\uB2C8\uAE4C?\u0020\uB85C\uCEEC \uC804\uC6A9\u0020\uC785\uB2C8\uB2E4\u002E
 
 arabic=\u0627\u0644\u0639\u0631\u0628\u064a\u0629 (\u0639\u0631\u0628\u064a\u0629)
 chinese=\u4e2d\u6587 (\u4e2d\u6587)

app/src/main/resources/localization/localization_nl.properties

@@ -89,6 +89,9 @@ turnof=is aan de beurt
 zwartepiet=Makkelijk: Zwarte Piet
 sinterklaas=Gemiddeld: Sint R. Klaas
 santa=Moeilijk: Santa
+reversiswapped=Gewisseld
+reversinotswapped=Niet gewisseld
+reversiswap=Wil je de Reversi-kleuren wisselen? Alleen lokaal.
 
 arabic=\u0627\u0644\u0639\u0631\u0628\u064a\u0629 (Arabisch)
 chinese=\u4e2d\u6587 (Chinees)

app/src/main/resources/localization/localization_ru.properties

@@ -89,6 +89,9 @@ turnof=\u0445\u043E\u0434\u0438\u0442
 zwartepiet=\u041b\u0435\u0433\u043a\u043e: Zwarte Piet
 sinterklaas=\u0421\u0440\u0435\u0434\u043d\u0438\u0439: Sint R. Klaas
 santa=\u0421\u043b\u043e\u0436\u043d\u043e: Santa
+reversiswapped=\u041F\u043E\u043C\u0435\u043D\u044F\u043D\u043E
+reversinotswapped=\u041D\u0435 \u043F\u043E\u043C\u0435\u043D\u044F\u043D\u043E
+reversiswap=\u0425\u043E\u0442\u0438\u0442\u0435 \u043F\u043E\u043C\u0435\u043D\u044F\u0442\u044C \u0446\u0432\u0435\u0442\u0430 \u0420\u0435\u0432\u0435\u0440\u0441\u0438?\u0020\u0422\u043E\u043B\u044C\u043A\u043E \u043B\u043E\u043A\u0430\u043B\u044C\u043D\u043E\u002E
 
 arabic=\u0627\u0644\u0639\u0631\u0628\u064a\u0629 (\u0410\u0440\u0430\u0431\u0441\u043a\u0438\u0439)
 chinese=\u4e2d\u6587 (\u041a\u0438\u0442\u0430\u0439\u0441\u043a\u0438\u0439)

app/src/main/resources/localization/localization_zh.properties

@@ -89,6 +89,9 @@ turnof=\u7684\u56DE\u5408
 zwartepiet=\u7b80\u5355: Zwarte Piet
 sinterklaas=\u4e2d\u7b49: Sint R. Klaas
 santa=\u56f0\u96be: Santa
+reversiswapped=\u5DF2\u4EA4\u6362
+reversinotswapped=\u672A\u4EA4\u6362
+reversiswap=\u662F\u5426\u8981\u4EA4\u6362\u9ED1\u767D\u68CB\u7684\u989C\u8272\uFF1F\u4EC5\u9650\u672C\u5730\u002E
 
 arabic=\u0627\u0644\u0639\u0631\u0628\u064a\u0629 (\u963f\u62c9\u4f2f\u8bed)
 chinese=\u4e2d\u6587

framework/pom.xml

@@ -147,6 +147,7 @@
             <version>2.42.0</version>
         </dependency>
 	</dependencies>
+
 	<build>
 		<plugins>
 			<plugin>

framework/src/main/java/org/toop/framework/game/gameThreads/ServerThreadBehaviour.java

@@ -1,72 +0,0 @@
-package org.toop.framework.game.gameThreads;
-
-import org.toop.framework.eventbus.EventFlow;
-import org.toop.framework.gameFramework.GameState;
-import org.toop.framework.gameFramework.model.game.PlayResult;
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-import org.toop.framework.gameFramework.model.game.threadBehaviour.AbstractThreadBehaviour;
-import org.toop.framework.gameFramework.model.player.Player;
-import org.toop.framework.gameFramework.view.GUIEvents;
-import org.toop.framework.utils.ImmutablePair;
-
-import java.util.function.Consumer;
-
-import static org.toop.framework.gameFramework.GameState.TURN_SKIPPED;
-import static org.toop.framework.gameFramework.GameState.WIN;
-
-public class ServerThreadBehaviour extends AbstractThreadBehaviour implements Runnable {
-    private Consumer<ImmutablePair<String, Integer>> onPlayerMove;
-    /**
-     * Creates a new base behaviour for the specified game.
-     *
-     * @param game the turn-based game to control
-     */
-    public ServerThreadBehaviour(TurnBasedGame game, Consumer<ImmutablePair<String, Integer>> onPlayerMove) {
-        super(game);
-    }
-
-    private void notifyPlayerMove(ImmutablePair<String, Integer> pair) {
-        onPlayerMove.accept(pair);
-    }
-
-    /** Starts the game loop in a new thread. */
-    @Override
-    public void start() {
-        if (isRunning.compareAndSet(false, true)) {
-            new Thread(this).start();
-        }
-    }
-
-    /** Stops the game loop after the current iteration. */
-    @Override
-    public void stop() {
-        isRunning.set(false);
-    }
-
-    @Override
-    public void run() {
-        while (isRunning.get()) {
-            Player currentPlayer = game.getPlayer(game.getCurrentTurn());
-            long move = currentPlayer.getMove(game.deepCopy());
-            PlayResult result = game.play(move);
-
-            GameState state = result.state();
-
-            if (state != TURN_SKIPPED){
-                notifyPlayerMove(new ImmutablePair<>(currentPlayer.getName(), Long.numberOfTrailingZeros(move)));
-            }
-
-            switch (state) {
-                case WIN, DRAW -> {
-                    isRunning.set(false);
-                }
-                case NORMAL, TURN_SKIPPED -> { /* continue normally */ }
-                default -> {
-                    logger.error("Unexpected state {}", state);
-                    isRunning.set(false);
-                    throw new RuntimeException("Unknown state: " + state);
-                }
-            }
-        }
-    }
-}

framework/src/main/java/org/toop/framework/game/players/ArtificialPlayer.java

@@ -1,55 +0,0 @@
-package org.toop.framework.game.players;
-
-import org.toop.framework.gameFramework.model.player.*;
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-
-/**
- * Represents a player controlled by an AI in a game.
- * <p>
- * This player uses an {@link AbstractAI} instance to determine its moves. The generic
- * parameter {@code T} specifies the type of {@link GameR} the AI can handle.
- * </p>
- *
- * @param <T> the specific type of game this AI player can play
- */
-public class ArtificialPlayer extends AbstractPlayer {
-
-    /** The AI instance used to calculate moves. */
-    private final AI ai;
-
-    /**
-     * Constructs a new ArtificialPlayer using the specified AI.
-     *
-     * @param ai the AI instance that determines moves for this player
-     */
-    public ArtificialPlayer(AI ai, String name) {
-        super(name);
-        this.ai = ai;
-    }
-
-    public ArtificialPlayer(ArtificialPlayer other) {
-        super(other);
-        this.ai = other.ai.deepCopy();
-    }
-
-    /**
-     * Determines the next move for this player using its AI.
-     * <p>
-     * This method overrides {@link AbstractPlayer#getMove(GameR)}. Because the AI is
-     * typed to {@code T}, a runtime cast is required. It is the caller's
-     * responsibility to ensure that {@code gameCopy} is of type {@code T}.
-     * </p>
-     *
-     * @param gameCopy a copy of the current game state
-     * @return the integer representing the chosen move
-     * @throws ClassCastException if {@code gameCopy} is not of type {@code T}
-     */
-    public long getMove(TurnBasedGame gameCopy) {
-        return ai.getMove(gameCopy);
-    }
-
-    @Override
-    public ArtificialPlayer deepCopy() {
-        return new ArtificialPlayer(this);
-    }
-}

framework/src/main/java/org/toop/framework/game/players/LocalPlayer.java

@@ -1,85 +0,0 @@
-package org.toop.framework.game.players;
-
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-import org.toop.framework.gameFramework.model.player.AbstractPlayer;
-
-import java.util.concurrent.CompletableFuture;
-import java.util.concurrent.ExecutionException;
-
-public class LocalPlayer extends AbstractPlayer {
-    // Future can be used with event system, IF unsubscribeAfterSuccess works...
-    // private CompletableFuture<Integer> LastMove = new CompletableFuture<>();
-
-    private CompletableFuture<Long> LastMove;
-
-    public LocalPlayer(String name) {
-        super(name);
-    }
-
-    public LocalPlayer(LocalPlayer other) {
-        super(other);
-    }
-
-    @Override
-    public long getMove(TurnBasedGame gameCopy) {
-        return getValidMove(gameCopy);
-    }
-
-    public void setMove(long move) {
-        LastMove.complete(move);
-    }
-
-    // TODO: helper function, would like to replace to get rid of this method
-    public static boolean contains(int[] array, int value){
-        for (int i : array) if (i == value) return true;
-        return false;
-    }
-
-    private long getMove2(TurnBasedGame gameCopy) {
-        LastMove = new CompletableFuture<>();
-        long move = 0;
-        try {
-            move = LastMove.get();
-            System.out.println(Long.toBinaryString(move));
-        } catch (InterruptedException | ExecutionException e) {
-            // TODO: Add proper logging.
-            e.printStackTrace();
-        }
-        return move;
-    }
-
-    protected long getValidMove(TurnBasedGame gameCopy){
-        // Get this player's valid moves
-        long validMoves = gameCopy.getLegalMoves();
-        // Make sure provided move is valid
-        // TODO: Limit amount of retries?
-        // TODO: Stop copying game so many times
-        long move = getMove2(gameCopy.deepCopy());
-        while ((validMoves & move) == 0) {
-            System.out.println("Not a valid move, try again");
-            move = getMove2(gameCopy.deepCopy());
-        }
-        return move;
-    }
-
-    @Override
-    public LocalPlayer deepCopy() {
-        return new LocalPlayer(this.getName());
-    }
-
-    /*public void register() {
-        // Listening to PlayerAttemptedMove
-        new EventFlow().listen(GUIEvents.PlayerAttemptedMove.class, event -> {
-            if (!LastMove.isDone()) {
-                LastMove.complete(event.move()); // complete the future
-            }
-        }, true); // auto-unsubscribe
-    }
-
-    // This blocks until the next move arrives
-    public int take() throws ExecutionException, InterruptedException {
-        int move = LastMove.get(); // blocking
-        LastMove = new CompletableFuture<>(); // reset for next move
-        return move;
-    }*/
-}

framework/src/main/java/org/toop/framework/game/players/OnlinePlayer.java

@@ -1,36 +0,0 @@
-package org.toop.framework.game.players;
-
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-import org.toop.framework.gameFramework.model.player.AbstractPlayer;
-import org.toop.framework.gameFramework.model.player.Player;
-
-/**
- * Represents a player controlled remotely or over a network.
- * <p>
- * This class extends {@link AbstractPlayer} and can be used to implement game logic
- * where moves are provided by an external source (e.g., another user or a server).
- * Currently, this class is a placeholder and does not implement move logic.
- * </p>
- */
-public class OnlinePlayer extends AbstractPlayer {
-
-    /**
-     * Constructs a new OnlinePlayer.
-     * <p>
-     * Currently, no additional initialization is performed. Subclasses or
-     * future implementations should provide mechanisms to receive moves from
-     * an external source.
-     */
-    public OnlinePlayer(String name) {
-        super(name);
-    }
-
-    public OnlinePlayer(OnlinePlayer other) {
-        super(other);
-    }
-
-    @Override
-    public Player deepCopy() {
-        return new OnlinePlayer(this);
-    }
-}

framework/src/main/java/org/toop/framework/game/players/ServerPlayer.java

@@ -1,46 +0,0 @@
-package org.toop.framework.game.players;
-
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-import org.toop.framework.gameFramework.model.player.AbstractPlayer;
-import org.toop.framework.gameFramework.model.player.Player;
-import org.toop.framework.networking.server.User;
-
-import java.util.concurrent.CompletableFuture;
-import java.util.concurrent.ExecutionException;
-
-public class ServerPlayer extends AbstractPlayer {
-    private User user;
-    private CompletableFuture<Long> lastMove;
-
-    public ServerPlayer(User user) {
-        super(user.name());
-        this.user = user;
-    }
-
-    public void setMove(long move) {
-        lastMove.complete(move);
-    }
-
-    @Override
-    public Player deepCopy() {
-        return null;
-    }
-
-    @Override
-    public long getMove(TurnBasedGame game) {
-        lastMove = new CompletableFuture<>();
-        System.out.println("Sending yourturn");
-        user.sendMessage("SVR GAME YOURTURN {TURNMESSAGE: \"<bericht voor deze beurt>\"}\n");
-        try {
-            return lastMove.get();
-        } catch (InterruptedException | ExecutionException e) {
-            e.printStackTrace();
-            return 0;
-        }
-    }
-
-    @Override
-    public String getName() {
-        return "";
-    }
-}

framework/src/main/java/org/toop/framework/gameFramework/controller/GameController.java

@@ -1,7 +1,8 @@
 package org.toop.framework.gameFramework.controller;
 
+import org.toop.framework.gameFramework.model.game.SupportsOnlinePlay;
 import org.toop.framework.gameFramework.model.game.threadBehaviour.Controllable;
-import org.toop.framework.networking.connection.events.NetworkEvents;
+import org.toop.framework.networking.events.NetworkEvents;
 
 public interface GameController extends Controllable, UpdatesGameUI {
     /** Called when it is this player's turn to make a move. */

framework/src/main/java/org/toop/framework/gameFramework/model/game/PlayerProvider.java

@@ -2,6 +2,6 @@ package org.toop.framework.gameFramework.model.game;
 
 import org.toop.framework.gameFramework.model.player.Player;
 
-public interface PlayerProvider {
-    Player getPlayer(int index);
+public interface PlayerProvider<T extends TurnBasedGame<T>> {
+    Player<T> getPlayer(int index);
 }

framework/src/main/java/org/toop/framework/gameFramework/model/game/SupportsOnlinePlay.java

@@ -1,5 +1,7 @@
 package org.toop.framework.gameFramework.model.game;
 
+import org.toop.framework.networking.events.NetworkEvents;
+
 /**
  * Interface for games that support online multiplayer play.
  * <p>

framework/src/main/java/org/toop/framework/gameFramework/model/game/TurnBasedGame.java

@@ -1,9 +1,6 @@
 package org.toop.framework.gameFramework.model.game;
 
-import org.toop.framework.gameFramework.model.player.Player;
-
-public interface TurnBasedGame extends Playable, PlayerProvider, BoardProvider, DeepCopyable<TurnBasedGame> {
-    void init(Player[] players);
+public interface TurnBasedGame<T extends TurnBasedGame<T>> extends Playable, DeepCopyable<T>, PlayerProvider<T>, BoardProvider {
     int getCurrentTurn();
     int getPlayerCount();
     int getWinner();

framework/src/main/java/org/toop/framework/gameFramework/model/game/threadBehaviour/AbstractThreadBehaviour.java

@@ -16,14 +16,14 @@ import java.util.function.Consumer;
  * a running flag, a game reference, and a logger.
  * Subclasses implement the actual game-loop logic.
  */
-public abstract class AbstractThreadBehaviour implements ThreadBehaviour {
+public abstract class AbstractThreadBehaviour<T extends TurnBasedGame<T>> implements ThreadBehaviour {
     private LongPairConsumer onSendMove;
     private Runnable onUpdateUI;
     /** Indicates whether the game loop or event processing is active. */
     protected final AtomicBoolean isRunning = new AtomicBoolean();
 
     /** The game instance controlled by this behaviour. */
-    protected final TurnBasedGame game;
+    protected final T game;
 
     /** Logger for the subclass to report errors or debug info. */
     protected final Logger logger = LogManager.getLogger(this.getClass());
@@ -33,7 +33,7 @@ public abstract class AbstractThreadBehaviour implements ThreadBehaviour {
      *
      * @param game the turn-based game to control
      */
-    public AbstractThreadBehaviour(TurnBasedGame game) {
+    public AbstractThreadBehaviour(T game) {
         this.game = game;
     }
 

framework/src/main/java/org/toop/framework/gameFramework/model/player/AI.java

@@ -3,5 +3,5 @@ package org.toop.framework.gameFramework.model.player;
 import org.toop.framework.gameFramework.model.game.DeepCopyable;
 import org.toop.framework.gameFramework.model.game.TurnBasedGame;
 
-public interface AI extends MoveProvider, DeepCopyable<AI> {
+public interface AI<T extends TurnBasedGame<T>> extends MoveProvider<T>, DeepCopyable<AI<T>> {
 }

framework/src/main/java/org/toop/framework/gameFramework/model/player/AbstractAI.java

@@ -12,6 +12,6 @@ import org.toop.framework.gameFramework.model.game.TurnBasedGame;
  *
  * @param <T> the specific type of game this AI can play, extending {@link GameR}
  */
-public abstract class AbstractAI implements AI {
+public abstract class AbstractAI<T extends TurnBasedGame<T>> implements AI<T> {
     // Concrete AI implementations should override findBestMove(T game, int depth)
 }

framework/src/main/java/org/toop/framework/gameFramework/model/player/AbstractPlayer.java

@@ -5,46 +5,66 @@ import org.apache.logging.log4j.Logger;
 import org.toop.framework.gameFramework.model.game.TurnBasedGame;
 
 /**
- * Abstract class representing a player in a game.
- * <p>
- * Players are entities that can make moves based on the current state of a game.
- * player types, such as human players or AI players.
- * </p>
- * <p>
- * Subclasses should override the {@link #getMove(GameR)} method to provide
- * specific move logic.
- * </p>
+ * Base class for players in a turn-based game.
+ *
+ * @param <T> the game type
  */
-public abstract class AbstractPlayer implements Player {
-    private final Logger logger = LogManager.getLogger(this.getClass());
+public abstract class AbstractPlayer<T extends TurnBasedGame<T>> implements Player<T> {
 
+    private final Logger logger = LogManager.getLogger(this.getClass());
     private final String name;
 
+    /**
+     * Creates a new player with the given name.
+     *
+     * @param name the player name
+     */
     protected AbstractPlayer(String name) {
         this.name = name;
     }
 
-    protected AbstractPlayer(AbstractPlayer other) {
+    /**
+     * Creates a copy of another player.
+     *
+     * @param other the player to copy
+     */
+    protected AbstractPlayer(AbstractPlayer<T> other) {
         this.name = other.name;
     }
+
     /**
-     * Determines the next move based on the provided game state.
+     * Gets the player's move for the given game state.
+     * A deep copy is provided so the player cannot modify the real state.
      * <p>
-     * The default implementation throws an {@link UnsupportedOperationException},
-     * indicating that concrete subclasses must override this method to provide
-     * actual move logic.
-     * </p>
+     * This method uses the Template Method Pattern: it defines the fixed
+     * algorithm and delegates the variable part to {@link #determineMove(T)}.
      *
-     * @param gameCopy a snapshot of the current game state
-     * @return an integer representing the chosen move
-     * @throws UnsupportedOperationException if the method is not overridden
+     * @param game the current game
+     * @return the chosen move
      */
-    public long getMove(TurnBasedGame gameCopy) {
-        logger.error("Method getMove not implemented.");
-        throw new UnsupportedOperationException("Not supported yet.");
+    public final long getMove(T game) {
+        return determineMove(game.deepCopy());
     }
 
-    public String getName(){
+
+    /**
+     * Determines the player's move using a safe copy of the game.
+     * <p>
+     * This method is called by {@link #getMove(T)} and should contain
+     * the player's strategy for choosing a move.
+     *
+     * @param gameCopy a deep copy of the game
+     * @return the chosen move
+     */
+    protected abstract long determineMove(T gameCopy);
+
+
+    /**
+     * Returns the player's name.
+     *
+     * @return the name
+     */
+    public String getName() {
         return this.name;
     }
 }

framework/src/main/java/org/toop/framework/gameFramework/model/player/MoveProvider.java

@@ -2,6 +2,6 @@ package org.toop.framework.gameFramework.model.player;
 
 import org.toop.framework.gameFramework.model.game.TurnBasedGame;
 
-public interface MoveProvider {
-    long getMove(TurnBasedGame game);
+public interface MoveProvider<T extends TurnBasedGame<T>> {
+    long getMove(T game);
 }

framework/src/main/java/org/toop/framework/gameFramework/model/player/Player.java

@@ -1,5 +1,7 @@
 package org.toop.framework.gameFramework.model.player;
 
 import org.toop.framework.gameFramework.model.game.DeepCopyable;
+import org.toop.framework.gameFramework.model.game.TurnBasedGame;
 
-public interface Player extends NameProvider, MoveProvider, DeepCopyable<Player> {}
+public interface Player<T extends TurnBasedGame<T>> extends NameProvider, MoveProvider<T>, DeepCopyable<Player<T>> {
+}

framework/src/main/java/org/toop/framework/networking/NetworkingClientEventListener.java

@@ -1,13 +1,13 @@
-package org.toop.framework.networking.connection;
+package org.toop.framework.networking;
 
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 import org.toop.framework.SnowflakeGenerator;
 import org.toop.framework.eventbus.EventFlow;
 import org.toop.framework.eventbus.bus.EventBus;
-import org.toop.framework.networking.connection.events.NetworkEvents;
-import org.toop.framework.networking.connection.exceptions.ClientNotFoundException;
-import org.toop.framework.networking.connection.interfaces.NetworkingClientManager;
+import org.toop.framework.networking.events.NetworkEvents;
+import org.toop.framework.networking.exceptions.ClientNotFoundException;
+import org.toop.framework.networking.interfaces.NetworkingClientManager;
 
 public class NetworkingClientEventListener {
     private static final Logger logger = LogManager.getLogger(NetworkingClientEventListener.class);

framework/src/main/java/org/toop/framework/networking/NetworkingClientManager.java

@@ -1,4 +1,4 @@
-package org.toop.framework.networking.connection;
+package org.toop.framework.networking;
 
 import java.util.*;
 import java.util.concurrent.ConcurrentHashMap;
@@ -10,13 +10,13 @@ import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 import org.toop.framework.eventbus.EventFlow;
 import org.toop.framework.eventbus.bus.EventBus;
-import org.toop.framework.networking.connection.events.NetworkEvents;
-import org.toop.framework.networking.connection.exceptions.ClientNotFoundException;
-import org.toop.framework.networking.connection.exceptions.CouldNotConnectException;
-import org.toop.framework.networking.connection.interfaces.NetworkingClient;
-import org.toop.framework.networking.connection.types.NetworkingConnector;
+import org.toop.framework.networking.events.NetworkEvents;
+import org.toop.framework.networking.exceptions.ClientNotFoundException;
+import org.toop.framework.networking.exceptions.CouldNotConnectException;
+import org.toop.framework.networking.interfaces.NetworkingClient;
+import org.toop.framework.networking.types.NetworkingConnector;
 
-public class NetworkingClientManager implements org.toop.framework.networking.connection.interfaces.NetworkingClientManager {
+public class NetworkingClientManager implements org.toop.framework.networking.interfaces.NetworkingClientManager {
     private static final Logger logger = LogManager.getLogger(NetworkingClientManager.class);
 
     private final EventBus eventBus;

framework/src/main/java/org/toop/framework/networking/clients/TournamentNetworkingClient.java

@@ -1,4 +1,4 @@
-package org.toop.framework.networking.connection.clients;
+package org.toop.framework.networking.clients;
 
 import io.netty.bootstrap.Bootstrap;
 import io.netty.channel.*;
@@ -12,9 +12,9 @@ import io.netty.util.CharsetUtil;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 import org.toop.framework.eventbus.bus.EventBus;
-import org.toop.framework.networking.connection.exceptions.CouldNotConnectException;
-import org.toop.framework.networking.connection.handlers.NetworkingGameClientHandler;
-import org.toop.framework.networking.connection.interfaces.NetworkingClient;
+import org.toop.framework.networking.exceptions.CouldNotConnectException;
+import org.toop.framework.networking.handlers.NetworkingGameClientHandler;
+import org.toop.framework.networking.interfaces.NetworkingClient;
 
 import java.net.InetSocketAddress;
 

framework/src/main/java/org/toop/framework/networking/connection/types/ServerMessage.java

@@ -1,3 +0,0 @@
-package org.toop.framework.networking.connection.types;
-
-public record ServerMessage(String message) {}

framework/src/main/java/org/toop/framework/networking/events/NetworkEvents.java

@@ -1,4 +1,4 @@
-package org.toop.framework.networking.connection.events;
+package org.toop.framework.networking.events;
 
 import java.util.*;
 import java.util.concurrent.CompletableFuture;
@@ -6,8 +6,8 @@ import java.util.concurrent.CompletableFuture;
 import org.toop.annotations.AutoResponseResult;
 import org.toop.framework.eventbus.GlobalEventBus;
 import org.toop.framework.eventbus.events.*;
-import org.toop.framework.networking.connection.interfaces.NetworkingClient;
-import org.toop.framework.networking.connection.types.NetworkingConnector;
+import org.toop.framework.networking.interfaces.NetworkingClient;
+import org.toop.framework.networking.types.NetworkingConnector;
 
 /**
  * Defines all event types related to the networking subsystem.
@@ -102,7 +102,7 @@ public class NetworkEvents extends EventsBase {
             implements GenericEvent {}
 
     /** Requests to accept an existing challenge. */
-    public record SendAcceptChallenge(long clientId, long challengeId)
+    public record SendAcceptChallenge(long clientId, int challengeId)
             implements GenericEvent {}
 
     /** Requests to forfeit the current game. */

framework/src/main/java/org/toop/framework/networking/exceptions/ClientNotFoundException.java

@@ -1,4 +1,4 @@
-package org.toop.framework.networking.connection.exceptions;
+package org.toop.framework.networking.exceptions;
 
 /**
  * Thrown when an operation is attempted on a networking client

framework/src/main/java/org/toop/framework/networking/exceptions/CouldNotConnectException.java

@@ -1,4 +1,4 @@
-package org.toop.framework.networking.connection.exceptions;
+package org.toop.framework.networking.exceptions;
 
 public class CouldNotConnectException extends RuntimeException {
 

framework/src/main/java/org/toop/framework/networking/exceptions/NetworkingInitializationException.java

@@ -1,4 +1,4 @@
-package org.toop.framework.networking.connection.exceptions;
+package org.toop.framework.networking.exceptions;
 
 public class NetworkingInitializationException extends RuntimeException {
     public NetworkingInitializationException(String message, Throwable cause) {

framework/src/main/java/org/toop/framework/networking/handlers/NetworkingGameClientHandler.java

@@ -1,4 +1,4 @@
-package org.toop.framework.networking.connection.handlers;
+package org.toop.framework.networking.handlers;
 
 import io.netty.channel.ChannelHandlerContext;
 import io.netty.channel.ChannelInboundHandlerAdapter;
@@ -9,7 +9,7 @@ import java.util.regex.Pattern;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 import org.toop.framework.eventbus.bus.EventBus;
-import org.toop.framework.networking.connection.events.NetworkEvents;
+import org.toop.framework.networking.events.NetworkEvents;
 
 public class NetworkingGameClientHandler extends ChannelInboundHandlerAdapter {
     private static final Logger logger = LogManager.getLogger(NetworkingGameClientHandler.class);

framework/src/main/java/org/toop/framework/networking/interfaces/NetworkingClient.java

@@ -1,6 +1,6 @@
-package org.toop.framework.networking.connection.interfaces;
+package org.toop.framework.networking.interfaces;
 
-import org.toop.framework.networking.connection.exceptions.CouldNotConnectException;
+import org.toop.framework.networking.exceptions.CouldNotConnectException;
 
 import java.net.InetSocketAddress;
 

framework/src/main/java/org/toop/framework/networking/interfaces/NetworkingClientManager.java

@@ -1,8 +1,8 @@
-package org.toop.framework.networking.connection.interfaces;
+package org.toop.framework.networking.interfaces;
 
-import org.toop.framework.networking.connection.exceptions.ClientNotFoundException;
-import org.toop.framework.networking.connection.exceptions.CouldNotConnectException;
-import org.toop.framework.networking.connection.types.NetworkingConnector;
+import org.toop.framework.networking.exceptions.ClientNotFoundException;
+import org.toop.framework.networking.exceptions.CouldNotConnectException;
+import org.toop.framework.networking.types.NetworkingConnector;
 
 public interface NetworkingClientManager {
     void startClient(

framework/src/main/java/org/toop/framework/networking/server/ConnectionHandler.java

@@ -1,155 +0,0 @@
-package org.toop.framework.networking.server;
-
-import io.netty.channel.ChannelHandlerContext;
-import io.netty.channel.SimpleChannelInboundHandler;
-import org.apache.maven.surefire.shared.utils.StringUtils;
-
-import java.util.Arrays;
-import java.util.Iterator;
-import java.util.LinkedList;
-import java.util.List;
-
-public class ConnectionHandler extends SimpleChannelInboundHandler<String> {
-
-    private final User user;
-    private final Server server;
-
-    public ConnectionHandler(User user, Server server) {
-        this.user = user;
-        this.server = server;
-    }
-
-    private String returnQuotedString(Iterator<String> strings) { // TODO more places this could be useful
-        return "\"" + StringUtils.join(strings, "\",\"") + "\"";
-    }
-
-    @Override
-    public void channelActive(ChannelHandlerContext ctx) {
-        ctx.writeAndFlush("WELCOME " + user.id() + "\n");
-
-        user.setCtx(ctx);
-        server.addUser(user); // TODO set correct name on login
-    }
-
-    @Override
-    protected void channelRead0(ChannelHandlerContext ctx, String msg) {
-
-        IO.println(msg);
-
-        ParsedMessage p = parse(msg);
-        if (p == null) return;
-
-        IO.println(p.command() + " " + Arrays.toString(p.args()));
-
-        switch (p.command()) {
-            case "ping" -> ctx.writeAndFlush("PONG\n");
-            case "login" -> handleLogin(p);
-            case "get" -> handleGet(p);
-            case "subscribe" -> handleSubscribe(p);
-            case "move" -> handleMove(p);
-            case "challenge" -> handleChallenge(p);
-            case "message" -> handleMessage(p);
-            case "help" -> handleHelp(p);
-            default -> ctx.writeAndFlush("ERROR Unknown command\n");
-        }
-    }
-
-    // DO NOT INVERT
-    private boolean hasArgs(String... args) {
-        return (args.length >= 1);
-    }
-
-    private void handleLogin(ParsedMessage p) {
-        if (!hasArgs(p.args())) return;
-
-        user.setName(p.args()[0]);
-    }
-
-    private void handleGet(ParsedMessage p) {
-        if (!hasArgs(p.args())) return;
-
-        switch (p.args()[0]) {
-            case "playerlist" -> {
-                var names = server.onlineUsers().stream().map(ServerUser::name).iterator();
-                user.ctx().writeAndFlush("SVR PLAYERLIST " + returnQuotedString(names) + "\n");
-            }
-            case "gamelist" -> {
-                var names = server.gameTypes().stream().iterator();
-                user.ctx().writeAndFlush("SVR GAMELIST " + returnQuotedString(names) + "\n");
-            }
-        }
-    }
-
-    private void handleSubscribe(ParsedMessage p) {
-        // TODO
-    }
-
-    private void handleHelp(ParsedMessage p) {
-        // TODO
-    }
-
-    private void handleMessage(ParsedMessage p) {
-        // TODO
-    }
-
-    private void handleChallenge(ParsedMessage p) {
-        if (!hasArgs(p.args())) return;
-        if (p.args().length < 2) return;
-
-        if (p.args()[0].equalsIgnoreCase("accept")) {
-            try {
-                long id = Long.parseLong(p.args()[1]);
-
-                if (id <= 0) {
-                    user.sendMessage("ERR id must be a positive number \n");
-                    return;
-                }
-
-                server.acceptChallenge(id);
-
-            } catch (NumberFormatException e) {
-                user.sendMessage("ERR id is not a valid number or too big \n");
-                return;
-            }
-            return;
-        }
-
-        server.challengeUser(user.name(), p.args()[0], p.args()[1]);
-    }
-
-    private void handleMove(ParsedMessage p) {
-        if(!hasArgs(p.args())) return;
-
-        // TODO check if not number
-        user.serverPlayer().setMove(1L << Integer.parseInt(p.args()[0]));
-    }
-
-    private ParsedMessage parse(String msg) {
-        // TODO, what if empty string.
-
-        if (msg.isEmpty()) return null;
-
-        msg = msg.trim().toLowerCase();
-
-        List<String> parts = new LinkedList<>(List.of(msg.split(" ")));
-
-        if (parts.size() > 1) {
-            String command = parts.removeFirst();
-            return new ParsedMessage(command, parts.toArray(String[]::new));
-        }
-        else {
-            return new ParsedMessage(msg);
-        }
-    }
-
-    @Override
-    public void exceptionCaught(ChannelHandlerContext ctx, Throwable cause) {
-        cause.printStackTrace();
-        ctx.close();
-    }
-
-    @Override
-    public void channelInactive(ChannelHandlerContext ctx) {
-        server.removeUser(user);
-    }
-}

framework/src/main/java/org/toop/framework/networking/server/Game.java

@@ -1,43 +0,0 @@
-package org.toop.framework.networking.server;
-
-import org.toop.framework.game.gameThreads.ServerThreadBehaviour;
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-
-public class Game implements OnlineGame<TurnBasedGame> {
-
-    private long id;
-    private User[] users;
-    private TurnBasedGame game;
-    private ServerThreadBehaviour gameThread;
-
-    public Game(TurnBasedGame game, User... users) {
-        this.game = game;
-        this.gameThread = new ServerThreadBehaviour(game, (pair) -> notifyMoveMade(pair.getLeft(), pair.getRight()));
-        this.users = users;
-    }
-
-    private void notifyMoveMade(String speler, int move){
-        users[0].sendMessage(String.format("SVR GAME MOVE {PLAYER: \"%s\", DETAILS: \"<reactie spel op zet>\", MOVE: \"%s\"}", speler, move));
-        users[1].sendMessage(String.format("SVR GAME MOVE {PLAYER: \"%s\", DETAILS: \"<reactie spel op zet>\", MOVE: \"%s\"}", speler, move));
-    }
-
-    @Override
-    public long id() {
-        return id;
-    }
-
-    @Override
-    public TurnBasedGame game() {
-        return game;
-    }
-
-    @Override
-    public User[] users() {
-        return users;
-    }
-
-    @Override
-    public void start(){
-        this.gameThread.start();
-    }
-}

framework/src/main/java/org/toop/framework/networking/server/GameChallenge.java

@@ -1,47 +0,0 @@
-package org.toop.framework.networking.server;
-
-import org.toop.framework.SnowflakeGenerator;
-
-public class GameChallenge {
-    private final long id = SnowflakeGenerator.nextId(); // I don't need this, but the tournament server uses it...
-
-    private final User from;
-    private final User to;
-    private final String gameType;
-    private final SimpleTimer timer;
-
-    private boolean isChallengeAccepted = false;
-
-    public GameChallenge(User from, User to, String gameType, SimpleTimer timer) {
-        this.from = from;
-        this.to = to;
-        this.gameType = gameType;
-        this.timer = timer;
-    }
-
-    public long id() {
-        return id;
-    }
-
-    public User[] getUsers() {
-        return new User[]{from, to};
-    }
-
-    public void forceExpire() {
-        timer.forceExpire();
-    }
-
-    public String acceptChallenge() {
-        isChallengeAccepted = true;
-        timer.forceExpire();
-        return gameType;
-    }
-
-    public boolean isChallengeAccepted() {
-        return isChallengeAccepted;
-    }
-
-    public boolean isExpired() {
-        return timer.isExpired();
-    }
-}

framework/src/main/java/org/toop/framework/networking/server/GameChallengeTimer.java

@@ -1,33 +0,0 @@
-package org.toop.framework.networking.server;
-
-import java.time.Instant;
-import java.time.Duration;
-
-public class GameChallengeTimer implements SimpleTimer {
-
-    private final Instant createdAt;
-    private final Duration timeout;
-
-    private boolean isExpired = false;
-
-    public GameChallengeTimer(Duration duration) {
-        this.createdAt = Instant.now();
-        this.timeout = duration;
-    }
-
-    @Override
-    public void forceExpire() {
-        this.isExpired = true;
-    }
-
-    @Override
-    public boolean isExpired() {
-        if (this.isExpired) return true;
-        return Instant.now().isAfter(createdAt.plus(timeout));
-    }
-
-    @Override
-    public long secondsRemaining() {
-        return Duration.between(Instant.now(), createdAt.plus(timeout)).toSeconds();
-    }
-}

framework/src/main/java/org/toop/framework/networking/server/GameDefinition.java

@@ -1,28 +0,0 @@
-package org.toop.framework.networking.server;
-
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-
-import java.lang.reflect.InvocationTargetException;
-
-public class GameDefinition<T> {
-    private final String name;
-    private final Class<T> game;
-
-    public GameDefinition(String name, Class<T> game) {
-        this.name = name;
-        this.game = game;
-    }
-
-    public String name() {
-        return name;
-    }
-
-    public T create(String... users) {
-        try {
-            return game.getDeclaredConstructor().newInstance(users);
-        } catch (InstantiationException | IllegalAccessException | InvocationTargetException | NoSuchMethodException e) {
-            throw new RuntimeException(e);
-        }
-    }
-
-}

framework/src/main/java/org/toop/framework/networking/server/GameServer.java

@@ -1,8 +0,0 @@
-package org.toop.framework.networking.server;
-
-public interface GameServer {
-//    List<?> gameTypes();
-//    List<?> ongoingGames();
-//    void startGame(String gameType, User... users);
-//    String[] onlineUsers();
-}

framework/src/main/java/org/toop/framework/networking/server/MasterServer.java

@@ -1,69 +0,0 @@
-package org.toop.framework.networking.server;
-
-import io.netty.bootstrap.ServerBootstrap;
-import io.netty.channel.*;
-import io.netty.channel.nio.NioEventLoopGroup;
-import io.netty.channel.nio.NioIoHandler;
-import io.netty.channel.socket.nio.NioServerSocketChannel;
-import io.netty.channel.socket.nio.NioSocketChannel;
-import io.netty.handler.codec.LineBasedFrameDecoder;
-import io.netty.handler.codec.string.StringDecoder;
-import io.netty.handler.codec.string.StringEncoder;
-import io.netty.handler.logging.LogLevel;
-import io.netty.handler.logging.LoggingHandler;
-import org.toop.framework.SnowflakeGenerator;
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-
-import java.time.Duration;
-import java.util.Map;
-
-public class MasterServer {
-    private final int port;
-    public final Server gs;
-
-    public MasterServer(int port, Map<String, Class<? extends TurnBasedGame>> gameTypes, Duration challengeDuration) {
-        this.port = port;
-        this.gs = new Server(gameTypes, challengeDuration);
-    }
-
-    public void start() throws InterruptedException {
-
-        EventLoopGroup bossGroup = new NioEventLoopGroup(1);
-        EventLoopGroup workerGroup = new MultiThreadIoEventLoopGroup(NioIoHandler.newFactory());
-
-        try {
-
-            ServerBootstrap bootstrap = new ServerBootstrap();
-            bootstrap.group(workerGroup);
-            bootstrap.channel(NioServerSocketChannel.class);
-            bootstrap.option(ChannelOption.SO_BACKLOG, 128);
-            bootstrap.childOption(ChannelOption.SO_KEEPALIVE, true);
-            bootstrap.handler(new LoggingHandler(LogLevel.INFO));
-            bootstrap.childHandler(
-                    new ChannelInitializer<NioSocketChannel>() {
-                        @Override
-                        protected void initChannel(NioSocketChannel ch) {
-
-                            ChannelPipeline pipeline = ch.pipeline();
-
-                            pipeline.addLast(new LineBasedFrameDecoder(8192));
-                            pipeline.addLast(new StringDecoder());
-                            pipeline.addLast(new StringEncoder());
-
-                            long userid = SnowflakeGenerator.nextId();
-                            User user = new User(userid, ""+userid);
-                            pipeline.addLast(new ConnectionHandler(user, gs));
-                        }
-                    }
-            );
-
-            ChannelFuture future = bootstrap.bind(port).sync();
-            System.out.println("MasterServer listening on port " + port);
-
-            future.channel().closeFuture().sync();
-        } finally {
-            bossGroup.shutdownGracefully();
-            workerGroup.shutdownGracefully();
-        }
-    }
-}

framework/src/main/java/org/toop/framework/networking/server/MessageStore.java

@@ -1,7 +0,0 @@
-package org.toop.framework.networking.server;
-
-public interface MessageStore {
-    void add(String message);
-    String get();
-    void reset();
-}

framework/src/main/java/org/toop/framework/networking/server/OnlineGame.java

@@ -1,8 +0,0 @@
-package org.toop.framework.networking.server;
-
-public interface OnlineGame<T> {
-    long id();
-    T game();
-    User[] users();
-    void start();
-}

framework/src/main/java/org/toop/framework/networking/server/ParsedMessage.java

@@ -1,4 +0,0 @@
-package org.toop.framework.networking.server;
-
-public record ParsedMessage(String command, String... args) {}
-

framework/src/main/java/org/toop/framework/networking/server/Parser.java

@@ -1,4 +0,0 @@
-package org.toop.framework.networking.server;
-
-public class Parser {
-}

framework/src/main/java/org/toop/framework/networking/server/ServableGame.java

@@ -1,4 +0,0 @@
-package org.toop.framework.networking.server;
-
-public interface ServableGame {
-}

framework/src/main/java/org/toop/framework/networking/server/Server.java

@@ -1,182 +0,0 @@
-package org.toop.framework.networking.server;
-
-import org.toop.framework.game.players.ServerPlayer;
-import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-import org.toop.framework.utils.ImmutablePair;
-
-import java.util.Arrays;
-import java.util.List;
-import java.util.Map;
-import java.util.concurrent.*;
-import java.time.Duration;
-
-public class Server implements GameServer {
-
-    final private Map<String, Class<? extends TurnBasedGame>> gameTypes;
-    final private Map<Long, User> users = new ConcurrentHashMap<>();
-    final private List<GameChallenge> gameChallenges = new CopyOnWriteArrayList<>();
-    final private List<OnlineGame<TurnBasedGame>> games = new CopyOnWriteArrayList<>();
-
-    final private Duration challengeDuration;
-    final private ScheduledExecutorService scheduler = Executors.newSingleThreadScheduledExecutor();
-
-    public Server(Map<String, Class<? extends TurnBasedGame>> gameTypes, Duration challengeDuration) {
-        this.gameTypes = gameTypes;
-        this.challengeDuration = challengeDuration;
-
-        scheduler.schedule(this::serverTask, 0, TimeUnit.MILLISECONDS);
-    }
-
-    private void serverTask() {
-        checkChallenges();
-        scheduler.schedule(this::serverTask, 500, TimeUnit.MILLISECONDS);
-    }
-
-    public void addUser(User user) {
-        users.putIfAbsent(user.id(), user);
-    }
-
-    public void removeUser(User user) {
-        users.remove(user.id());
-    }
-
-    public List<String> gameTypes() {
-        return gameTypes.keySet().stream().toList();
-    }
-
-    public List<OnlineGame<TurnBasedGame>> ongoingGames() {
-        return games;
-    }
-
-    public User getUser(String username) {
-        return users.values().stream().filter(e -> e.name().equalsIgnoreCase(username)).findFirst().orElse(null);
-    }
-
-    public User getUser(long id) {
-        return users.get(id);
-    }
-
-    public void challengeUser(String fromUser, String toUser, String gameType) {
-
-        User from = getUser(fromUser);
-        if (from == null) {
-            return;
-        }
-
-        if (!gameTypes.containsKey(gameType)) {
-            from.sendMessage("ERR gametype not found \n");
-            return;
-        }
-
-        User to = getUser(toUser);
-        if (to == null) {
-            from.sendMessage("ERR user not found \n");
-            return;
-        }
-
-        var ch = new GameChallenge(from, to, gameType, new GameChallengeTimer(challengeDuration));
-
-        to.sendMessage(
-                "SVR GAME CHALLENGE {CHALLENGER: \"%s\", CHALLENGENUMBER: \"%s\", GAMETYPE: \"%s\"} \n"
-                        .formatted(from.name(), ch.id(), gameType)
-        );
-
-        if (!isValidChallenge(ch)) {
-            warnUserExpiredChallenge(from, ch.id());
-            ch.forceExpire();
-            return;
-        }
-
-        gameChallenges.addLast(ch);
-    }
-
-    private void warnUserExpiredChallenge(User user, long challengeId) {
-        user.sendMessage("SVR GAME CHALLENGE CANCELLED {CHALLENGENUMBER: \"" + challengeId + "\"}" + "\n");
-    }
-
-    private boolean isValidChallenge(GameChallenge gameChallenge) {
-        for (var user : gameChallenge.getUsers()) {
-            if (users.get(user.id()) == null) {
-                return false;
-            }
-
-            if (user.game() != null) {
-                return false;
-            }
-
-            if (gameChallenge.isExpired()) {
-                return false;
-            }
-        }
-
-        return true;
-    }
-
-    public void checkChallenges() {
-        for (int i = gameChallenges.size() - 1; i >= 0; i--) {
-            var challenge = gameChallenges.get(i);
-
-            if (isValidChallenge(challenge)) continue;
-
-            if (challenge.isExpired()) {
-                if (!challenge.isChallengeAccepted()) Arrays.stream(challenge.getUsers())
-                        .forEach(user -> warnUserExpiredChallenge(user, challenge.id()));
-
-                gameChallenges.remove(i);
-            }
-        }
-    }
-
-    public void acceptChallenge(long challengeId) {
-        for (var challenge : gameChallenges) {
-            if (challenge.id() == challengeId) {
-                startGame(challenge.acceptChallenge(), challenge.getUsers());
-                break;
-            }
-        }
-    }
-
-    public List<GameChallenge> gameChallenges() {
-        return gameChallenges;
-    }
-
-    public void startGame(String gameType, User... users) {
-        if (!gameTypes.containsKey(gameType)) return;
-
-        try {
-            ServerPlayer[] players = new ServerPlayer[users.length];
-            var game = new Game(gameTypes.get(gameType).getDeclaredConstructor().newInstance(), users);
-
-            for (int i = 0; i < users.length; i++) {
-                players[i] = new ServerPlayer(users[i]);
-                users[i].addGame(new ImmutablePair<>(game, players[i]));
-            }
-            System.out.println("Starting Game");
-
-            game.game().init(players);
-            games.addLast(game);
-
-            users[0].sendMessage(String.format("SVR GAME MATCH {PLAYERTOMOVE: \"%s\", GAMETYPE: \"%s\", OPPONENT: \"%s\"}\n",
-                    users[0].name(),
-                    gameType,
-                    users[1].name()));
-            users[1].sendMessage(String.format("SVR GAME MATCH {PLAYERTOMOVE: \"%s\", GAMETYPE: \"%s\", OPPONENT: \"%s\"}\n",
-                    users[0].name(),
-                    gameType,
-                    users[0].name()));
-            game.start();
-        } catch (Exception ignored) {}
-    }
-
-    public List<User> onlineUsers() {
-        return users.values().stream().toList();
-    }
-
-    public void closeServer() {
-        scheduler.shutdown();
-        gameChallenges.clear();
-        games.clear();
-        users.clear();
-        gameTypes.clear();
-    }
-}

framework/src/main/java/org/toop/framework/networking/server/ServerMessageStore.java

@@ -1,27 +0,0 @@
-package org.toop.framework.networking.server;
-
-import java.util.Queue;
-
-public class ServerMessageStore implements MessageStore {
-
-    Queue<String> messageQueue;
-
-    public ServerMessageStore(Queue<String> messageQueue) {
-        this.messageQueue = messageQueue;
-    }
-
-    @Override
-    public void add(String message) {
-        messageQueue.offer(message);
-    }
-
-    @Override
-    public String get() {
-        return messageQueue.poll();
-    }
-
-    @Override
-    public void reset() {
-        messageQueue.clear();
-    }
-}

framework/src/main/java/org/toop/framework/networking/server/ServerUser.java

@@ -1,15 +0,0 @@
-package org.toop.framework.networking.server;
-
-import org.toop.framework.game.players.ServerPlayer;
-import org.toop.framework.utils.Pair;
-
-public interface ServerUser {
-    long id();
-    String name();
-    Game game();
-    ServerPlayer serverPlayer();
-    void addGame(Pair<Game, ServerPlayer> gamePair);
-    void removeGame();
-    void setName(String name);
-    void sendMessage(String message);
-}

framework/src/main/java/org/toop/framework/networking/server/SimpleTimer.java

@@ -1,7 +0,0 @@
-package org.toop.framework.networking.server;
-
-public interface SimpleTimer {
-    void forceExpire();
-    boolean isExpired();
-    long secondsRemaining();
-}

framework/src/main/java/org/toop/framework/networking/server/User.java

@@ -1,69 +0,0 @@
-package org.toop.framework.networking.server;
-
-import io.netty.channel.ChannelHandlerContext;
-import org.toop.framework.game.players.ServerPlayer;
-import org.toop.framework.utils.Pair;
-
-public class User implements ServerUser {
-    final private long id;
-    private String name;
-    private Pair<Game, ServerPlayer> gamePair;
-    private ChannelHandlerContext connectionContext;
-
-    public User(long userId, String name) {
-        this.id = userId;
-        this.name = name;
-    }
-
-    @Override
-    public long id() {
-        return id;
-    }
-
-    @Override
-    public String name() {
-        return name;
-    }
-
-    @Override
-    public void addGame(Pair<Game, ServerPlayer> gamePair) {
-        if (this.gamePair == null) {
-            this.gamePair = gamePair;
-        }
-    }
-
-    @Override
-    public void removeGame() {
-        this.gamePair = null;
-    }
-
-    @Override
-    public Game game() {
-        return this.gamePair.getLeft();
-    }
-
-    public ServerPlayer serverPlayer() {
-        return this.gamePair.getRight();
-    }
-
-    @Override
-    public void setName(String name) {
-        this.name = name;
-    }
-
-    @Override
-    public void sendMessage(String message) {
-        IO.println(message);
-        ctx().channel().writeAndFlush(message);
-    }
-
-    public ChannelHandlerContext ctx() {
-        return connectionContext;
-    }
-
-    public void setCtx(ChannelHandlerContext ctx) {
-        this.connectionContext = ctx;
-    }
-
-
-}

framework/src/main/java/org/toop/framework/networking/types/NetworkingConnector.java

@@ -1,4 +1,4 @@
-package org.toop.framework.networking.connection.types;
+package org.toop.framework.networking.types;
 
 import java.util.concurrent.TimeUnit;
 

framework/src/main/java/org/toop/framework/networking/types/ServerCommand.java

@@ -1,3 +1,3 @@
-package org.toop.framework.networking.connection.types;
+package org.toop.framework.networking.types;
 
 public record ServerCommand(long clientId, String command) {}

framework/src/main/java/org/toop/framework/networking/types/ServerMessage.java

@@ -0,0 +1,3 @@
+package org.toop.framework.networking.types;
+
+public record ServerMessage(String message) {}

framework/src/main/java/org/toop/framework/resource/resources/MusicAsset.java

@@ -82,6 +82,10 @@ public class MusicAsset extends BaseResource implements LoadableResource, AudioR
         getMediaPlayer().pause();
     }
 
+    public double getVolume() {
+        return volume;
+    }
+
     @Override
     public void stop() {
         getMediaPlayer().stop();

framework/src/main/java/org/toop/framework/resource/resources/SettingsAsset.java

@@ -2,6 +2,9 @@ package org.toop.framework.resource.resources;
 
 import java.io.File;
 import java.util.Locale;
+
+import org.toop.framework.resource.ResourceManager;
+import org.toop.framework.resource.ResourceMeta;
 import org.toop.framework.settings.Settings;
 
 public class SettingsAsset extends JsonAsset<Settings> {
@@ -54,6 +57,10 @@ public class SettingsAsset extends JsonAsset<Settings> {
         return getContent().firstReversi;
     }
 
+    public boolean getSwitchReversi() {
+        return getContent().switchReversi;
+    }
+
     public void setVolume(int volume) {
         getContent().volume = volume;
         save();
@@ -108,4 +115,51 @@ public class SettingsAsset extends JsonAsset<Settings> {
         getContent().firstReversi = firstReversi;
         save();
     }
+
+    public void setSwitchReversi(boolean switchReversi) {
+        getContent().switchReversi = switchReversi;
+        save();
+    }
+
+    public void setContent(Settings settings) {
+        setVolume(settings.volume);
+        setFxVolume(settings.fxVolume);
+        setMusicVolume(settings.musicVolume);
+        setLocale(settings.locale);
+        setFullscreen(settings.fullScreen);
+        setTheme(settings.theme);
+        setLayoutSize(settings.layoutSize);
+        setTutorialFlag(settings.showTutorials);
+        setFirstTTT(settings.firstTTT);
+        setFirstConnect4(settings.firstConnect4);
+        setFirstReversi(settings.firstReversi);
+        setSwitchReversi(settings.switchReversi);
+    }
+
+    public static SettingsAsset getPath() {
+        SettingsAsset settingsAsset;
+        try {
+            settingsAsset = ResourceManager.get("settings.json");
+        } catch (Exception e) {
+        String os = System.getProperty("os.name").toLowerCase();
+        String basePath;
+
+        if (os.contains("win")) {
+            basePath = System.getenv("APPDATA");
+            if (basePath == null) {
+                basePath = System.getProperty("user.home");
+            }
+        } else if (os.contains("mac")) {
+            basePath = System.getProperty("user.home") + "/Library/Application Support";
+        } else {
+            basePath = System.getProperty("user.home") + "/.config";
+        }
+
+        File settingsFile =
+                new File(basePath + File.separator + "ISY1" + File.separator + "settings.json");
+        settingsAsset = new SettingsAsset(settingsFile);
+        ResourceManager.addAsset(new ResourceMeta<>("settings.json", settingsAsset));
+        }
+        return settingsAsset;
+    }
 }

framework/src/main/java/org/toop/framework/settings/Settings.java

@@ -12,5 +12,5 @@ public class Settings {
     public Boolean firstReversi;
     public Boolean firstTTT;
     public Boolean firstConnect4;
-
+    public boolean switchReversi = false;
 }

framework/src/main/java/org/toop/framework/utils/ImmutablePair.java

@@ -1,21 +0,0 @@
-package org.toop.framework.utils;
-
-public class ImmutablePair<T, K> implements Pair<T,K> {
-    final T left;
-    final K right;
-
-    public ImmutablePair(T left, K right) {
-        this.left = left;
-        this.right = right;
-    }
-
-    @Override
-    public T getLeft() {
-        return left;
-    }
-
-    @Override
-    public K getRight() {
-        return right;
-    }
-}

framework/src/main/java/org/toop/framework/utils/MutablePair.java

@@ -1,29 +0,0 @@
-package org.toop.framework.utils;
-
-public class MutablePair<T, K> implements Pair<T,K> {
-    T left;
-    K right;
-
-    public MutablePair(T left, K right) {
-        this.left = left;
-        this.right = right;
-    }
-
-    @Override
-    public T getLeft() {
-        return left;
-    }
-
-    public void setLeft(T left) {
-        this.left = left;
-    }
-
-    @Override
-    public K getRight() {
-        return right;
-    }
-
-    public void setRight(K right) {
-        this.right = right;
-    }
-}

framework/src/main/java/org/toop/framework/utils/Pair.java

@@ -1,6 +0,0 @@
-package org.toop.framework.utils;
-
-public interface Pair<T, K> {
-    T getLeft();
-    K getRight();
-}

framework/src/test/java/org/toop/framework/audio/MusicAssetTest.java

@@ -0,0 +1,128 @@
+package org.toop.framework.audio;
+
+import javafx.application.Platform;
+import javafx.scene.media.MediaPlayer;
+import javafx.util.Duration;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Test;
+import org.toop.framework.resource.resources.MusicAsset;
+
+import java.io.File;
+import java.net.URL;
+import java.util.concurrent.CountDownLatch;
+
+import static org.junit.jupiter.api.Assertions.*;
+
+
+public class MusicAssetTest {
+
+    private static boolean started = false;
+    private MusicAsset musicAsset;
+
+    @BeforeAll
+    static void initJavaFX() throws Exception {
+        if (started) {
+            return;
+        }
+        CountDownLatch countDownLatch = new CountDownLatch(1);
+        Platform.startup(countDownLatch::countDown);
+        countDownLatch.await();
+        started = true;
+    }
+
+    private static void run(Runnable action) {
+        CountDownLatch countDownLatch = new CountDownLatch(1);
+        Platform.runLater(() -> {
+            try {
+                action.run();
+            } finally {
+                countDownLatch.countDown();
+            }
+        });
+        try {
+            countDownLatch.await();
+        } catch (InterruptedException e) {
+            e.printStackTrace();
+        }
+    }
+
+    private MusicAsset createMusicAsset() {
+        URL asset = getClass().getClassLoader().getResource("test.mp3");
+        assertNotNull(asset, "Test MP3 not found");
+        File testFile = new File(asset.getFile());
+        assertTrue(testFile.exists(), "File doesn't exist");
+        return new MusicAsset(testFile);
+    }
+
+    private void wait(MediaPlayer player, MediaPlayer.Status status, long timeout) throws InterruptedException {
+        long start = System.currentTimeMillis();
+        while (player.getStatus() != status && System.currentTimeMillis() - start < timeout) {
+            Thread.sleep(10);
+        }
+    }
+
+    @Test
+    void loadMusicAssetAsLoaded() {
+        MusicAsset musicAsset = createMusicAsset();
+        run(musicAsset::load);
+        assertTrue(musicAsset.isLoaded());
+        assertNotNull(musicAsset.getMediaPlayer());
+    }
+
+    @Test
+    void loadMusicAssetAsNotLoaded() {
+        MusicAsset musicAsset = createMusicAsset();
+        run(musicAsset::load);
+        assertTrue(musicAsset.isLoaded());
+        run(musicAsset::unload);
+        assertFalse(musicAsset.isLoaded());
+        assertNotNull(musicAsset.getMediaPlayer());
+    }
+
+    @Test
+    void updateVolume() {
+        MusicAsset musicAsset = createMusicAsset();
+        musicAsset.updateVolume(0.5);
+        assertEquals(0.5, musicAsset.getVolume(), 0.0000001);
+    }
+
+    @Test
+    void playPauseStop() throws InterruptedException {
+        MusicAsset musicAsset = createMusicAsset();
+        run(musicAsset::load);
+
+        run(musicAsset::play);
+        wait(musicAsset.getMediaPlayer(), MediaPlayer.Status.PLAYING, 500);
+        assertEquals(MediaPlayer.Status.PLAYING, musicAsset.getMediaPlayer().getStatus());
+
+        run(musicAsset::pause);
+        wait(musicAsset.getMediaPlayer(), MediaPlayer.Status.PAUSED, 500);
+        assertEquals(MediaPlayer.Status.PAUSED, musicAsset.getMediaPlayer().getStatus());
+
+        run(musicAsset::stop);
+        wait(musicAsset.getMediaPlayer(), MediaPlayer.Status.STOPPED, 500);
+        assertEquals(MediaPlayer.Status.STOPPED, musicAsset.getMediaPlayer().getStatus());
+    }
+
+    @Test
+    void duration() throws InterruptedException {
+        MusicAsset musicAsset = createMusicAsset();
+        run(musicAsset::load);
+        wait(musicAsset.getMediaPlayer(), MediaPlayer.Status.READY, 500);
+        Duration duration = musicAsset.getMediaPlayer().getTotalDuration();
+        assertTrue(duration.toMillis() > 200); // file is shorter than 1 second, so gotta put it to milliseconds
+    }
+
+    @Test
+    void currentPosition() throws InterruptedException {
+        MusicAsset musicAsset = createMusicAsset();
+        run(musicAsset::load);
+
+        run(musicAsset::play);
+        wait(musicAsset.getMediaPlayer(), MediaPlayer.Status.PLAYING, 500);
+        Thread.sleep(200);
+        Duration currentPos = musicAsset.getMediaPlayer().getCurrentTime();
+        assertTrue(currentPos.toMillis() > 0); // file is shorter than 1 second, so gotta put it to milliseconds
+    }
+}
+

framework/src/test/java/org/toop/framework/audio/MusicManagerTest.java

@@ -51,7 +51,7 @@ class MockAudioResource extends BaseResource implements AudioResource {
         stopped = true;
     }
 
-    public void pause() { paused = false; }
+    public void pause() { paused = true; }
 
     @Override
     public long duration() {

framework/src/test/java/org/toop/framework/networking/NetworkingClientManagerTest.java

@@ -9,7 +9,7 @@
 //import org.mockito.*;
 //import org.toop.framework.SnowflakeGenerator;
 //import org.toop.framework.eventbus.EventFlow;
-//import org.toop.framework.networking.connection.events.NetworkEvents;
+//import org.toop.framework.networking.events.NetworkEvents;
 //
 //class NetworkingClientManagerTest {
 //

framework/src/test/java/org/toop/framework/networking/events/NetworkEventsTest.java

@@ -1,4 +1,4 @@
-//package org.toop.framework.networking.connection.events;
+//package org.toop.framework.networking.events;
 //
 //import static org.junit.jupiter.api.Assertions.*;
 //

framework/src/test/java/org/toop/framework/networking/server/ServerTest.java

@@ -1,167 +0,0 @@
-//package org.toop.framework.networking.server;
-//
-//import org.junit.jupiter.api.Assertions;
-//import org.junit.jupiter.api.BeforeEach;
-//import org.junit.jupiter.api.Test;
-//import org.toop.framework.gameFramework.model.game.PlayResult;
-//import org.toop.framework.gameFramework.model.game.TurnBasedGame;
-//import org.toop.framework.gameFramework.model.player.Player;
-//
-//import java.time.Duration;
-//import java.util.Arrays;
-//import java.util.concurrent.ConcurrentHashMap;
-//
-//public class ServerTest {
-//
-//    static class TurnBasedGameMock implements TurnBasedGame {
-//        private Player[] players;
-//
-//        public TurnBasedGameMock() {}
-//
-//        @Override
-//        public void init(Player[] players) {
-//            this.players = players;
-//        }
-//
-//        @Override
-//        public int getCurrentTurn() {
-//            return 0;
-//        }
-//
-//        @Override
-//        public int getPlayerCount() {
-//            return 0;
-//        }
-//
-//        @Override
-//        public int getWinner() {
-//            return 0;
-//        }
-//
-//        @Override
-//        public long[] getBoard() {
-//            return new long[0];
-//        }
-//
-//        @Override
-//        public TurnBasedGame deepCopy() {
-//            return null;
-//        }
-//
-//        @Override
-//        public long getLegalMoves() {
-//            return 0;
-//        }
-//
-//        @Override
-//        public PlayResult play(long move) {
-//            return null;
-//        }
-//
-//        @Override
-//        public Player getPlayer(int index) {
-//            return null;
-//        }
-//
-//    }
-//
-//    static class TestUser implements ServerUser {
-//
-//        final private long id;
-//
-//        private String name;
-//
-//        public TestUser(long id, String name) {
-//            this.id = id;
-//            this.name = name;
-//        }
-//
-//        @Override
-//        public long id() {
-//            return id;
-//        }
-//
-//        @Override
-//        public String name() {
-//            return name;
-//        }
-//
-//        @Override
-//        public Game[] games() {
-//            return new Game[0];
-//        }
-//
-//        @Override
-//        public void addGame(Game game) {
-//
-//        }
-//
-//        @Override
-//        public void removeGame(Game game) {
-//
-//        }
-//
-//        @Override
-//        public void setName(String name) {
-//            this.name = name;
-//        }
-//
-//        @Override
-//        public void sendMessage(String message) {
-//
-//        }
-//    }
-//
-//    private Server server;
-//    private Duration waitTime = Duration.ofSeconds(2);
-//
-//    @BeforeEach
-//    void setup() {
-//
-//        var games = new ConcurrentHashMap<String, Class<? extends TurnBasedGame>>();
-//        games.put("tictactoe", TurnBasedGameMock.class);
-//        games.put("reversi", TurnBasedGameMock.class);
-//
-//        server = new Server(games, waitTime);
-//    }
-//
-//    @Test
-//    void testGameTypes() {
-//        String[] expected = {"tictactoe", "reversi"};
-//        String[] actual = server.gameTypes();
-//
-//        Arrays.sort(expected);
-//        Arrays.sort(actual);
-//
-//        Assertions.assertArrayEquals(expected, actual);
-//    }
-//
-//    @Test
-//    void testChallenge() {
-//        server.addUser(new TestUser(1, "test1"));
-//        server.addUser(new TestUser(2, "test2"));
-//        server.challengeUser("test1", "test2", "tictactoe");
-//
-//        IO.println(server.gameChallenges());
-//
-//        Assertions.assertEquals(1, server.gameChallenges().size());
-//
-//        try {
-//            Thread.sleep(waitTime.plusMillis(100));
-//        } catch (InterruptedException e) {
-//            throw new RuntimeException(e);
-//        }
-//
-//        Assertions.assertEquals(0, server.gameChallenges().size());
-//    }
-//
-//    @Test
-//    void testStartGame() {
-//        server.startGame("tictactoe", new User(0, "A"), new User(1, "B"));
-//        Assertions.assertEquals(1, server.ongoingGames().size());
-//        server.startGame("reversi", new User(0, "A"), new User(1, "B"));
-//        Assertions.assertEquals(2, server.ongoingGames().size());
-//    }
-//
-//
-//}

framework/src/test/java/org/toop/framework/networking/server/servertests.ps1

@@ -1,17 +0,0 @@
-$client = New-Object System.Net.Sockets.TcpClient("localhost", 6666)
-$stream = $client.GetStream()
-$writer = New-Object System.IO.StreamWriter($stream)
-$reader = New-Object System.IO.StreamReader($stream)
-
-$client2 = New-Object System.Net.Sockets.TcpClient("localhost", 6666)
-$stream2 = $client2.GetStream()
-$writer2 = New-Object System.IO.StreamWriter($stream2)
-$reader2 = New-Object System.IO.StreamReader($stream2)
-
-$writer.WriteLine("login john")
-$writer.Flush()
-$reader.ReadLine()
-
-$writer2.WriteLine("login hendrik")
-$writer2.Flush()
-$reader2.ReadLine()

framework/src/test/java/org/toop/framework/settings/SettingsAssetTest.java

@@ -0,0 +1,135 @@
+package org.toop.framework.settings;
+
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.toop.framework.resource.resources.SettingsAsset;
+
+import java.util.Locale;
+
+import static org.junit.jupiter.api.Assertions.*;
+
+public class SettingsAssetTest {
+
+    private Settings copySettings;
+    private SettingsAsset settingsAsset;
+
+    @BeforeEach
+    void setup() {
+       this.settingsAsset = SettingsAsset.getPath();
+        Settings original = settingsAsset.getContent();
+        this.copySettings = new Settings();
+        this.copySettings.volume = original.volume;
+        this.copySettings.fxVolume = original.fxVolume;
+        this.copySettings.musicVolume = original.musicVolume;
+        this.copySettings.locale = original.locale;
+        this.copySettings.fullScreen = original.fullScreen;
+        this.copySettings.theme = original.theme;
+        this.copySettings.layoutSize = original.layoutSize;
+        this.copySettings.showTutorials = original.showTutorials;
+        this.copySettings.firstTTT = original.firstTTT;
+        this.copySettings.firstConnect4 = original.firstConnect4;
+        this.copySettings.firstReversi = original.firstReversi;
+    }
+
+
+    @AfterEach
+    void teardown() {
+        settingsAsset.setContent(copySettings);
+    }
+
+    @Test
+    void testVolume() {
+        settingsAsset.setVolume(55);
+        assertEquals(55, settingsAsset.getVolume());
+    }
+
+    @Test
+    void testFxVolume() {
+        settingsAsset.setFxVolume(33);
+        assertEquals(33, settingsAsset.getFxVolume());
+    }
+
+    @Test
+    void testMusicVolume() {
+        settingsAsset.setMusicVolume(77);
+        assertEquals(77, settingsAsset.getMusicVolume());
+    }
+
+    @Test
+    void testLocale() {
+        settingsAsset.setLocale("es-ES");
+        assertEquals(Locale.forLanguageTag("es-ES"), settingsAsset.getLocale());
+    }
+
+    @Test
+    void testFullscreen() {
+        settingsAsset.setFullscreen(true);
+        assertTrue(settingsAsset.getFullscreen());
+    }
+
+    @Test
+    void testTheme() {
+        settingsAsset.setTheme("dark");
+        assertEquals("dark", settingsAsset.getTheme());
+    }
+
+    @Test
+    void testLayoutsize() {
+        settingsAsset.setLayoutSize("large");
+        assertEquals("large", settingsAsset.getLayoutSize());
+    }
+
+    @Test
+    void testTutorialflag() {
+        settingsAsset.setTutorialFlag(false);
+        assertFalse(settingsAsset.getTutorialFlag());
+    }
+
+    @Test
+    void testTTT() {
+        settingsAsset.setFirstTTT(true);
+        assertTrue(settingsAsset.getFirstTTT());
+    }
+
+    @Test
+    void testConnect4() {
+        settingsAsset.setFirstConnect4(false);
+        assertFalse(settingsAsset.getFirstConnect4());
+    }
+
+    @Test
+    void testReversi() {
+        settingsAsset.setFirstReversi(true);
+        assertTrue(settingsAsset.getFirstReversi());
+    }
+
+    @Test
+    void testReplaceFields() {
+        Settings newSettings = new Settings();
+        newSettings.volume = 10;
+        newSettings.fxVolume = 20;
+        newSettings.musicVolume = 30;
+        newSettings.locale = "fr-FR";
+        newSettings.fullScreen = true;
+        newSettings.theme = "light";
+        newSettings.layoutSize = "medium";
+        newSettings.showTutorials = false;
+        newSettings.firstTTT = false;
+        newSettings.firstConnect4 = true;
+        newSettings.firstReversi = false;
+        settingsAsset.setContent(newSettings);
+        assertEquals(10, settingsAsset.getVolume());
+        assertEquals(20, settingsAsset.getFxVolume());
+        assertEquals(30, settingsAsset.getMusicVolume());
+        assertEquals(Locale.forLanguageTag("fr-FR"), settingsAsset.getLocale());
+        assertTrue(settingsAsset.getFullscreen());
+        assertEquals("light", settingsAsset.getTheme());
+        assertEquals("medium", settingsAsset.getLayoutSize());
+        assertFalse(settingsAsset.getTutorialFlag());
+        assertFalse(settingsAsset.getFirstTTT());
+        assertTrue(settingsAsset.getFirstConnect4());
+        assertFalse(settingsAsset.getFirstReversi());
+    }
+}
+

game/src/main/java/org/toop/game/BitboardGame.java

@@ -1,44 +1,39 @@
-package org.toop.framework.game;
+package org.toop.game;
 
 import org.toop.framework.gameFramework.model.game.TurnBasedGame;
 import org.toop.framework.gameFramework.model.player.Player;
 
 import java.util.Arrays;
+import java.util.concurrent.atomic.AtomicInteger;
 
 // There is AI performance to be gained by getting rid of non-primitives and thus speeding up deepCopy
-public abstract class BitboardGame implements TurnBasedGame {
+public abstract class BitboardGame<T extends BitboardGame<T>> implements TurnBasedGame<T> {
 	private final int columnSize;
 	private final int rowSize;
 
-    private Player[] players;
+    private Player<T>[] players;
 
 	// long is 64 bits. Every game has a limit of 64 cells maximum.
 	private final long[] playerBitboard;
 	private int currentTurn = 0;
-	private final int playerCount;
 
-	public BitboardGame(int columnSize, int rowSize, int playerCount) {
+	public BitboardGame(int columnSize, int rowSize, int playerCount, Player<T>[] players) {
 		this.columnSize = columnSize;
 		this.rowSize = rowSize;
-		this.playerCount = playerCount;
-        this.playerBitboard = new long[playerCount];
-	}
-
-	@Override
-	public void init(Player[] players) {
-		this.players = players;
+        this.players = players;
+		this.playerBitboard = new long[playerCount];
 
 		Arrays.fill(playerBitboard, 0L);
 	}
 
-	public BitboardGame(BitboardGame other) {
+	public BitboardGame(BitboardGame<T> other) {
 		this.columnSize = other.columnSize;
 		this.rowSize = other.rowSize;
-        this.playerCount = other.playerCount;
+
 		this.playerBitboard = other.playerBitboard.clone();
 		this.currentTurn = other.currentTurn;
         this.players = Arrays.stream(other.players)
-                .map(Player::deepCopy)
+                .map(Player<T>::deepCopy)
                 .toArray(Player[]::new);
 	}
 
@@ -66,7 +61,7 @@ public abstract class BitboardGame implements TurnBasedGame {
 		return getCurrentPlayerIndex();
 	}
 
-    public Player getPlayer(int index) {return players[index];}
+    public Player<T> getPlayer(int index) {return players[index];}
 
 	public int getCurrentPlayerIndex() {
         return currentTurn % playerBitboard.length;
@@ -76,7 +71,7 @@ public abstract class BitboardGame implements TurnBasedGame {
 		return (currentTurn + 1) % playerBitboard.length;
 	}
 
-    public Player getCurrentPlayer(){
+    public Player<T> getCurrentPlayer(){
         return players[getCurrentPlayerIndex()];
     }
 

game/src/main/java/org/toop/game/Move.java

@@ -1,3 +1,3 @@
-package org.toop.framework.game;
+package org.toop.game;
 // TODO: Remove this, only used in ReversiCanvas. Needs to not
 public record Move(int position, char value) {}

game/src/main/java/org/toop/game/gameThreads/LocalFixedRateThreadBehaviour.java

@@ -1,4 +1,4 @@
-package org.toop.framework.game.gameThreads;
+package org.toop.game.gameThreads;
 
 import org.toop.framework.eventbus.EventFlow;
 import org.toop.framework.gameFramework.GameState;
@@ -16,7 +16,7 @@ import java.util.function.Consumer;
  * Runs a separate thread that executes game turns at a fixed frequency (default 60 updates/sec),
  * applying player moves, updating the game state, and dispatching UI events.
  */
-public class LocalFixedRateThreadBehaviour extends AbstractThreadBehaviour implements Runnable {
+public class LocalFixedRateThreadBehaviour<T extends TurnBasedGame<T>> extends AbstractThreadBehaviour<T> implements Runnable {
 
 
     /**
@@ -24,7 +24,7 @@ public class LocalFixedRateThreadBehaviour extends AbstractThreadBehaviour imple
      *
      * @param game    the game instance
      */
-    public LocalFixedRateThreadBehaviour(TurnBasedGame game) {
+    public LocalFixedRateThreadBehaviour(T game) {
         super(game);
     }
 
@@ -59,7 +59,7 @@ public class LocalFixedRateThreadBehaviour extends AbstractThreadBehaviour imple
             if (now >= nextUpdate) {
                 nextUpdate += UPDATE_INTERVAL;
 
-                Player currentPlayer = game.getPlayer(game.getCurrentTurn());
+                Player<T> currentPlayer = game.getPlayer(game.getCurrentTurn());
                 long move = currentPlayer.getMove(game.deepCopy());
                 PlayResult result = game.play(move);
 

game/src/main/java/org/toop/game/gameThreads/LocalThreadBehaviour.java

@@ -1,4 +1,4 @@
-package org.toop.framework.game.gameThreads;
+package org.toop.game.gameThreads;
 
 import org.toop.framework.eventbus.EventFlow;
 import org.toop.framework.gameFramework.model.game.threadBehaviour.AbstractThreadBehaviour;
@@ -16,14 +16,14 @@ import java.util.function.Consumer;
  * Repeatedly gets the current player's move, applies it to the game,
  * updates the UI, and stops when the game ends or {@link #stop()} is called.
  */
-public class LocalThreadBehaviour extends AbstractThreadBehaviour implements Runnable {
+public class LocalThreadBehaviour<T extends TurnBasedGame<T>> extends AbstractThreadBehaviour<T> implements Runnable {
 
     /**
      * Creates a new behaviour for a local turn-based game.
      *
      * @param game    the game instance
      */
-    public LocalThreadBehaviour(TurnBasedGame game) {
+    public LocalThreadBehaviour(T game) {
         super(game);
     }
 
@@ -48,7 +48,7 @@ public class LocalThreadBehaviour extends AbstractThreadBehaviour implements Run
     @Override
     public void run() {
         while (isRunning.get()) {
-            Player currentPlayer = game.getPlayer(game.getCurrentTurn());
+            Player<T> currentPlayer = game.getPlayer(game.getCurrentTurn());
             long move = currentPlayer.getMove(game.deepCopy());
             PlayResult result = game.play(move);
 

game/src/main/java/org/toop/game/gameThreads/OnlineThreadBehaviour.java

@@ -1,4 +1,4 @@
-package org.toop.framework.game.gameThreads;
+package org.toop.game.gameThreads;
 
 import org.toop.framework.eventbus.EventFlow;
 import org.toop.framework.gameFramework.model.game.threadBehaviour.AbstractThreadBehaviour;
@@ -6,7 +6,7 @@ import org.toop.framework.gameFramework.view.GUIEvents;
 import org.toop.framework.gameFramework.model.game.TurnBasedGame;
 import org.toop.framework.gameFramework.model.game.SupportsOnlinePlay;
 import org.toop.framework.gameFramework.model.player.Player;
-import org.toop.framework.game.players.OnlinePlayer;
+import org.toop.game.players.OnlinePlayer;
 
 /**
  * Handles online multiplayer game logic.
@@ -14,17 +14,17 @@ import org.toop.framework.game.players.OnlinePlayer;
  * Reacts to server events, sending moves and updating the game state
  * for the local player while receiving moves from other players.
  */
-public class OnlineThreadBehaviour extends AbstractThreadBehaviour implements SupportsOnlinePlay {
+public class OnlineThreadBehaviour<T extends TurnBasedGame<T>> extends AbstractThreadBehaviour<T> implements SupportsOnlinePlay {
     /**
      * Creates behaviour and sets the first local player
      * (non-online player) from the given array.
      */
-    public OnlineThreadBehaviour(TurnBasedGame game) {
+    public OnlineThreadBehaviour(T game) {
         super(game);
     }
 
     /** Finds the first non-online player in the array. */
-    private int getFirstNotOnlinePlayer(Player[] players) {
+    private int getFirstNotOnlinePlayer(Player<T>[] players) {
         for (int i = 0; i < players.length; i++) {
             if (!(players[i] instanceof OnlinePlayer)) {
                 return i;

game/src/main/java/org/toop/game/gameThreads/OnlineWithSleepThreadBehaviour.java

@@ -1,6 +1,7 @@
-package org.toop.framework.game.gameThreads;
+package org.toop.game.gameThreads;
 
 import org.toop.framework.gameFramework.model.game.TurnBasedGame;
+import org.toop.framework.networking.events.NetworkEvents;
 
 /**
  * Online thread behaviour that adds a fixed delay before processing
@@ -9,14 +10,14 @@ import org.toop.framework.gameFramework.model.game.TurnBasedGame;
  * This is identical to {@link OnlineThreadBehaviour}, but inserts a
  * short sleep before delegating to the base implementation.
  */
-public class OnlineWithSleepThreadBehaviour extends OnlineThreadBehaviour {
+public class OnlineWithSleepThreadBehaviour<T extends TurnBasedGame<T>> extends OnlineThreadBehaviour<T> {
 
     /**
      * Creates the behaviour and forwards the players to the base class.
      *
      * @param game the online-capable turn-based game
      */
-    public OnlineWithSleepThreadBehaviour(TurnBasedGame game) {
+    public OnlineWithSleepThreadBehaviour(T game) {
         super(game);
     }
 

game/src/main/java/org/toop/game/games/reversi/BitboardReversi.java

@@ -1,30 +1,25 @@
-package org.toop.framework.game.games.reversi;
+package org.toop.game.games.reversi;
 
-import org.toop.framework.game.BitboardGame;
 import org.toop.framework.gameFramework.GameState;
 import org.toop.framework.gameFramework.model.game.PlayResult;
 import org.toop.framework.gameFramework.model.player.Player;
-import org.toop.framework.game.BitboardGame;
+import org.toop.game.BitboardGame;
 
-public class BitboardReversi extends BitboardGame {
+public class BitboardReversi extends BitboardGame<BitboardReversi> {
 
     public record Score(int black, int white) {}
 
 	private final long notAFile = 0xfefefefefefefefeL;
 	private final long notHFile = 0x7f7f7f7f7f7f7f7fL;
 
-	public BitboardReversi() {
-		super(8, 8, 2);
-	}
+	public BitboardReversi(Player<BitboardReversi>[] players) {
+		super(8, 8, 2, players);
 
-	@Override
-	public void init(Player[] players) {
-		super.init(players);
-        // Black (player 0)
-        setPlayerBitboard(0, (1L << (3 + 4 * 8)) | (1L << (4 + 3 * 8)));
+		// Black (player 0)
+		setPlayerBitboard(0, (1L << (3 + 4 * 8)) | (1L << (4 + 3 * 8)));
 
-        // White (player 1)
-        setPlayerBitboard(1, (1L << (3 + 3 * 8)) | (1L << (4 + 4 * 8)));
+		// White (player 1)
+		setPlayerBitboard(1, (1L << (3 + 3 * 8)) | (1L << (4 + 4 * 8)));
 	}
 
     public BitboardReversi(BitboardReversi other) {

game/src/main/java/org/toop/game/games/tictactoe/BitboardTicTacToe.java

@@ -1,11 +1,11 @@
-package org.toop.framework.game.games.tictactoe;
+package org.toop.game.games.tictactoe;
 
 import org.toop.framework.gameFramework.GameState;
 import org.toop.framework.gameFramework.model.game.PlayResult;
 import org.toop.framework.gameFramework.model.player.Player;
-import org.toop.framework.game.BitboardGame;
+import org.toop.game.BitboardGame;
 
-public class BitboardTicTacToe extends BitboardGame {
+public class BitboardTicTacToe extends BitboardGame<BitboardTicTacToe> {
 	private final long[] winningLines = {
 		0b111000000L, // top row
 		0b000111000L, // middle row
@@ -17,15 +17,9 @@ public class BitboardTicTacToe extends BitboardGame {
 		0b001010100L  // anti-diagonal
 	};
 
-	public BitboardTicTacToe() {
-		super(3, 3, 2);
+	public BitboardTicTacToe(Player<BitboardTicTacToe>[] players) {
+		super(3, 3, 2, players);
 	}
-
-	@Override
-	public void init(Player[] players) {
-		super.init(players);
-	}
-
     public BitboardTicTacToe(BitboardTicTacToe other) {
         super(other);
     }

game/src/main/java/org/toop/game/players/ArtificialPlayer.java

@@ -0,0 +1,55 @@
+package org.toop.game.players;
+
+import org.toop.framework.gameFramework.model.player.*;
+import org.toop.framework.gameFramework.model.game.TurnBasedGame;
+
+/**
+ * Represents a player controlled by an AI.
+ *
+ * @param <T> the type of turn-based game
+ */
+public class ArtificialPlayer<T extends TurnBasedGame<T>> extends AbstractPlayer<T> {
+
+    private final AI<T> ai;
+
+    /**
+     * Creates a new AI-controlled player.
+     *
+     * @param ai the AI controlling this player
+     * @param name the player's name
+     */
+    public ArtificialPlayer(AI<T> ai, String name) {
+        super(name);
+        this.ai = ai;
+    }
+
+    /**
+     * Creates a copy of another AI-controlled player.
+     *
+     * @param other the player to copy
+     */
+    public ArtificialPlayer(ArtificialPlayer<T> other) {
+        super(other);
+        this.ai = other.ai.deepCopy();
+    }
+
+    /**
+     * Determines the player's move using the AI.
+     *
+     * @param gameCopy a copy of the current game
+     * @return the move chosen by the AI
+     */
+    protected long determineMove(T gameCopy) {
+        return ai.getMove(gameCopy);
+    }
+
+    /**
+     * Creates a deep copy of this AI player.
+     *
+     * @return a copy of this player
+     */
+    @Override
+    public ArtificialPlayer<T> deepCopy() {
+        return new ArtificialPlayer<>(this);
+    }
+}

game/src/main/java/org/toop/game/players/LocalPlayer.java

@@ -0,0 +1,87 @@
+package org.toop.game.players;
+
+import org.toop.framework.gameFramework.model.game.TurnBasedGame;
+import org.toop.framework.gameFramework.model.player.AbstractPlayer;
+
+import java.util.concurrent.CompletableFuture;
+import java.util.concurrent.ExecutionException;
+
+/**
+ * Represents a local player who provides moves manually.
+ *
+ * @param <T> the type of turn-based game
+ */
+public class LocalPlayer<T extends TurnBasedGame<T>> extends AbstractPlayer<T> {
+
+    private CompletableFuture<Long> LastMove = new CompletableFuture<>();
+
+    /**
+     * Creates a new local player with the given name.
+     *
+     * @param name the player's name
+     */
+    public LocalPlayer(String name) {
+        super(name);
+    }
+
+    /**
+     * Creates a copy of another local player.
+     *
+     * @param other the player to copy
+     */
+    public LocalPlayer(LocalPlayer<T> other) {
+        super(other);
+        this.LastMove = other.LastMove;
+    }
+
+    /**
+     * Waits for and returns the player's next legal move.
+     *
+     * @param gameCopy a copy of the current game
+     * @return the chosen move
+     */
+    @Override
+    protected long determineMove(T gameCopy) {
+        long legalMoves = gameCopy.getLegalMoves();
+        long move;
+
+        do {
+            move = getLastMove();
+        } while ((legalMoves & move) == 0);
+
+        return move;
+    }
+
+    /**
+     * Sets the player's last move.
+     *
+     * @param move the move to set
+     */
+    public void setLastMove(long move) {
+        LastMove.complete(move);
+    }
+
+    /**
+     * Waits for the next move from the player.
+     *
+     * @return the chosen move or 0 if interrupted
+     */
+    private long getLastMove() {
+        LastMove = new CompletableFuture<>(); // Reset the future
+        try {
+            return LastMove.get();
+        } catch (ExecutionException | InterruptedException e) {
+            return 0;
+        }
+    }
+
+    /**
+     * Creates a deep copy of this local player.
+     *
+     * @return a copy of this player
+     */
+    @Override
+    public LocalPlayer<T> deepCopy() {
+        return new LocalPlayer<>(this);
+    }
+}