Zeichen von Telnet lesen

[stulleman]

Hallo Forum

Mein Bruder und ich wollen einen kleine E-Mail Server und Client schreiben. Ich bin dabei für den Server zuständig, der SMTP und POP3 können soll. Natürlich funktioniert das ganze nicht in echt sondern soll nur Lokal in unserem Netzwerk funktionieren.
Mein stand zur Zeit ist das ich zu meinem SMTP Server auf port 25000 via Telnet verbinden kann und eine Begrüßungsnachricht erhalte. Die Probleme entstehen beim Nachrichten empfangen!
Ich weiß nicht ob das normal ist, aber sobald ich ein Zeichen in Telnet eintippe, empfängt mein Server das. Das führt zu einigen Problemen die ich jetzt nur notdürftig beheben konnte:

byte byteIn = 0;
			String message = "";
			
			while(true) {
				byteIn = getMessage();
				
				if(byteIn == 8) {
					message = message.substring(0, message.length()-1);		
				} else if(byteIn == 13) {
					break;
				} else {
					byte[] a = new byte[]{byteIn};
					message += new String(a);
				}
			}
			
			System.out.println(message);

Da ich davon ausgehe das dass sehr schlecht Programmiert ist, hoffe ich jemand kann es mir besser zeigen

 

[turtle]

Können wir auch den Code von getMessage() sehen?

 

[stulleman]

Das wäre vielleicht sinnvoll gewesen

private void sendMessage(String message) {
		try {
			out.write((message + END).getBytes());
		} catch (IOException e) {
			closeConnection();
			return;
		}
	}
	
	private byte[] getMessage() {
		byte[] buffer = new byte[1];
		
		try {
			in.read(buffer);
		} catch (IOException e) {
			e.printStackTrace();
		}
		
		return buffer;
	}

 

[turtle]

Schon besser, aber...

Besonders interessiert hier wie die Objekte in und out erzeugt werden.

Da vermute ich, dass kein BufferedReader genommen wird, auf dem dann ein readLine(), also lese alles bis zum Cariage Return, möglich ist.

 

[stulleman]

Ne ich benutze einemal DataInputStream und einmal DataOutputStream. Da bin ich mir nie sicher welche man am besten verwendet. Zumal ich lesen bis zum nächsten return ja jetzt auch selber implementiert habe

 

[stulleman]

Hmm schade das mir keiner weiterhelfen kann.
Ich habe es jetzt einfach so geregelt, falls es jemanden interessiert

while(true) {
				byte[] charBytes = getMessage();

				if(charBytes[0] != 13) {
					message += new String(charBytes).trim();
				} else {
					break;
				}
			}