NullPointerException

[mavinatic]

Hallo Community,

ich habe eine Frage zum Thema "NullpointerException":

public String[] findOnlineUser() {
    	Query query = entityManager.createQuery("SELECT x FROM UserInformation x WHERE x.online='"+true+"'");
    	List<UserInformation> userList = (List<UserInformation>) query.getResultList();
    	String[] users = new String[userList.size()];
    	int i=0;
    	for(UserInformation user:userList) {
    		users[i] = user.getId();
    		i++;
    	}
    	
    	if(userList.size()==0)
    		return new String[] {};
    	
		return users;
    }

Das ist meine Methode...und ich möchte umgehen, dass die Methode "null" zurückgibt. Jedoch tut sie das im Moment?! obwohl ich ein neues leeres Array erzeuge.

Hat jemand eine Idee wieso?

 

[Gast2]

Die Methode kann nicht null zurückgeben, entweder gibt sie

new String[userList.size()]

oder

new String[] {}

zurück, das ist beides nicht null.
Verwechselst du das vielleicht und es fliegt in Wirklichkeit ne NullPointerException in der Methode?

Die beiden Zeilen 11 und 12 solltest du übrigens etwas weiter oben ansetzen, direkt nachdem du die userList geladen hast.

 

[mavinatic]

Ja dieser Ansicht bin ich auch, deswegen...frage ich hier nochmal nach. Das ganze geht über eine Schnittstelle und über einen Webservice, deswegen wird da die NPE entstehen, welche ich auf der anderen Seite bekomme. :-/

 

[pl4gu33]

kannst du die Exception mal posten ?

 

[Landei]

Im Stacktrace der Exception steht die Zeile, wo sie "fliegt".

 

[mavinatic]

Auf Serverseite fliegt keine Exception.....nur auf Clientseite...weil ich durch ein Array laufe, welches Null ist, aber laut meiner Methode "findOnlineUsers" dürfte das array nicht null sein.

 

[Gast2]

Poste doch mal den Stacktrace.

 

[mavinatic]

Exception in thread "main" java.lang.NullPointerException
	at com.mycompany.openflow.test.ws.UserServiceTest.testFindAllOnlineUsers(UserServiceTest.java:55)
	at com.mycompany.openflow.test.ws.UserServiceTest.<init>(UserServiceTest.java:23)
	at com.mycompany.openflow.test.ws.UserServiceTest.main(UserServiceTest.java:77)

 

[Marcinek]

Was steht in

UserServiceTest.java in Zeile 55?

 

[pl4gu33]

sind das denn wirklich die gleichen Methoden weil dort steht "testFindAllOnlineUsers" / oben steht findOnlineUser und was genau steht in der Zeile

 

[mavinatic]

Es sind nicht die gleichen Methoden würdet ihr lesen was ich geschrieben habe, dann wüsstet ihr das es Clientseitig ist. (Ist nicht böse gemeint!)
Nochmal der Erklärungsversuch:

Auf meinem Server befindet sich eine EJB, welche die Methode findOnlineUsers beinhaltet, genau wie im 1. Beitrag beschrieben.

Nun greife ich über einen Webservice auf die EJB zu und erhalte eine Nullpointer-Exception, jedoch habe ich es vermieden explizit "NULL" zurückzugeben.

SourceCode meines Testprogramms:

Zeile 53:UserService userService = service.getPort(UserService.class);
         Zeile 54:String[] onlineUsers = userService.findOnlineUser("myId");
         Zeile 55:	for(String x:onlineUsers) {
	 Zeile 56:		System.out.println(x);
	 Zeile 57:	}

 

[tagedieb]

Dann bau mal ein paar System.out.println(..) ein oder Debug durch den Code, dann wirst du schon finden wo 'null' zurückgegeben wird und was auf der Serverseite passiert.

Vielleicht ist der Fehler ja auch im Webservice und nicht im EJB

 

[FArt]

Es sind nicht die gleichen Methoden würdet ihr lesen was ich geschrieben habe, dann wüsstet ihr das es Clientseitig ist. (Ist nicht böse gemeint!)

Riskierst eine ganz schön große Lippe, dafür dass du nicht in der Lage bist einen einfachen NPE zu sehen. (Ist nicht böse gemeint!)

