From ec2f795b26f5dd820c9be1e4c37ce519196313dd Mon Sep 17 00:00:00 2001
From: Revivius <Revivius@users.noreply.github.com>
Date: Mon, 5 Oct 2020 22:01:35 +0300
Subject: [PATCH] Update Client_GUI.java
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Selam;

Sınıf ismi;
Try, catch, for vs keyword'lerden sonra blok açarken boşluk bırakılması
Her mesaj gönderileceğinde socket açılıp sunucudan cevap beklenmeden kapatılması yerine 1 kere socket açılıp kullanılması
Javadoc açıklamalarının büyük harf ile başlaması, metot arguman ve geri dönüş tiplerinin açıklanması
---
 src/main/java/Client_GUI.java | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

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);