Socket Client-Server-Applikation bewerten

[chuvak]

Ich hab eine kleine Client-Server-Applikation geschrieben.
Bei dem Server können sich beliebig viele Clients anmelden und miteinander kommunizieren (nur Client mit Server, nicht Client mit Client).

Wenn jemand Zeit hat, könnte er sich das ja mal angucken und bewerten.
Was ist gut? Was kann ich verbessern?

Dankeschön!

 

[XHelp]

So auf den ersten Blick: etwas gängiges wie zip nehmen und mit einem Oberverzeichnis packen 😉

 

[chuvak]

Ich sehe zwar keine großen Vorteile darin, aber ist erledigt!

 

[XHelp]

Naja, wenn man das ins aktuelle Verzeichnis entpackt, muss man anschließend nicht zusammensuchen, was jetzt zu deinem Programm gehört.

 

[tuxedo]

Verbesserungsvorschläge:

1) Bei den Objekten die du serialisierst, solltest du eine SerialVersionUID benutzen. Damit lassen sich gleichnamige Klassen mit unterschiedlichen/erweiterten implementierungen besser unterscheiden

2) Für den Client und den Server hast du Code dupliziert: Inhalt und Information. Üblicherweise, wenn zwei "Projekte" Codefragmente teilen, erstellt man ein drittes Projekt und legt darin den gemeinsamen Code für Server und Client ab. Server und Client haben dann einfach eine Abhängigkeit zu diesem dritten, gemeinsamen Projekt. So musst du nur eine Codestelle anfassen wenn du die gemeinsamen Klassen erweiterst/änderst.

Ansonsten: Ganz okay. hab so auf die schnelle keine groben Schnitzer gefunden.