From 7be22292804b8a731ad223ac65984bde1c22d153 Mon Sep 17 00:00:00 2001 From: Lars Simon Winzer Date: Sat, 14 Mar 2026 13:33:46 +0100 Subject: [PATCH 1/6] Style: Sort imports --- .../cs108/casono/server/network/NetworkManager.java | 12 +++++------- .../casono/server/network/sessions/Session.java | 9 ++++----- 2 files changed, 9 insertions(+), 12 deletions(-) diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java index 50b53d9..cd2f677 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java @@ -1,17 +1,15 @@ package ch.unibas.dmi.dbis.cs108.casono.server.network; -import java.io.IOException; -import java.net.ServerSocket; -import java.net.Socket; - -import org.apache.logging.log4j.LogManager; -import org.apache.logging.log4j.Logger; - import ch.unibas.dmi.dbis.cs108.casono.server.network.events.DisconnectEvent; import ch.unibas.dmi.dbis.cs108.casono.server.network.events.EventBus; import ch.unibas.dmi.dbis.cs108.casono.server.network.sessions.Session; import ch.unibas.dmi.dbis.cs108.casono.server.network.sessions.SessionManager; import ch.unibas.dmi.dbis.cs108.casono.server.network.transport.TcpTransport; +import java.io.IOException; +import java.net.ServerSocket; +import java.net.Socket; +import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; /** * Creates and manages the server socket. Accepts new incoming connections and creates sessions. diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/Session.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/Session.java index d2464f1..11b57d5 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/Session.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/Session.java @@ -1,13 +1,12 @@ package ch.unibas.dmi.dbis.cs108.casono.server.network.sessions; -import java.io.EOFException; -import java.io.IOException; -import org.apache.logging.log4j.LogManager; -import org.apache.logging.log4j.Logger; - import ch.unibas.dmi.dbis.cs108.casono.server.network.events.DisconnectEvent; import ch.unibas.dmi.dbis.cs108.casono.server.network.events.EventBus; import ch.unibas.dmi.dbis.cs108.casono.server.network.transport.TransportLayer; +import java.io.EOFException; +import java.io.IOException; +import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; /** * Represents a client session in the network server. -- 2.52.0 From 573c5fd36d67aaf66c49d294a1f535005cc55655 Mon Sep 17 00:00:00 2001 From: Lars Simon Winzer Date: Sat, 14 Mar 2026 13:37:04 +0100 Subject: [PATCH 2/6] Style: Singleline JavaDoc comments --- .../dmi/dbis/cs108/casono/server/ServerApp.java | 4 +--- .../casono/server/network/NetworkManager.java | 12 +++--------- .../server/network/events/DisconnectEvent.java | 4 +--- .../casono/server/network/events/Event.java | 4 +--- .../casono/server/network/events/EventBus.java | 4 +--- .../casono/server/network/sessions/Session.java | 16 +++++----------- .../server/network/sessions/SessionId.java | 8 ++------ .../server/network/sessions/SessionManager.java | 8 ++------ .../server/network/transport/TcpTransport.java | 4 +--- .../server/network/transport/TransportLayer.java | 4 +--- 10 files changed, 18 insertions(+), 50 deletions(-) diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/ServerApp.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/ServerApp.java index a7814b0..d4cc99b 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/ServerApp.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/ServerApp.java @@ -5,9 +5,7 @@ import ch.unibas.dmi.dbis.cs108.casono.server.network.events.DisconnectEvent; import ch.unibas.dmi.dbis.cs108.casono.server.network.events.EventBus; import ch.unibas.dmi.dbis.cs108.casono.server.network.sessions.SessionManager; -/** - * Application class for starting the server. - */ +/** Application class for starting the server. */ public class ServerApp { public static void start(String arg) { int port = Integer.parseInt(arg); diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java index cd2f677..016cd47 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java @@ -11,9 +11,7 @@ import java.net.Socket; import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; -/** - * Creates and manages the server socket. Accepts new incoming connections and creates sessions. - */ +/** Creates and manages the server socket. Accepts new incoming connections and creates sessions. */ public class NetworkManager implements Runnable { private Integer port; private Logger logger; @@ -39,9 +37,7 @@ public class NetworkManager implements Runnable { this.eventBus.subscribe(DisconnectEvent.class, event -> clientDisconnected(event)); } - /** - * Starts the internal thread to accept new connections. - */ + /** Starts the internal thread to accept new connections. */ public void start() { logger.debug("Starting server at port " + port); thread.start(); @@ -56,9 +52,7 @@ public class NetworkManager implements Runnable { logger.info("Session " + event.sessionId().value() + " disconnected adhasghd"); } - /** - * Runs the network manager loop, accepting connections. - */ + /** Runs the network manager loop, accepting connections. */ @Override public void run() { try (ServerSocket serverSocket = new ServerSocket(port)) { diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/DisconnectEvent.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/DisconnectEvent.java index b6b0d78..a866d56 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/DisconnectEvent.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/DisconnectEvent.java @@ -2,7 +2,5 @@ package ch.unibas.dmi.dbis.cs108.casono.server.network.events; import ch.unibas.dmi.dbis.cs108.casono.server.network.sessions.SessionId; -/** - * Represents a disconnect event for a session. - */ +/** Represents a disconnect event for a session. */ public record DisconnectEvent(SessionId sessionId) implements Event {} diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/Event.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/Event.java index 75c0954..1a8fc4b 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/Event.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/Event.java @@ -1,6 +1,4 @@ package ch.unibas.dmi.dbis.cs108.casono.server.network.events; -/** - * Marker interface for events in the event bus system. - */ +/** Marker interface for events in the event bus system. */ interface Event {} diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/EventBus.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/EventBus.java index 4141ab8..85b8ff8 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/EventBus.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/EventBus.java @@ -7,9 +7,7 @@ import java.util.concurrent.ConcurrentHashMap; import java.util.concurrent.CopyOnWriteArrayList; import java.util.function.Consumer; -/** - * A simple event bus for publishing and subscribing to events. - */ +/** A simple event bus for publishing and subscribing to events. */ public class EventBus { private final Map, List>> handlers = new ConcurrentHashMap<>(); diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/Session.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/Session.java index 11b57d5..d716ac0 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/Session.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/Session.java @@ -8,9 +8,7 @@ import java.io.IOException; import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; -/** - * Represents a client session in the network server. - */ +/** Represents a client session in the network server. */ public class Session implements Runnable { private SessionId id; private Thread thread; @@ -18,12 +16,12 @@ public class Session implements Runnable { private Logger logger; private Boolean running; private EventBus eventBus; - + /** * Creates a new Session with the given transport and event bus. * * @param transport the transport layer for communication - * @param eventBus the event bus for publishing events + * @param eventBus the event bus for publishing events * @throws IOException if an I/O error occurs during initialization */ public Session(TransportLayer transport, EventBus eventBus) throws IOException { @@ -46,9 +44,7 @@ public class Session implements Runnable { return this.id; } - /** - * Starts the session thread. - */ + /** Starts the session thread. */ public void start() { thread.start(); } @@ -63,9 +59,7 @@ public class Session implements Runnable { this.running = false; } - /** - * Runs the session loop, reading from the transport. - */ + /** Runs the session loop, reading from the transport. */ @Override public void run() { while (running) { diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/SessionId.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/SessionId.java index 431ebdc..2645323 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/SessionId.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/SessionId.java @@ -2,15 +2,11 @@ package ch.unibas.dmi.dbis.cs108.casono.server.network.sessions; import java.util.UUID; -/** - * Represents a unique identifier for a session. - */ +/** Represents a unique identifier for a session. */ public class SessionId { private final UUID value; - /** - * Creates a new SessionId with a randomly generated UUID. - */ + /** Creates a new SessionId with a randomly generated UUID. */ public SessionId() { this.value = UUID.randomUUID(); } diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/SessionManager.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/SessionManager.java index 42bc4ac..e12b319 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/SessionManager.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/SessionManager.java @@ -3,15 +3,11 @@ package ch.unibas.dmi.dbis.cs108.casono.server.network.sessions; import java.util.Map; import java.util.concurrent.ConcurrentHashMap; -/** - * Manages active sessions in the server. - */ +/** Manages active sessions in the server. */ public class SessionManager { private Map sessions; - /** - * Constructs a new SessionManager. - */ + /** Constructs a new SessionManager. */ public SessionManager() { this.sessions = new ConcurrentHashMap<>(); } diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/transport/TcpTransport.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/transport/TcpTransport.java index 3f80d6c..bd754bd 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/transport/TcpTransport.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/transport/TcpTransport.java @@ -6,9 +6,7 @@ import java.io.IOException; import java.net.Socket; import java.nio.charset.StandardCharsets; -/** - * Implements TCP-based transport layer for network communication. - */ +/** Implements TCP-based transport layer for network communication. */ public class TcpTransport implements TransportLayer { private Socket socket; private DataInputStream in; diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/transport/TransportLayer.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/transport/TransportLayer.java index f487a9c..4bd9e36 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/transport/TransportLayer.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/transport/TransportLayer.java @@ -2,9 +2,7 @@ package ch.unibas.dmi.dbis.cs108.casono.server.network.transport; import java.io.IOException; -/** - * Interface for transport layer implementations. - */ +/** Interface for transport layer implementations. */ public interface TransportLayer { /** * Reads data from the transport layer. -- 2.52.0 From 34c89b671e3961e8f20cd47e2cac3eacc7622041 Mon Sep 17 00:00:00 2001 From: Lars Simon Winzer Date: Sat, 14 Mar 2026 13:44:58 +0100 Subject: [PATCH 3/6] Style: Unnecessary spaces in JavaDoc parameter and return value labels --- .../dmi/dbis/cs108/casono/server/network/NetworkManager.java | 4 ++-- .../dmi/dbis/cs108/casono/server/network/events/EventBus.java | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java index 016cd47..d962139 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java @@ -23,9 +23,9 @@ public class NetworkManager implements Runnable { /** * Creates a new NetworkManager with the given port, session manager, and event bus. * - * @param port the port to listen on + * @param port the port to listen on * @param sessionManager the session manager to use - * @param eventBus the event bus for events + * @param eventBus the event bus for events */ public NetworkManager(Integer port, SessionManager sessionManager, EventBus eventBus) { this.port = port; diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/EventBus.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/EventBus.java index 85b8ff8..b5e3457 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/EventBus.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/EventBus.java @@ -15,7 +15,7 @@ public class EventBus { * Subscribes a handler to a specific event type. * * @param eventType the class of the event to subscribe to - * @param handler the consumer to handle the event + * @param handler the consumer to handle the event */ @SuppressWarnings("unchecked") // This cast is safe, because handlers only get passed the type they subscribed to public void subscribe(Class eventType, Consumer handler) { -- 2.52.0 From 59494ea2d4c8dc6591d78ba47f707c1a99d8f6ea Mon Sep 17 00:00:00 2001 From: Lars Simon Winzer Date: Sat, 14 Mar 2026 13:45:29 +0100 Subject: [PATCH 4/6] Style: Line length --- .../ch/unibas/dmi/dbis/cs108/casono/server/ServerApp.java | 6 ++++-- .../dbis/cs108/casono/server/network/NetworkManager.java | 5 +++-- .../dbis/cs108/casono/server/network/events/EventBus.java | 4 +++- 3 files changed, 10 insertions(+), 5 deletions(-) diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/ServerApp.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/ServerApp.java index d4cc99b..ad90416 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/ServerApp.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/ServerApp.java @@ -9,11 +9,13 @@ import ch.unibas.dmi.dbis.cs108.casono.server.network.sessions.SessionManager; public class ServerApp { public static void start(String arg) { int port = Integer.parseInt(arg); - System.out.println("You've selected the server. It will accept connections at port " + port); + System.out.println( + "You've selected the server. It will accept connections at port " + port); EventBus eventBus = new EventBus(); SessionManager sessionManager = new SessionManager(); - eventBus.subscribe(DisconnectEvent.class, event -> sessionManager.removeSession(event.sessionId())); + eventBus.subscribe( + DisconnectEvent.class, event -> sessionManager.removeSession(event.sessionId())); NetworkManager networkManager = new NetworkManager(port, sessionManager, eventBus); networkManager.start(); } diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java index d962139..faa2ebd 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java @@ -58,8 +58,9 @@ public class NetworkManager implements Runnable { try (ServerSocket serverSocket = new ServerSocket(port)) { while (running) { Socket clientSocket = serverSocket.accept(); - - System.out.println("Accepted connection from " + clientSocket.getRemoteSocketAddress()); + + System.out.println( + "Accepted connection from " + clientSocket.getRemoteSocketAddress()); Session session = new Session(new TcpTransport(clientSocket), eventBus); sessionManager.addSession(session); diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/EventBus.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/EventBus.java index b5e3457..eb4293d 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/EventBus.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/events/EventBus.java @@ -17,7 +17,9 @@ public class EventBus { * @param eventType the class of the event to subscribe to * @param handler the consumer to handle the event */ - @SuppressWarnings("unchecked") // This cast is safe, because handlers only get passed the type they subscribed to + @SuppressWarnings( + "unchecked") // This cast is safe, because handlers only get passed the type they + // subscribed to public void subscribe(Class eventType, Consumer handler) { handlers.computeIfAbsent(eventType, k -> new CopyOnWriteArrayList<>()) .add((Consumer) (Consumer) handler); -- 2.52.0 From 3ce0a44a189f8b14187bd5ab3662c9894da03fa5 Mon Sep 17 00:00:00 2001 From: Lars Simon Winzer Date: Sat, 14 Mar 2026 13:46:11 +0100 Subject: [PATCH 5/6] Style: Trailing whitespace --- .../cs108/casono/server/network/transport/TcpTransport.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/transport/TcpTransport.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/transport/TcpTransport.java index bd754bd..a7867d3 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/transport/TcpTransport.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/transport/TcpTransport.java @@ -47,7 +47,7 @@ public class TcpTransport implements TransportLayer { byte[] rawPayload = payload.getBytes(StandardCharsets.UTF_8); out.writeInt(rawPayload.length); out.write(rawPayload); - out.flush(); + out.flush(); } /** -- 2.52.0 From 1aafc0bc4b26f39a7b9a0407feef6005e6808a5f Mon Sep 17 00:00:00 2001 From: Lars Simon Winzer Date: Sat, 14 Mar 2026 14:20:01 +0100 Subject: [PATCH 6/6] Style: Use logger instead of System.out.println --- .../ch/unibas/dmi/dbis/cs108/casono/server/ServerApp.java | 7 +++++-- .../dbis/cs108/casono/server/network/NetworkManager.java | 7 +++---- .../dbis/cs108/casono/server/network/sessions/Session.java | 2 +- .../casono/server/network/sessions/SessionManager.java | 2 -- 4 files changed, 9 insertions(+), 9 deletions(-) diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/ServerApp.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/ServerApp.java index ad90416..702d1c8 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/ServerApp.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/ServerApp.java @@ -4,13 +4,16 @@ import ch.unibas.dmi.dbis.cs108.casono.server.network.NetworkManager; import ch.unibas.dmi.dbis.cs108.casono.server.network.events.DisconnectEvent; import ch.unibas.dmi.dbis.cs108.casono.server.network.events.EventBus; import ch.unibas.dmi.dbis.cs108.casono.server.network.sessions.SessionManager; +import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; /** Application class for starting the server. */ public class ServerApp { public static void start(String arg) { int port = Integer.parseInt(arg); - System.out.println( - "You've selected the server. It will accept connections at port " + port); + + Logger logger = LogManager.getLogger(ServerApp.class); + logger.info("Starting server at port {}", port); EventBus eventBus = new EventBus(); SessionManager sessionManager = new SessionManager(); diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java index faa2ebd..ad2e5b3 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/NetworkManager.java @@ -39,7 +39,7 @@ public class NetworkManager implements Runnable { /** Starts the internal thread to accept new connections. */ public void start() { - logger.debug("Starting server at port " + port); + logger.debug("Starting at port {}", port); thread.start(); } @@ -49,7 +49,7 @@ public class NetworkManager implements Runnable { * @param event the disconnect event */ public void clientDisconnected(DisconnectEvent event) { - logger.info("Session " + event.sessionId().value() + " disconnected adhasghd"); + logger.info("Session {} disconnected", event.sessionId().value()); } /** Runs the network manager loop, accepting connections. */ @@ -59,8 +59,7 @@ public class NetworkManager implements Runnable { while (running) { Socket clientSocket = serverSocket.accept(); - System.out.println( - "Accepted connection from " + clientSocket.getRemoteSocketAddress()); + logger.debug("Accepted connection from {}", clientSocket.getRemoteSocketAddress()); Session session = new Session(new TcpTransport(clientSocket), eventBus); sessionManager.addSession(session); diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/Session.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/Session.java index d716ac0..3ed8f88 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/Session.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/Session.java @@ -64,7 +64,7 @@ public class Session implements Runnable { public void run() { while (running) { try { - System.out.println("Recieved: " + transport.read()); + logger.debug("Recieved: {}", transport.read()); } catch (EOFException e) { logger.info("Client disconnected"); eventBus.publish(new DisconnectEvent(id)); diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/SessionManager.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/SessionManager.java index e12b319..6702543 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/SessionManager.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/network/sessions/SessionManager.java @@ -19,7 +19,6 @@ public class SessionManager { */ public void addSession(Session session) { sessions.put(session.getId(), session); - System.out.println("Added session " + session.getId().value() + " to session manager"); } /** @@ -29,7 +28,6 @@ public class SessionManager { * @return the removed session, or null if not found */ public Session removeSession(SessionId id) { - System.out.println("Removed session " + id.value() + " from session manager"); return sessions.remove(id); } -- 2.52.0