H
Hallo,
hier ein paar Vorschläge:
import java.io.;
import java.net.;
Schöner ist es, wenn du nur die Klassen einbindest, die du auch wirklich brauchst.
- Die Fehlerbehandlung ist mies, alle Fehler werden einfach verschluckt und deine Verbinden Methode wirft auch nie im Leben Fehler.
- "public void NetWrapper(String IP, int Port)" kannst du umwandeln zu "public void NetWrapper (Uri url)", so kannst du dir die Überprüfung von Port & IP auf Richtigkeit sparen.
char[] buf = new char[zeichen];
try
{
m_empfang.read(buf, ab, zeichen);
Du verschwendest Speicher für "buf" wenn "ab" > 0 ist.
Die Variablennamen könnten etwas aussagekräftiger sein imo.
- Die verschiedenen Readers werden nirgends geschlossen. Das ist ein Memoryleak.. am Besten rufst du jeweils die close() Methode in einem finally-konstrukt auf.
- Da man ein Objekt dieser Klasse nur für eine Verbindung benutzen kann: Warum nicht die Funktion von Verbinden() in den Konstruktor packen?
Viele Grüße