Deine Information ist recht dürftig. Erst postest du nur diese langweilige Methode, dann ist es ein Webservice und plötzlich haben wir EJBs. Genug Möglichkeiten bei der Delegation und beim Mapping etwas zu verschlampen.

Was deinen Code angeht: EikeB hat es schon gesagt.
Über Logging und/oder Debugging sollte leicht zu finden sein, wo die Referenz "verloren geht".

P.S.: interessante for Schleife:

        int i=0;
        for(UserInformation user:userList) {
            users[i] = user.getId();
            i++;
        }

 

[timbeau]

Die Methode oben erwartet aber keinen Parameter ("myID"). Hast du da was überladen?

 

[faetzminator]

Und du bist dir sicher, dass nicht einfach die Query serverseitig eine Exception schmeisst?
Ganz abgesehen davon, der von dir gepostete Code kann auch einfacher geschrieben werden:

public String[] findOnlineUser() {
    Query query = entityManager.createQuery("SELECT x FROM UserInformation x WHERE x.online='true'");
    List<String> result = new ArrayList<String>();
    for(UserInformation user : (List<UserInformation>) query.getResultList()) {
        result.add(user.getId());
    }
    return result.toArray(new String[result.size()]);
}

(Warum man auch immer ein [c]String[][/c] statt [c]List<String>[/c] benötigt...)

 

[mavinatic]

@FArt:
1) Bin ich mir ziemlich sicher, dass keine Exception fliegt, weil ich debugged habe, sowie im Log nachgeschaut habe. Beim debuggen springt er in die Zeile und gibt das Array zurück.

Riskierst eine ganz schön große Lippe, dafür dass du nicht in der Lage bist einen einfachen NPE zu sehen. (Ist nicht böse gemeint!)

Die simple NPE habe ich gelöst, wenn du mal gelesen hättest (ja mal wieder), dann weißt du das es sich letztendlich mehr um den Rückgabewert handelt anstatt um die NPE. (Fehler meinerseits der Titel des Threads hätte anders lauten können)

@faetzminator
Wenn du der Meinung bist, dass deine Schreibweise besser ist, dann nehme ich das so an. Ich habe die Erfahrung gemacht, dass ein Array schneller arbeitet als eine ArrayList/List.

 

[faetzminator]

Wenn du der Meinung bist, dass deine Schreibweise besser ist, dann nehme ich das so an. Ich habe die Erfahrung gemacht, dass ein Array schneller arbeitet als eine ArrayList/List.

Natürlich sind "native" Types (primitive und Arrays) schneller und günstiger als Objekte. Das spielt aber im Normalfall überhaupt keine Rolle. 1. 99% allen Codes ist nicht zeitkritisch 2. solange Programmierer irgendwelche Laufzeiten von [c]O(n^2)[/c] statt [c]O(n * log(n))[/c] oder [c]O(n)[/c] entwickeln, spielts sowieso keine Rolle...

 

[tagedieb]

Würdest du lesen was wir geschrieben haben, wüsstest du dass der Fehler nicht umbedingt im EJB zu suchen ist, sondern auch im WebService liegen könnte ;-)

Also schau dir mal den Response vom Webservice an, denn da wird ja null zurückgegeben und arbeite dich von da an Rückwärts bis zum EJB.

 

[FArt]

@FArt:
1) Bin ich mir ziemlich sicher, dass keine Exception fliegt, weil ich debugged habe, sowie im Log nachgeschaut habe. Beim debuggen springt er in die Zeile und gibt das Array zurück.

Das habe ich nicht behauptet (lies genau). Ich habe EikeB recht gegeben, dass DORT nicht null zurückgelierfert wird. Im Client (hast du selber gesagt) fliegt eine NPE weil null ankommt. Die Ursache dafür zu finden sollte mit Logging und Debugging wirklich einfach sein: sie liegt dazwischen!

Stichworte noch einmal: Delegation, Mapping (<-). Wie schon angemerkt: da passiert bestimmt noch ein wenig. Alleine schon, weil die Methodensignaturen nicht zusammenpassen.

Beliebtes Antipattern: eine (Runtime)Exception wird gefangen, nicht geloggt und null zurückgeliefert...