Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Client_GUI.java #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update Client_GUI.java #1

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 15 additions & 0 deletions src/main/java/Client_GUI.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -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.*;
Expand All @@ -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;
Expand All @@ -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());
Expand Down Expand Up @@ -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());
Expand Down Expand Up @@ -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);
Expand Down