Verbesserungsvorschläge zur Struktur einer Client Server Desktop Chat App

dvdlly

Aktives Mitglied
Hallo,

Ich bastle gerade an oben beschriebener App herum. Folgendes ist der aktuelle Code
Java:
import java.io.*;
import java.net.Socket;

public class Controller {
    private final String serverIP;
    private final int serverPort;

    public Controller(String ip, int port){
        serverIP = ip;
        serverPort = port;
    }

    public void sendRequest(Request request){
        try {
            Socket clientSocket = new Socket(this.serverIP, this.serverPort);
            ObjectOutputStream out = new ObjectOutputStream(clientSocket.getOutputStream());
            //ObjectInputStream in = new ObjectInputStream(new BufferedInputStream(clientSocket.getInputStream()));
            out.writeObject(request);
        }
        catch (Exception e) {
            e.printStackTrace();
        }
    }

    public static void main(String[]args){
        Controller controller = new Controller("localhost",6666);
        Request message = new Request("message");
        message.setAttribute("sender","1");
        message.setAttribute("receiver","2");
        message.setAttribute("id","default");
        message.setAttribute("message","hello friend!");
        controller.sendRequest(message);
        Request quit = new Request("quit");
        controller.sendRequest(quit);
    }
}
Java:
import java.io.*;
import java.net.*;
import java.sql.*;

public class Server {
    private final int port;


    public Server (int port){
        this.port = port;
    }

    public void addMessage(Request request){
        try {
            if (!request.getType().equals("message")){
                throw new RuntimeException("Error, Request Object is not of type message.");
            }
            Connection connection = DriverManager.
                    getConnection("jdbc:mysql://localhost:3306/Whatsapp","david","21da04ma");
            PreparedStatement stmt = connection.prepareStatement("insert into messages (?,?,?,?)");
            stmt.setString(1,"default");
            stmt.setString(2,request.getAttribute("sender"));
            stmt.setString(3,request.getAttribute("receiver"));
            stmt.setString(4,request.getAttribute("message"));
            int i = stmt.executeUpdate();
            System.out.println(i + " rows inserted!");
            connection.close();
        }
        catch (Exception e){
            e.printStackTrace();
        }
    }

    public void addUser(Request request){
        try {
            if (!request.getType().equals("adduser")){
                throw new RuntimeException("Error, Request Object is not of type adduser");
            }
            Connection connection = DriverManager.
                    getConnection("jdbc:mysql://localhost:3306/Whatsapp","nichtdasrichtigepw","123");

            PreparedStatement stmt = connection.prepareStatement("insert into users (?,?,?)");
            stmt.setString(1,"default");
            stmt.setString(2,request.getAttribute("firstname"));
            stmt.setString(3,request.getAttribute("lastname"));
            int i = stmt.executeUpdate();
            System.out.println(i + " rows inserted!");
            connection.close();
        }
        catch (Exception e){
            e.printStackTrace();
        }
    }

    public void start(){
        boolean running = true;
        while (running) {
            try {
                ServerSocket serverSocket = new ServerSocket(this.port);
                Socket clientSocket = serverSocket.accept();
                ObjectOutputStream out = new ObjectOutputStream(clientSocket.getOutputStream());
                ObjectInputStream in = new ObjectInputStream(new BufferedInputStream(clientSocket.getInputStream()));
                Request request = (Request) in.readObject();
                String type = request.getType();

                switch (type) {
                    case "message":
                        this.addMessage(request);
                        System.out.println("addMessage(request) was called.");
                        break;
                    case "adduser":
                        this.addUser(request);
                        break;
                    case "quit":
                        running = false;
                        break;
                    default:
                        break;
                }
                out.close();
                in.close();
                clientSocket.close();
                serverSocket.close();
            } catch (Exception e) {
                e.printStackTrace();
            }
        }
    }


    public static void main(String[] args){
        Server server = new Server(6666);
        server.start();
    }
}
Java:
import java.io.Serializable;
import java.util.HashMap;
import java.util.Map;

public class Request implements Serializable {
    private Map<String,String> data = new HashMap<>();
    private final String type;
    String getType(){
        return this.type;
    }
    public Request(String type){
        this.type = type;
    }
    public void setAttribute(String key, String value){
        data.put(key,value);
    }
    public String getAttribute(String key){
        return data.get(key);
    }
}
Wenn ich die beiden main Methoden (erst die vom Server, dann vom Controller) ausführe wird allerdings nichts in die DB eingegeben, weiß jemand woran das liegen könnte?
 

