diff --git a/src/main/java/Client_GUI.java b/src/main/java/Client_GUI.java index 738efae..065fabe 100644 --- a/src/main/java/Client_GUI.java +++ b/src/main/java/Client_GUI.java @@ -1,3 +1,5 @@ +// GG: +// Paket yok. Default paket kullanımı yerine belirgin bir paket altında olmalı tüm kod. import org.json.JSONObject; import javax.swing.*; @@ -10,6 +12,9 @@ /** * @author ismet abacı */ + +// GG: +// Sınıf isimlerinde _ gibi karakterler genellikle C# stilidir ve Java konvensiyonunda kullanılmazlar. public class Client_GUI extends JFrame{ private JTextField senderField; private JTextField receiverField; @@ -31,6 +36,9 @@ public Client_GUI() { public void actionPerformed(ActionEvent e) { if(fieldChecker()){ + // GG: + // Mesaj nesnesinin oluşturulması bir başka metotta olsa + // okunabilirlik ve cohesion açısından daha iyi olur. JSONObject msg = new JSONObject(); msg.put("sender",senderField.getText()); msg.put("receiver",receiverField.getText()); @@ -69,7 +77,12 @@ private String getPriority(int selectedIndex) { * This method sends a message to server. */ private void sendTheMessage(JSONObject msg) { + // GG: + // Her seferinde bağlantı açıp server'ın yanıtını beklemeden kapatıyorsun. Bunun yerine bağlantıyı + // bir kere açmalı ve server ile iletişim kapsamında devamlı açık tutmalısın. int port = 49999; + // GG: + // try, catch, for vs keywordler öncesi boşluk olmalı try{ Socket s = new Socket("localhost",port); OutputStreamWriter out = new OutputStreamWriter(s.getOutputStream()); @@ -105,6 +118,8 @@ private boolean fieldChecker() { * this method sets the frame for the program */ public static void main(String[] args) { + // GG: + // Frame burada değil, Client_GUI içinde oluşturulmalı. JFrame frame = new JFrame("App"); frame.setContentPane(new Client_GUI().mainPanel); frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);