diff --git a/src/main/java/Server.java b/src/main/java/Server.java index 6885e9b..8c3cacd 100644 --- a/src/main/java/Server.java +++ b/src/main/java/Server.java @@ -9,12 +9,16 @@ * @author ismet abacı */ public class Server { + // GG: + // Neden private değil ? final static Logger logger = Logger.getLogger(String.valueOf(Server.class)); /** * this method listens for a message on a port and process received messages */ public static void main(String[] args) { + // GG: + // Port vs gibi bilgiler ayarlanabilir olmalıdır. Hardcoded olarak kullanılmamalıdır. int port = 49999; try { logger.info("Server started listening (port:" + port + ")..."); @@ -29,11 +33,17 @@ public static void main(String[] args) { */ private static void runServer(int portNumber) throws IOException, InterruptedException { ServerSocket ss = new ServerSocket(portNumber); + // GG: + // Değişkenler neden kullanılacakları yerden uzakta tanımlanıyor ? Değişkenler + // mümkün mertebe kullanılacakları yere yakın tanımlanmalıdır. Socket s; BufferedReader in; JSONObject msg; String input; int numberOfThreads = 3,threadIndex; + // GG: + // Thread oluşturmak masraflı bir işlemdir ve oluşturulan treadlerin yönetilmesi karmaşıktır + // Threadleri kendimiz yaratmak dışında kullanabileceğiniz yöntem nedir ? MyThread [] myThreads = new MyThread[numberOfThreads]; for (int i=0;i