LimDul

Top Contributor
Wird irgendwas auf der Konsole ausgegeben?
Wenn nein, lass dir mal beim swtch-case im default Fall, den String type ausgeben.
 

LimDul

Top Contributor
Sollte nicht. Im Zweifelsfall mal debuggen. Laufen den wirklich auch beide Programme? Nicht das intellij den server abschießt, wenn du den client startest.
 
M

Mart

Gast
Java:
while (running) {
            try {
                ServerSocket serverSocket = new ServerSocket(this.port);
                Socket clientSocket = serverSocket.accept();
                ObjectOutputStream out = new ObjectOutputStream(clientSocket.getOutputStream());
                ObjectInputStream in = new ObjectInputStream(new BufferedInputStream(clientSocket.getInputStream()));
du erstellst bei jedem durchlauf einen neuen stream.. ich denke nicht dass das gewollt ist..

und das mit dem switch sieht verdächtig aus...kannst du dir mal den string type ausgeben lassen bevor du in den switch rein gehst
 

LimDul

Top Contributor
Nur mal zur Sicherheit - wenn du beide Programme startest, ist dir bewusst das du auch zwei Konsolen-Fenster hat? (Keine Ahnung wie das bei Intellij ist, bei Eclipse kann ich zwischen denen wechseln).
 

LimDul

Top Contributor
java.sql.SQLException: No suitable driver found for jdbc:mysql://localhost:3306/Whatsapp
addMessage(request) was called.
at java.sql/java.sql.DriverManager.getConnection(DriverManager.java:702)
at java.sql/java.sql.DriverManager.getConnection(DriverManager.java:228)
at de.limdul.javaforum.Server.addMessage(Server.java:24)
at de.limdul.javaforum.Server.start(Server.java:72)
at de.limdul.javaforum.Server.main(Server.java:96)
Bei mir läuft er "sauber" durch (ich war zu faul den JDBC Treiber zu installieren, aber man auf dem Log der Server Anwendung sieht, wird addMessage aufgerufen.
 

dvdlly

Aktives Mitglied
Nur mal zur Sicherheit - wenn du beide Programme startest, ist dir bewusst das du auch zwei Konsolen-Fenster hat?
Ja ist mir bewusst, ich weiß nicht wie das intern bei intellij abläuft. Deswegen habe ich gefragt, ob es damit zusammenhängen kann
 
M

Mart

Gast
eig sollte es dir die IDE im "Regelfall" nicht versau beuteln... eher mal debuggen und schauen wo es harpert
 

LimDul

Top Contributor
Was übrigens nicht funktioniert ist das "quit" senden, weil du jedes mal einen neuen ServerSocket aufmacht, das zerhaut die Logik des Programms und das quit geht in der Regel verloren. Das wird versucht an den "alten" Server Socket zu schicken.
 
M

Mart

Gast
und du läaufst die While einmal durch und schließt sofort wieder die Verbindung ... machst wieder auf usw ... das mit der Schleife und den Objekten darin hoppt noch nich
 
M

Mart

Gast
zusätzlich dein "Server" macht im Moment
- Verbindung aufbauen
- Auf Port hören
- Daten auswerten
- Ausgabe
...usw... das is nich schick und führt zu Problemen die du jetzt hast
 

LimDul

Top Contributor
Grundsätzlich - Eins nach dem anderen.

Bist du sicher das die Server Kommunikation funktioniert?
Bist du sicher das die DB Kommunikation funktioniert?

Die Antwort dürfte auf beide Fragen nein sein.

Schritt 1: Eins von beiden rauswerfen - entweder erst mal eine funktionierende DB Kommunikation bauen oder eine funktionierte TCP/IP Kommunikation - ohne jeweils das andere. Dann, wenn das sauber funktioniert den nächsten Schritt.

Wenn du mit der Server-Kommunikation anfängst: Erstmal nur ein Objekt übertragen und schauen was ankommt, nicht direkt schon Logik mit Verwaltung, Switch & Co bauen. Einfach erstmal einen simplen String übertragen. Kommt der an?
 

Jw456

Top Contributor
Die erste Verbindung und somit auch die erste string wird wohl gehn. Aber danach.
Die Verbindung wird vom Client nicht beendet. Es wird einfach wider eine neue aufgebaut. Ohne die alte zubeenden.
 
M

Mart

Gast
Java:
      while (running) {
                ServerSocket serverSocket = new ServerSocket(this.port);
                Socket clientSocket = serverSocket.accept();
                ObjectOutputStream out = new ObjectOutputStream(clientSocket.getOutputStream());
                ObjectInputStream in = new ObjectInputStream(new BufferedInputStream(clientSocket.getInputStream()));
                Request request = (Request) in.readObject();
                String type = request.getType();

// ...
            
                out.close();
                in.close();
                clientSocket.close();
                serverSocket.close();}
dsa kann einfach so nicht auf dem Server funktionieren... ich weis nicht wie mans besser erklären kann...
ausser "die Objekte die in der While schleife erzeugt werden passen nicht so ganz"

den user hab ich mir noch gar nicht angesehen..


eine if Abrage dauert um die 0,4 nano sec ... du könntest mal einen Zähler mit laufen lassen wie oft sich der Server neu startet in der Sekunde... und nach der Sekunde dann den Zähler ausgeben lässt

dsa ist wie wenn du eine Antenne aufstellst und sobald sie Fertig ist abreisst weil du ja ne neue Brauchst :D
 

Jw456

Top Contributor
Ja schon im vorherigen Thread wurde auch schon gesagt das es besser ist das Serversocket vor der while Schleife zu erstellen. Wurde aber als Tipp nicht angenommen.
 

Jw456

Top Contributor
Das sollte ein Scherz sein.

Hier mal ein link wie ein Server und Client aussehen könnten sollten.
Nicht Thread sicher

Auch bei deinem Code Beispiel sollte das Erstellen des serverSocket nicht im der while sein.
 
M

Mart

Gast
Das sollte ein Scherz sein.

Hier mal ein link wie ein Server und Client aussehen könnten sollten.
Nicht Thread sicher

Auch bei deinem Code Beispiel sollte das Erstellen des serverSocket nicht im der while sein.
ich hatte doch nur seinen Code kopiert und das im Moment nicht so wichtige raus gelöscht
 
Ähnliche Java Themen
  Titel Forum Antworten Datum
N BMI Rechner Was haltet ihr von dem Code habt ihr Verbesserungsvorschläge weil design teschnisch ist das nicht das geilste würde das gerne überarbeiten Java Basics - Anfänger-Themen 12
Vivien Bitte um Optimierungsvorschläge / Verbesserungsvorschläge / allgemeines Feedback Java Basics - Anfänger-Themen 8
N Verbesserungsvorschläge zu Wegfinder Programm Java Basics - Anfänger-Themen 26
S verbesserungsvorschläge? Java Basics - Anfänger-Themen 19
fLooojava first project - Verbesserungsvorschläge am Code Java Basics - Anfänger-Themen 8
Z Zahl Pi probabilistisch berechnen (Kritik/Verbesserungsvorschläge) Java Basics - Anfänger-Themen 4
B Erste Schritte Wechselgeld berechen. Verbesserungsvorschläge Java Basics - Anfänger-Themen 10
D Verbesserungsvorschläge zum Quellcode Java Basics - Anfänger-Themen 15
M Bitte um Verbesserungsvorschläge Java Basics - Anfänger-Themen 14
L Suche Verbesserungsvorschläge für mein erstes Programm Java Basics - Anfänger-Themen 34
Chucky Einfacher Taschenrechner Verbesserungsvorschläge Java Basics - Anfänger-Themen 13
V JSON-Objs aus JSON-Obj filtern und löschen (Manipulation ohne Kenntnis der vollst. Struktur) Java Basics - Anfänger-Themen 12
Soranix Erste Schritte Struktur als Anfänger // Von einer Klasse auf ein Objekt einer anderen Klasse zugreifen. Java Basics - Anfänger-Themen 6
I JSON / XML Struktur mit Vererbung / Interfaces Java Basics - Anfänger-Themen 0
C Java Array Struktur, welche ist wann besser? Java Basics - Anfänger-Themen 12
B Collections Streams - Hilfestellung bei komplexer Struktur Java Basics - Anfänger-Themen 9
M Struktur eines Fotobuches Java Basics - Anfänger-Themen 6
N Struktur zerstört Java Basics - Anfänger-Themen 5
T "Einkauf" & if else Struktur Java Basics - Anfänger-Themen 7
B Import Tabelle - Struktur durch User einstellbar Java Basics - Anfänger-Themen 4
B MVC Struktur mit mehreren Szenen - wer schaut mal bitte drüber? Java Basics - Anfänger-Themen 2
H Empfohlene Projekt-Struktur und mehr ... Java Basics - Anfänger-Themen 5
W OOP Zugriff auf mit Objekt erstellte Struktur Java Basics - Anfänger-Themen 7
P Layer Struktur -> File- Verfügbarkeit Java Basics - Anfänger-Themen 0
N Zugriff bei Package Struktur Java Basics - Anfänger-Themen 6
B Grundsätzliche Klassen-Struktur/Stil Java Basics - Anfänger-Themen 12
F OOP Objekterzeugung in IF-Struktur Java Basics - Anfänger-Themen 2
R BlueJ - Hilfe bei for-Struktur Java Basics - Anfänger-Themen 11
K Bibliothek per "Struktur" anlegen Java Basics - Anfänger-Themen 5
S Ordner / Package Struktur bei größerem Projekt? Java Basics - Anfänger-Themen 8
D Welche API für komplexe XML-Struktur? Java Basics - Anfänger-Themen 25
M Kompositum mit vorgegebener hierarchischer Struktur Java Basics - Anfänger-Themen 6
M Client - DB-Server Struktur Java Basics - Anfänger-Themen 7
B OOP "Optimale" Code-Struktur bei Applikationen mit GUIs Java Basics - Anfänger-Themen 3
T Collections Java Anfänger - Array-Struktur vollkommen unklar Java Basics - Anfänger-Themen 2
P Struktur von Java Programmen? Java Basics - Anfänger-Themen 5
D Methode um Struktur eines Binäbaumes auszugeben Java Basics - Anfänger-Themen 14
? key - value Struktur in ArrayList Java Basics - Anfänger-Themen 3
B Klasse nach Aufbau und Struktur checken? Java Basics - Anfänger-Themen 15
S JTable Struktur in Datei speichern Java Basics - Anfänger-Themen 7
lomtas JTree Struktur im Nachhinein ändern Java Basics - Anfänger-Themen 2
V parent-child Struktur bei Listenern Java Basics - Anfänger-Themen 3
T .für Anfänger - Struktur des Quellcodes in Java? Java Basics - Anfänger-Themen 8
G Struktur bei Animationen mit Threads Java Basics - Anfänger-Themen 4
K Verzeichniss Struktur verstehen Java Basics - Anfänger-Themen 3
M Struktur von Webapplikationsproj. bei Publikation auf Server Java Basics - Anfänger-Themen 2
J Zugriff auf Objektparameter in einer Struktur Java Basics - Anfänger-Themen 3
D Struktur in einer Logdatei Java Basics - Anfänger-Themen 2
D Objekte als XML Struktur speichern Java Basics - Anfänger-Themen 8
V Ergebnisse in einer Struktur speichern Java Basics - Anfänger-Themen 4
M Struktur eines Softwareprojektes Java Basics - Anfänger-Themen 11
A Struktur -> Beste Lösung? Java Basics - Anfänger-Themen 3
M Ausgabe einer ArrayList ensteht nur als Hashcode, nicht als Objekt Java Basics - Anfänger-Themen 16
D 2 ArrayListen gleich sortieren bzw. eine Liste anhand einer anderen Sortieren Java Basics - Anfänger-Themen 6
ixChronos Letzten 4 Ziffern einer großen Zahl ausgeben Java Basics - Anfänger-Themen 3
P Objekt einer Methode eines anderen Objektes übergeben Java Basics - Anfänger-Themen 5
L Variablenwerte aus einer Methode übergeben Java Basics - Anfänger-Themen 2
E Arrays in einer ArrayList miteinander vergleichen Java Basics - Anfänger-Themen 12
Simon16 Java ArrayListe von einer Klasse sortieren Java Basics - Anfänger-Themen 2
Shadowrunner Variablen Gibt es eine Möglichkeit die Ziffern/Stellen einer Zahl fest zu legen? Java Basics - Anfänger-Themen 3
D remove Object von einer Liste von Obejcts Java Basics - Anfänger-Themen 3
FunkyPhil94 Wert in einer Lambda Funktion erhöhen Java Basics - Anfänger-Themen 3
T Aufruf der Methode einer Oberklasse, wenn sie in der Unterklasse überschrieben ist. Polymorphie. Java Basics - Anfänger-Themen 2
B Kommunikation mit Seriellen Schnittstellen + Integration einer lib Java Basics - Anfänger-Themen 1
A Daten aus einer HashMap aus einer DB speichern und mit neuen Werten vergleichen Java Basics - Anfänger-Themen 8
P Welches SDK für das erstellen einer ausführbaren Datei? Java Basics - Anfänger-Themen 4
D Länge einer Liste aufrufen. Java Basics - Anfänger-Themen 19
J Klassen Instanzen einer Klasse in einer anderen unabhängigen Klasse nutzen Java Basics - Anfänger-Themen 4
B Alle Strings bis zu einer Maimallänge aufzählen, die Bedingung erfüllen Java Basics - Anfänger-Themen 13
marcelnedza Finde meinen Fehler in einer Methode nicht, Java Karol Java Basics - Anfänger-Themen 15
MoxMorris Wie macht man String[] = String[] aus einer anderer Methode? Java Basics - Anfänger-Themen 18
T Fibonacci mit einer Hilfsmethode berechnen Java Basics - Anfänger-Themen 10
S Hilfe zu einer Aufgabe Java Basics - Anfänger-Themen 5
M Radius von einer ellipse bestimmen Java Basics - Anfänger-Themen 7
Say Fehlenden Code finden in einer while-Schleife? Java Basics - Anfänger-Themen 11
M Zufallszahl generieren mit einer linken und rechten Grenze Java Basics - Anfänger-Themen 3
N Was Passiert mit dem Namen einer Variable, wenn man diese einer Liste Hinzufügt Java Basics - Anfänger-Themen 16
_user_q Wie eine Methode/Funktion aus einer Klasse mit Constructor aufrufen? Java Basics - Anfänger-Themen 20
W String einer Textdatei in einzelne Stringobjekte pro Zeile aufteilen Java Basics - Anfänger-Themen 14
W Objekte einer ArrayList in txt-datei schreiben mit Paths? Java Basics - Anfänger-Themen 2
S Best Practice Fragen zu Projektstruktur einer Datenbank-Abfrage-App (MVC) Java Basics - Anfänger-Themen 13
T Variable von Objekten in einer Methode überprüfen Java Basics - Anfänger-Themen 26
nelsonmandela Problem bei Ausgabe einer Switch - Case Funktion Java Basics - Anfänger-Themen 5
S Textausgabe in einer For-Schleife Java Basics - Anfänger-Themen 12
M Spezifischen Wert einer Zeile aus .txt Datei entnehmen Java Basics - Anfänger-Themen 15
B Popups mit Klicksabfangen zumAusfüllen einer .ods Datei Java Basics - Anfänger-Themen 0
M RandomAccessFile int und String gleichzeitig in einer Datei Java Basics - Anfänger-Themen 49
E Suchfunktion in einer Liste Java Basics - Anfänger-Themen 39
T ungeordnete Werte-Paare in einer Liste Java Basics - Anfänger-Themen 7
FireHorses Einen Command erst nach einer Chateingabe aktivieren Java Basics - Anfänger-Themen 1
frager2345 Singleton-Muster Java ->Nur eine Instanz einer Klasse erzeugen können Java Basics - Anfänger-Themen 45
F wie kann ich die Position des letzten Vokals innerhalb einer Zeichenkette ermitteln? Java Basics - Anfänger-Themen 5
H Kapselung protected aber in einer Kindklasse nicht zugänglich Java Basics - Anfänger-Themen 5
R Methoden Werte einer ArrayList als Parameter übergeben. Java Basics - Anfänger-Themen 4
B Den Dateipfad einer Java Datei durch Code in Selbiger finden? Java Basics - Anfänger-Themen 10
LilliCherry Array in einer Zeile ausgeben Java Basics - Anfänger-Themen 6
B Attribute eines Objekts einer Klasse durch statische Methode einer 2. Klasse ändern? Java Basics - Anfänger-Themen 32
L Dauerhaftes Speichern einer Eingabe bei einer ArrayList Java Basics - Anfänger-Themen 26
V Hilfe bei Implementierung einer boolean Methode Java Basics - Anfänger-Themen 6
G Position einer unbekannten 3-stelligen-Zahl in einem String finden Java Basics - Anfänger-Themen 15

Ähnliche Java Themen

Neue Themen


Oben