diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/BudaLogConfig.java b/src/main/java/ch/unibas/dmi/dbis/cs108/BudaLogConfig.java index 0d809eb..0ee0b70 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/BudaLogConfig.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/BudaLogConfig.java @@ -8,7 +8,7 @@ import org.apache.logging.log4j.core.config.Configuration; import org.apache.logging.log4j.core.config.LoggerConfig; /** - * + *A Class that lets us set the RootLogger Level of all Classes in this Project */ public class BudaLogConfig { diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/NightTrainToBudapest.java b/src/main/java/ch/unibas/dmi/dbis/cs108/NightTrainToBudapest.java index 35a3b20..2c61262 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/NightTrainToBudapest.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/NightTrainToBudapest.java @@ -7,6 +7,11 @@ import java.net.InetAddress; public class NightTrainToBudapest { + /** + * The main Method of the jar build of this project. calls either the main method of the server or + * the main method of the client + * @param args the arguments are either client : [] or server + */ public static void main(String[] args) { try { String clientOrServer = args[0]; diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/ClientGameInfoHandler.java b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/ClientGameInfoHandler.java index 375dc47..9317836 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/ClientGameInfoHandler.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/ClientGameInfoHandler.java @@ -1,6 +1,5 @@ package ch.unibas.dmi.dbis.cs108.gamelogic; -import ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur.Passenger; /** * Handles all communication Client to Server concerning gamestate updates i.e. client a has voted diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/Game.java b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/Game.java index ea1db1c..6ab28ee 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/Game.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/Game.java @@ -81,7 +81,7 @@ public class Game implements Runnable { LOGGER.info(gameState.toString()); - for (ClientHandler client : lobbyClients) { + for (ClientHandler client : lobbyClients) {//begins filling the train with clients int index = order[i]; if (passengerTrain[index].getIsGhost()) { //if there is a ghost GhostPlayer ghostPlayer = new GhostPlayer(passengerTrain[index].getPosition(), @@ -94,7 +94,7 @@ public class Game implements Runnable { } i++; } - while (i < order.length) { + while (i < order.length) {//fills the rest of the train with npcs int index = order[i]; if (passengerTrain[index].getIsGhost()) { //if they are a ghost GhostNPC ghostNPC = new GhostNPC(passengerTrain[index].getPosition(), "NPC" + passengerTrain[index].getPosition(),passengerTrain[index].getIsOG()); @@ -109,7 +109,7 @@ public class Game implements Runnable { LOGGER.info(gameState.toString()); i = 0; - while (isOngoing == true) { + while (isOngoing == true) {//game cycle if (!isDay) { LOGGER.info("NIGHT"); gameOverCheck = voteHandler.ghostVote(gameState.getPassengerTrain(), this); diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/GameState.java b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/GameState.java index 3d2388a..344ec44 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/GameState.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/GameState.java @@ -7,8 +7,6 @@ import ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur.Human; import ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur.HumanNPC; import ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur.Passenger; import ch.unibas.dmi.dbis.cs108.multiplayer.server.ClientHandler; -import java.util.Arrays; -import java.util.HashSet; import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/NoiseHandler.java b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/NoiseHandler.java index 4b29d23..cdda0ee 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/NoiseHandler.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/NoiseHandler.java @@ -1,6 +1,6 @@ package ch.unibas.dmi.dbis.cs108.gamelogic; -import ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur.Ghost; + import ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur.Passenger; /** diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/ServerGameInfoHandler.java b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/ServerGameInfoHandler.java index 253b231..061301c 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/ServerGameInfoHandler.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/ServerGameInfoHandler.java @@ -1,12 +1,10 @@ package ch.unibas.dmi.dbis.cs108.gamelogic; import ch.unibas.dmi.dbis.cs108.BudaLogConfig; -import ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur.Ghost; import ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur.GhostNPC; import ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur.HumanNPC; import ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur.Passenger; import ch.unibas.dmi.dbis.cs108.multiplayer.helpers.Protocol; -import ch.unibas.dmi.dbis.cs108.multiplayer.server.ClientHandler; import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/Train.java b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/Train.java index 10212b5..d6041d9 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/Train.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/Train.java @@ -75,8 +75,5 @@ public class Train { return false; } - public static void main(String[] args) { - System.out.println("Hallo"); - } } diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/VoteHandler.java b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/VoteHandler.java index 5fe9e1d..457dee1 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/VoteHandler.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/VoteHandler.java @@ -1,7 +1,6 @@ package ch.unibas.dmi.dbis.cs108.gamelogic; import ch.unibas.dmi.dbis.cs108.BudaLogConfig; -import ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur.Ghost; import ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur.Passenger; import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/klassenstruktur/Ghost.java b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/klassenstruktur/Ghost.java index 515d2b3..b17ab40 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/klassenstruktur/Ghost.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/klassenstruktur/Ghost.java @@ -1,7 +1,6 @@ package ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur; import ch.unibas.dmi.dbis.cs108.BudaLogConfig; -import ch.unibas.dmi.dbis.cs108.gamelogic.Game; import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/klassenstruktur/GhostNPC.java b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/klassenstruktur/GhostNPC.java index c9793cd..c218814 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/klassenstruktur/GhostNPC.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/klassenstruktur/GhostNPC.java @@ -2,9 +2,7 @@ package ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur; import ch.unibas.dmi.dbis.cs108.BudaLogConfig; import ch.unibas.dmi.dbis.cs108.gamelogic.Game; -import ch.unibas.dmi.dbis.cs108.gamelogic.GameState; import ch.unibas.dmi.dbis.cs108.gamelogic.ServerGameInfoHandler; -import ch.unibas.dmi.dbis.cs108.gamelogic.TrainOverflow; import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; @@ -77,10 +75,4 @@ public class GhostNPC extends Ghost { } } - /** - * Decides what to do when a noise ist heard, currently just always broadcasts it - * TODO: Make NPC smarter - */ - - } diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/klassenstruktur/Human.java b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/klassenstruktur/Human.java index 0ac7dbd..f6af8cc 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/klassenstruktur/Human.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/klassenstruktur/Human.java @@ -1,7 +1,6 @@ package ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur; import ch.unibas.dmi.dbis.cs108.BudaLogConfig; -import ch.unibas.dmi.dbis.cs108.gamelogic.Game; import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/klassenstruktur/Spectator.java b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/klassenstruktur/Spectator.java index be229c6..9569714 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/klassenstruktur/Spectator.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/gamelogic/klassenstruktur/Spectator.java @@ -3,7 +3,6 @@ package ch.unibas.dmi.dbis.cs108.gamelogic.klassenstruktur; import ch.unibas.dmi.dbis.cs108.BudaLogConfig; import ch.unibas.dmi.dbis.cs108.gamelogic.Game; import ch.unibas.dmi.dbis.cs108.gamelogic.ServerGameInfoHandler; -import ch.unibas.dmi.dbis.cs108.multiplayer.server.ClientHandler; import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; @@ -23,7 +22,11 @@ public class Spectator extends Passenger{ isSpectator = true; } - + /** + * Sends a message to the Client assosiated with this Spectator via ServerGameInfoHandler.spectatorFormat + * @param msg the message that is sent to this player. + * @param game the game the Passenger lives on + */ @Override public void send(String msg, Game game) { clientHandler.sendMsgToClient(ServerGameInfoHandler.spectatorFormat(msg, this, game)); diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/Client.java b/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/Client.java index 7c666c1..436f601 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/Client.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/Client.java @@ -62,7 +62,6 @@ public class Client { sendMsgToServer(Protocol.clientLogin + "$" + systemName); this.chatApp = new ChatApp(new ClientModel(systemName, this)); this.chatGui = new GUI(this.chatApp); - chatGui.setName(systemName); clientPinger = new ClientPinger(this, this.socket); } catch (IOException e) { e.printStackTrace(); @@ -205,6 +204,10 @@ public class Client { } } + /** + * The main Method used for testing in IDE + * @param args not used in this main + */ public static void main(String[] args) { Scanner sc = new Scanner(System.in); String hostname; @@ -239,6 +242,12 @@ public class Client { } + /** + * The main-method used for the jar build of this project + * @param address the IP address of the Server (can be localhost) + * @param port the port for the connection + * @param username the username of this client + */ public static void main(InetAddress address, int port, String username) { Scanner sc = new Scanner(System.in); Socket socket; diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/JClientProtocolParser.java b/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/JClientProtocolParser.java index 739158c..f497bf0 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/JClientProtocolParser.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/JClientProtocolParser.java @@ -31,7 +31,6 @@ public class JClientProtocolParser { } switch (header) { case Protocol.pingFromServer: - c.sendMsgToServer(Protocol.pingBack); break; case Protocol.pingBack: diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/gui/GUI.java b/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/gui/GUI.java index f7a47d3..06a9d30 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/gui/GUI.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/gui/GUI.java @@ -11,16 +11,13 @@ public class GUI implements Runnable { public static final Logger LOGGER = LogManager.getLogger(GUI.class); public static final BudaLogConfig l = new BudaLogConfig(LOGGER); - ChatApp chatApp; - private String name; + private ChatApp chatApp; + public GUI(ChatApp chatApp) { this.chatApp = chatApp; } - public void setName(String name) { - this.name = name; - } /** * When an object implementing interface {@code Runnable} is used to create a thread, starting the @@ -32,11 +29,9 @@ public class GUI implements Runnable { * @see Thread#run() */ - @Override public void run() { LOGGER.info("here"); - //String name = Application.launch(this.chatApp.getClass()); } } diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/gui/chat/ChatApp.java b/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/gui/chat/ChatApp.java index 5386a24..8b04c5a 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/gui/chat/ChatApp.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/gui/chat/ChatApp.java @@ -1,7 +1,6 @@ package ch.unibas.dmi.dbis.cs108.multiplayer.client.gui.chat; import ch.unibas.dmi.dbis.cs108.BudaLogConfig; -import ch.unibas.dmi.dbis.cs108.multiplayer.client.Client; import ch.unibas.dmi.dbis.cs108.multiplayer.client.gui.ClientModel; import java.net.URL; import java.util.Objects; @@ -25,19 +24,28 @@ public class ChatApp extends Application { public ChatApp() { super(); LOGGER.info("Empty ChatApp constructor got called: "); - } - public ChatApp(ClientModel clientModel) { - this.clientModel = clientModel; - this.chatController = new ChatController(clientModel); + public ChatApp(ClientModel clientM) { + clientModel = clientM; + chatController = new ChatController(clientM); } - public static void setChatController( - ChatController chatC) { + /** + * Sets the ChatController for the Application, needs to be static, but only one application can + * be launched per programm + * + * @param chatC the ChatController to be linked to this chatApp + */ + public static void setChatController(ChatController chatC) { chatController = chatC; } + /** + * Sets the non-static ClientModel field of this class + * + * @param cModel the non static ClientModel to be added + */ public void setcModel(ClientModel cModel) { this.cModel = cModel; } @@ -54,6 +62,11 @@ public class ChatApp extends Application { return clientModel; } + public ChatController getChatController() { + return chatController; + } + + /** * The main entry point for all JavaFX applications. The start method is called after the init * method has returned, and after the system is ready for the application to begin running. @@ -69,11 +82,9 @@ public class ChatApp extends Application { */ @Override public void start(Stage primaryStage) throws Exception { - LOGGER.info("made it here"); this.setcModel(clientModel); URL resource = ChatApp.class.getResource( "splitPaneChatView.fxml"); - LOGGER.info("1"); if (resource == null) { System.out.println("File wasnt found"); } @@ -82,23 +93,16 @@ public class ChatApp extends Application { Parent root = FXMLLoader.load( Objects.requireNonNull(ChatApp.class.getResource( "splitPaneChatView.fxml"))); - LOGGER.info("2"); // TODO bin chatController.getChatPaneRoot() border to root border for rezising Scene scene = new Scene(root); - LOGGER.info("3"); scene.setRoot(root); - LOGGER.info("4"); primaryStage.setScene(scene); - LOGGER.info("5"); } catch (Exception e) { e.printStackTrace(); } primaryStage.setResizable(true); - LOGGER.info("6"); primaryStage.setTitle("Chat"); - LOGGER.info("7"); primaryStage.show(); - LOGGER.info("8"); } @@ -107,7 +111,4 @@ public class ChatApp extends Application { launch(args); } - public ChatController getChatController() { - return chatController; - } } diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/gui/chat/ChatController.java b/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/gui/chat/ChatController.java index 1f535d9..11b6f5a 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/gui/chat/ChatController.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/client/gui/chat/ChatController.java @@ -22,7 +22,6 @@ import javafx.scene.control.SplitPane; import javafx.scene.control.TextArea; import javafx.scene.control.TextField; import javafx.scene.layout.Background; -import javafx.scene.layout.BackgroundFill; import javafx.scene.layout.VBox; import javafx.scene.paint.Color; import org.apache.logging.log4j.LogManager; @@ -53,7 +52,10 @@ public class ChatController implements Initializable { private static final String chatToAll = Protocol.chatMsgToAll; private static final String chatToLobby = Protocol.chatMsgToLobby; - public ChatController() { //TODO: why does this get called + /** + * Needs to stay, because it gets called in initialisation + */ + public ChatController() { super(); whisperTargetChosen = new SimpleBooleanProperty(); cmd = Protocol.chatMsgToLobby + "$"; @@ -154,13 +156,15 @@ public class ChatController implements Initializable { } /** - * @return the client who's chat controller this is + * @return the ClientModel whose chat controller this is */ public static ClientModel getClient() { return client; } /** + * Sets the ClientModel of this ChatController + * * @param client who's gui controller this should be */ public void setClient(ClientModel client) { @@ -172,7 +176,7 @@ public class ChatController implements Initializable { } /** - * The client calls this method to foreward a chat message to the chat gui + * The client calls this method to forward a chat message to the chat gui * * @param msg the message to be displayed */ diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/helpers/ServerPinger.java b/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/helpers/ServerPinger.java index 3ff3421..938fae6 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/helpers/ServerPinger.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/helpers/ServerPinger.java @@ -3,7 +3,6 @@ package ch.unibas.dmi.dbis.cs108.multiplayer.helpers; import ch.unibas.dmi.dbis.cs108.BudaLogConfig; import ch.unibas.dmi.dbis.cs108.multiplayer.server.ClientHandler; import java.io.BufferedWriter; -import java.io.IOException; import java.net.Socket; import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/server/JServerProtocolParser.java b/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/server/JServerProtocolParser.java index b5bb04b..c15f7b8 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/server/JServerProtocolParser.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/server/JServerProtocolParser.java @@ -2,11 +2,6 @@ package ch.unibas.dmi.dbis.cs108.multiplayer.server; import ch.unibas.dmi.dbis.cs108.BudaLogConfig; -import ch.unibas.dmi.dbis.cs108.gamelogic.ClientVoteData; -import ch.unibas.dmi.dbis.cs108.gamelogic.Game; -import ch.unibas.dmi.dbis.cs108.gamelogic.GameState; -import ch.unibas.dmi.dbis.cs108.gamelogic.TrainOverflow; -import ch.unibas.dmi.dbis.cs108.gamelogic.VoteHandler; import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; import ch.unibas.dmi.dbis.cs108.multiplayer.helpers.Protocol; diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/server/Server.java b/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/server/Server.java index fcf5c72..1df003e 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/server/Server.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/multiplayer/server/Server.java @@ -43,6 +43,9 @@ public class Server { } } + /** + * closes the Server socket of this server + */ public void closeServerSocket() { try { if (serverSocket != null) { @@ -53,6 +56,10 @@ public class Server { } } + /** + * The main method used for testing + * @param args not used in this method + */ public static void main(String[] args) { ServerSocket serverSocket = null; gamePort = 1873; @@ -65,6 +72,10 @@ public class Server { server.startServer(); } + /** + * The main method of the Server that is used for the jar build of this project + * @param port the port the server will open on + */ public static void main(int port) { gamePort = port; ServerSocket serverSocket = null;