NullPointerException, aber warum?

[Gelöschtes Mitglied 12486]

Ich habe ein Applet mit Socketverbindung zu einem Server geschrieben. Nun habe ich das Problem, dass bei bestimmten Funktionen eine NullPointerException geworfen wird, wenn ich aus dem InputStream etwas einlese.

Die Verbindungen zum Server wird bei mir in eine Klasse gespeichert, hier die run() methode:

public void run() {
		while(true){
			try{
				if(in!=null && quit == false){
					String str = in.readLine();
					System.out.println(str);
					GameApplet.parseCommand(str);
				}else{
					break;
				}
			}catch(IOException e){
				System.out.println("*** Error reading Data from Server ***");
				disconnectFromServer();
			}
		}
		System.out.println("*** Error reading Data from Server ***");
	}

Die hier genutzte parseCommand() Methode sieht so aus.

public void parseCommand(String command){
		if(!command.equals("<pong>"))
			System.out.println("in from Server: "+command);
		if(command.startsWith("<list>")){
			lobbyPanel.setContactList(command);
		}
...

Ich erhalten nun ab und zu eine NullPointerException, nachdem in der run() methode der String eingelesen wurde und an die parseCommand() übergeben wurde. Ich kann direkt vor der Übergabe den String per System.out.println() ausgeben und es ist kein null-objekt. Wenn jedoch dann direkt danach die Übergabe passiert, dann wirft meine parseCommand() MEthode eine NullPointerException, weil angeblich ein null-Objekt übergeben wurde, an eben genau dieser Stelle...


Hat dazu jemand eine Idee?!

 

[SlaterB]

meine Idee ist, dass du etwas falsch machst,

schreibe

public void parseCommand(String command){ 
    if (command == null) {
      System.out.println("command war null"); 
      return;
   } 
      if(!command.equals("<pong>")) 
         System.out.println("in from Server: "+command); 
      if(command.startsWith("<list>")){ 
         lobbyPanel.setContactList(command); 
      }

und du bekommst keine Exception mehr, wozu braucht man da eine Idee?

 

[Gelöschtes Mitglied 12486]

das würde zwar die exception verhindern, jedoch ändert es doch nichts daran, dass dort ein null objekt auftaucht wo keins sein sollte oder?

 

[lhein]

Wenn das readLine null liefert, dann solltest Du in Erwägung ziehen, daß die Socket-Verbindung geschlossen wurde bzw. Du das Ende des Streams erreicht hast.
Diesen Fall sollte man auch abfangen.

lr

 

[SlaterB]

nun gut, du hattest vorhin ja wirklich schon
System.out.println(str);
drin, dachte du würdest über irgendwelche Möglichkeiten sprechen,

aber wie siehst nun aus, wirkt diese Abfrage oder nicht?

--------

vielleicht hast du deine Ausgabe nur falsch gelesen,
Ssytem.out und System.err werden gerne mal in falscher Reihenfolge oder gar verzahnt ausgegeben,
wenn du ganz sicher sein willst:
führe eine Exemplarvariable count ein und schreibe

String str = in.readLine();
count++;
System.out.println("Vorausgabe "+count+": "+str);
GameApplet.parseCommand(str);

+

public void parseCommand(String command){
if (command == null) {
System.out.println("command war null bei count = "+count);
// return; kein return, soll ruhig zur Exception kommen
}
try {
........
} catch(NullPointerException e) {
System.out.println("NullPointerException bei count = "+count);
throw e;
}



und dann schaue ganz genau nach welche Nachrichten zu welchem Zeitpunkt kommen,

 

[Gelöschtes Mitglied 12486]

ich habe es so umgesetzt wie du es vorgeschlagen hast, jedoch hilft mir das nicht weiter.

Ich habe nun folgendes in der Console stehen:

Exception in thread "Thread-5" java.lang.NullPointerException
at GameApplet.parseCommand(GameApplet.java:123)
at ClientThread.run(ClientThread.java:35)
out to Server: <startgame>2
Vorausgabe: 10: null
command war null bei count = 10
NullPointerException bei count = 10

Das heisst, dass der inputstream an dieser stelle tatsächlich ein null-objekt einliest oder? dann bleibt die frage, woher kommt dieses? Wenn ein Outputstream ein Null-objekt schreiben soll, dann wirft der doch auch eine Exception oder? d.h es kann ja nicht aus Versehen vom Server kommen...

 

[lhein]

hast Du meinen Kommentar gelesen?

 

[SlaterB]

> jedoch hilft mir da nicht weiter

na hilft doch schon mal entscheidend weiter, das mysteriöse 'null owohl String nicht null war' zu entschärfen,

so, du liest in einer Endlosschleife Zeilen vom Server,
was soll denn passieren wenn der Server nichts mehr zu senden hat?
dann kommt eben null als Ausgabe,

ich weiß jetzt nicht genau, ob das nur dann passiert, wenn die Connection aufgelöst/ geschlossen wird,
wenn die Connection noch offen ist, kann es auch sein, dass so ein read-Aufruf wartet, bis wieder Daten da sind?
kannst du alles testen

 

[lhein]

kleiner Auszug aus der API Doku:

public String readLine()
throws IOException

Read a line of text. A line is considered to be terminated by any one of a line feed ('\n'), a carriage return ('\r'), or a carriage return followed immediately by a linefeed.

Returns:
A String containing the contents of the line, not including any line-termination characters, or null if the end of the stream has been reached
Throws:
IOException - If an I/O error occurs

 

[Gelöschtes Mitglied 12486]

*edit*


hmm ich teste dann mal noch was... danke für den denkanstoss! mal schauen obs klappt...

 

[Gelöschtes Mitglied 12486]

Das oben beschriebene Problem taucht bei mir 2 mal im Programm auf. Einmal auf der Clientseite, einmal auf der Serverseite. Die Clientseite hab ich nun anscheinend geregelt, da war ein denkfehler drin. Anstatt zu prüfen ob der eingelesene String = null ist habe ich dort immer geprüft ob der BufferedReader = null ist. Was hier quatsch war.

run() des Clients:

public void run() {
		while(true){
			try{
				if(quit == false){
					String str = in.readLine();
					if(str != null){
					count++;
					Calendar cal = Calendar.getInstance();
					SimpleDateFormat formater = new SimpleDateFormat("mm:ss:SSS");
					String time = formater.format(cal.getTime());
					System.out.println(time+" Ausgabe: "+count+": "+str);
					GameApplet.parseCommand(str);
					}
				}else{
					break;
				}
			}catch(IOException e){
				System.out.println("*** Error reading Data from Server ***");
				disconnectFromServer();
			}
		}
		System.out.println("*** Error reading Data from Server ***");
	}

Auf der Serverseite habe ich dann geschaut, ob ich dies auch implementiert hatte.

public void run(){
		String line="";
		try {
			while(true) {
					Calendar cal = Calendar.getInstance();
					SimpleDateFormat formater = new SimpleDateFormat("mm:ss:SSS");
					String time = formater.format(cal.getTime());
				line=in.readLine();
				System.out.println(time+" Ausgabe: "+count+": "+line);
				if(line!=null){
					count++;
					pongServer.parseCommand(line,myid);
				}
				Thread.sleep(10);
			}
		} 
		catch (Exception e){
			e.printStackTrace();
			try{ 
				clientSocket.close();
			}
			catch(Exception e2){
				System.err.println(e2);
			}
			System.err.println(e);
		}
	}

Hier steht ja auch diese if-Abfrage. Dennoch bekomme ich eine NullPointerException.