Mein Chat

[U2nt]

Hallo erstmal, mich gibts auch wieder... War längere Zeit nichtmehr wirklich aktiv... ;(
Anyways, hab mal ne Frage an euch:

Bin grad dabei nen kleinen GUT ORGANISIERTEN Broadcastchat zu coden. Klappt alles einwandfrei, bin eig. auch schon fertig. Wollte nurmal fragen ob ihr vielleicht bestimmte Dinge anders machen würdet oder allgemein ob bestimmte Sachen einfach unnötig sind... Also wie gesagt es soll eben gut organisiert programmiert sein, deswegen hab ich mir mehr Arbeit als nötig gemacht, ums dann später, wenn der "Chat" evtl. mehr Funktionen bietet (aufjedenfall ein MySQL Accountsystem ) es sehr leicht habe.

Nundenn, im Anhang liegt mein ECLIPSE-Projekt Ordner! Ich empfehle es einfach in eclipse zu importieren.

Jaja ich red mal wieder zuviel... Ich mach hier einfachmal Schluss

PS: Bitte nicht "klauen" und sagen es wär seins...

Edit: Glaube passt besser ins Netzwerkprogrammierungsforum...

 

[eRaaaa]

Wollte nurmal fragen ob ihr vielleicht bestimmte Dinge anders machen würdet oder allgemein ob bestimmte Sachen einfach unnötig sind...

Was mir so beim schnellen Überblicken aufgefallen ist:

- Vector würde ich nicht mehr benutzen. Nimm lieber eine der anderen "Listen". Besser wäre es auch, wenn du hier gegen das Interface programmieren würdest, also List<EConnection>.

- ich persönlich bin kein Freund vom Ableiten von JFrame, wenn man da nicht wirklich grundlegendes ändert.

- Exception-Handling könnte man ein wenig überarbeiten +g*

- deine Threads starten sich alle selbst in ihrem Konstruktor...kann man sich jetzt streiten ob man das überhaupt so machen sollte, aber wenn, dann sollten die Klassen in jedem Falle final sein!

- System.exit könntest du auch leicht umgehen. Passe deine Endlosschleifen an(while(running) -> running auf false setzen etc.), rufe dispose() in der GUI auf etc.

 

[U2nt]

Danke erstmal eRaaaa!

Hab mir die Kritik mal durchgelesen und das meiste abgeändert, hier kurz ne Liste. Bitte melde dich darauf wieder, und sag mir ob das so ungefähr deinen Vorstellungen entspricht.

• Der Vector wurde durch eine ArrayList ersetzt!
• JFrame hat jetzt nurnoch eine Klasse welche NICHT von JFrame erbt. (+ Dispose wird nun aufgerufen, statt einfachdem EXIT_ON_CLOSE)
• Exceptionhandling... Gut hab da jetzt ein wenig geändert, aber das werde ich erst "verschönern", wenn die ersten User den "Chat" benutzen. Momentan reicht mir bei jeder Exception die Ausgabe des StackTraces.
• Hab jetzt startThread(), bzw. interruptThread() Methoden angelegt, welche in der jeweiligen Main Klasse aufgerufen werden.
• Könntest mir vielleicht mal erklären, was genau du mit "leicht umgehen" bei dem System.exit meinst. Hab da jetzt nicht wirklich etwas verändert.

Projekt wie eben, im Anhang.

PS: Ich könnt wetten durch die Änderung hat sich mind. 1 Flüchtigkeitsfehler eingeschlichen, werde mir das nach dem Post hier auchnochma angucken und halt selber nochma durchn Kopf gehn lassen.

 

[eRaaaa]

Danke erstmal eRaaaa!

Hab mir die Kritik mal durchgelesen und das meiste abgeändert, hier kurz ne Liste. Bitte melde dich darauf wieder, und sag mir ob das so ungefähr deinen Vorstellungen entspricht.

Also Kritik wollte ich nicht äußern, hab nur geschrieben was ich mir dabei gedacht habe

Der Vector wurde durch eine ArrayList ersetzt!

Gut

[*]JFrame hat jetzt nurnoch eine Klasse welche NICHT von JFrame erbt. (+ Dispose wird nun aufgerufen, statt einfachdem EXIT_ON_CLOSE)

Naja, es geht eher um den Aufruf von System.exit, das sollte man möglichst verhindern, wenn man es kann. Und du kannst es eig. Ich meinte nicht dass du dispose überschreiben sollst um dann dort dein System.exit zu platzieren ^^

Exceptionhandling... Gut hab da jetzt ein wenig geändert, aber das werde ich erst "verschönern", wenn die ersten User den "Chat" benutzen. Momentan reicht mir bei jeder Exception die Ausgabe des StackTraces.

Ok, aber umso länger du damit wartest und umso mehr Code du noch einfügst, desto schwieriger wird es später das noch sauber zu implementieren.

Hab jetzt startThread(), bzw. interruptThread() Methoden angelegt, welche in der jeweiligen Main Klasse aufgerufen werden.
Könntest mir vielleicht mal erklären, was genau du mit "leicht umgehen" bei dem System.exit meinst. Hab da jetzt nicht wirklich etwas verändert.

Naja, ich meinte eben, anstelle dann hart System.exit aufzurufen, kannst du ja einfach isRunning auf false setzen, damit endet deine while-Schleife und somit dann zwangsläufig auch dein Programm.

In EConnection wird btw. immer noch der Thread im Konstruktor gestartet.

 

[U2nt]

Jetzt hab ich verstanden was du meinst! Danke dir.

Ich werd den Post hier in den nächsten 15 Mins. dann nochmal editieren, um dann die neue Source hochzuladen.

 

[U2nt]

Hey... Sorry nochmal aber irgendwie fällt mir auf, das ich nicht ums System.exit herumkomme:
Wenn ich einfach in dispose schreibe "isRunning = false" dann hört ja nur der Thread auf, und nicht das gesamte Programm...? Könntest mir das vlt. nochma genauer erklären, wär richtig cool.

EDIT:
Habe das so abgeändert, dass System.exit(1); nurnoch aufgerufen wird, um das GUI zu schließen. Im Server kommt dieser Begriff nur garnichtmehr vor. Außerdem auch die Exceptions bissel besser gehandelt mit nem JOptionPane (und einer User freundlichen Error-Nachricht). -> Glaube aber eher das ich dann später für die Errors .log Dateien erstellen werde, wo dann der StackTrace etc. drinsteht.

Außerdem hab ich das mit dem Thread jetzt auch in der EConnection geändert.

Hätte jetzt mal eine ganz andere Frage: Findest du meine Einteilung der Packages sinnvoll? Findest du EConnection sollte ins Server package? (Wollte zuerst das EConnection von beiden Seiten geused wird, hat sich jetzt aber so ergeben...)

Danke.

 

[eRaaaa]

Hey... Sorry nochmal aber irgendwie fällt mir auf, das ich nicht ums System.exit herumkomme:
Wenn ich einfach in dispose schreibe "isRunning = false" dann hört ja nur der Thread auf, und nicht das gesamte Programm...? Könntest mir das vlt. nochma genauer erklären, wär richtig cool.

Also irgendwie vermischst du jetzt zwei Sachen. Das dispose sollst du nicht überschreiben, das sollst du aufrufen um das GUI-Fenster zu schließen(und dann somit auch den GUI-Thread wenn keine Fenster mehr sichtbar sind) ! Das mit dem boolean-Flag bezog sich auf deine Endlosschleifen im Server etc.

 

[U2nt]

Jo hast recht, hab da wirklich so ziemlich was durcheinander gebracht. Denke wie ichs jetzt hab, hast dus gemeint. Bitte schaus dir so nochmal an.

Achja, VIELEN DANK das du dir überhaupt die Mühe machst dir das durchzulesen und anzuschauen.

 

[eRaaaa]

Hab mir jetzt nicht nochmal alles angeschaut, aber ja so ungefähr

 

[U2nt]

Dann mach ich mal weiter und bau neue Dinge dazu Aber das werde ich dann nichtmehr in diesem Thread machen.

 

[U2nt]

Ich werde weiterhin Kritik/Verbesserungsvorschläge annehmen. SEHR GERNE auch von anderen Personen.