Swing JOptionPane Verwirrung

[lobnews]

Guten Tag.
Ich bin schon seit ein paar Tagen ein kleines Programm für einen Bekannten zu schreiben. Dieses Programm nutzt JOptionPane, da mir die JFrames bis jetzt zu kompliziert sind. Dieses Programm organisiert einfach nur Turniere. Bei der Programmierung bin ich aber gestern auf ein Problem gestossen. Einen NullPointerException, um genau zu sein java.lang.NullPointerException: null.
Dieser wird mir in der ersten Codezeile angezeigt.

JOptionPane.showMessageDialog(null,"Die nächsten Spiele sind:" + "/n"
                    + s0.getMan1().getSpieler() + " gegen " + s0.getMan2().getSpieler() + "/n"
                    + s1.getMan1().getSpieler() + " gegen " + s1.getMan2().getSpieler() + "/n"
                    + s2.getMan1().getSpieler() + " gegen " + s2.getMan2().getSpieler(),
                    "", JOptionPane.PLAIN_MESSAGE);

Erklärung:
s0, s1 und s2 sind Spiele-Objekte, die ich selber geschrieben habe.
Sie besitzen jeweils 2 Mannschafts-Objekte, die ich mir mit getMan1() oder getMan2() wieder geben lassen kann.
Jedes dieser Mannschafts-Objekte, die ich auch selber geschrieben habe, besitz einen String Spieler, den ich mir mit getSpieler() wieder geben lassen kann.

Vorhin habe ich eine andere Variante ausprobiert:

JOptionPane.showMessageDialog(null, "Das nächste Spiel ist: " + "\n"
                    + s0.getMan1().getSpieler() + " gegen " + s0.getMan2().getSpieler(),
                    "", JOptionPane.PLAIN_MESSAGE);

Erklärung:
Die Legende ist die Selbe wie bei dem ersten Codeschnippsel.

Dabei bekomme ich keinen Fehler. Die Fehler tauchen aber erst auf, wenn ich das Programm audführe und nicht in der Programmierumgebung. Dies hat mich ein wenig Verwirrt. Aber ich würde gerne die erste Variante verwenden. Also woran liegt der NullPointerException und wie würde ich es hinbekommen, dass die erste Variante funktioniert?

Software die ich benutze:
Windows 7 64-Bit
IntelliJ zum eigentlichen Programmieren und
BlueJ zum builden der jar und zum beheben der letzten Fehler.
Und ich weiß es geht anders, aber diese Variante funktioniert.
Ich hoffe man kann mir helfen.

 

[Timothy Truckle]

Bei der Programmierung bin ich aber gestern auf ein Problem gestossen. Einen NullPointerException, um genau zu sein java.lang.NullPointerException: null.

In der Fehlermeldung stehen auch Zeilennummern. Welche deiner Zeilen ist den die oberste (nicht die höchste), die Aufgeführt ist?

bye
TT

 

[lobnews]

ich meinte damit die erste Codezeile den ich kopiert habe also:

JOptionPane.showMessageDialog(null,"Die nächsten Spiele sind:" + "/n"

 

[Camino]

Zeig doch mal die ersten paar Zeilen deiner Fehlermeldung.

 

[lobnews]

Die habe ich schon geschrieben. BlueJ, welches ich auch zum testen benutze ist etwas sparsam mit den Fehlermeldungen. java.lang.NullPointerException: null

 

[Camino]

Das ist wirklich nicht sehr aussagekräftig. Führ dein Programm doch mal auf der Konsole aus und schau, was für Fehlermeldungen da kommen.

 

[lobnews]

Exception in thread "main" java.lang.NullPointerException
at Turniermodule.main(Turniermodule.java:35)
Wenn dir das weiter hilft. Zeile 35 in Turniermodule ist die schon weiter oben genannte Zeile.

 

[X5-599]

Es kann eigentlich nur sein das:
s0, s1, s2 sind "null" (Eins oder mehrere davon).

getMan1() oder getMan2() liefern wenigstens einmal "null"

Ansonsten wird auf nichts zugegriffen was eine NullPointerException auslösen würde...
Ich würd mir vor aufruf des OptionPane's mal alle diese Objekte per System.out.println() ausgeben lassen. Dann weisst du bescheid welche davon "null" sind.

 

[lobnews]

Davon kann keiner null sein, weil ich die jeweils neu in einer for-Schleife definiere. Die Liste, aus der ich die definiere, hat so viele Argumente, wie sie braucht um noch adressiert zu werden, bevor ein nicht vorhandenes Element erreicht. Ausserdem funktionirt die 2. Variante ja also könnte es daran eher nicht liegen (ist nur eine Vermutung, kann aber auch falsch sein). Sollte mehr Code von nöten sein bitte bescheid sagen. Ausserdem wird kein JOptionPane angezeigt, obwohl ich mehr als 3 Spiele habe.

 

[X5-599]

Wenn das mit s0 funktioniert muss es an s1 oder s2 liegen. Bzw deren getMan1 getMan2 Methoden.

Wie sieht denn die Schleife aus in der du die Objekte erzeugst?

 

[Timothy Truckle]

Ausserdem funktionirt die 2. Variante

Na dann baue doch die 2. Variante Stück für Stück zur ersten aus: immer ein Element aus Variante 1 hinzufügen und ausprobieren.
Wenn die NPE wieder kommt weist Du, dass das letzte hinzugefügte Element die Ursache ist.

bye
TT

 

[lobnews]

Wie sieht denn die Schleife aus in der du die Objekte erzeugst?

for (int i=0; i < spiele.size()-2; i++){
            Spiele s0 = spiele.get(i);
            Spiele s1 = spiele.get(i++);
            Spiele s2 = spiele.get(i+2);
            JOptionPane.showMessageDialog(null, "Die nächsten Spiele sind:" + "/n"
                    + s0.getMan1().getSpieler() + " gegen " + s0.getMan2().getSpieler() + "/n"
                    + s1.getMan1().getSpieler() + " gegen " + s1.getMan2().getSpieler() + "/n"
                    + s2.getMan1().getSpieler() + " gegen " + s2.getMan2().getSpieler(),
                    "", JOptionPane.PLAIN_MESSAGE);
            String ergebnis1 = JOptionPane.showInputDialog( "Bitte die Tore für Mannschaft " + s0.getMan1().getSpieler() + " eingeben.");
            String ergebnis2 = JOptionPane.showInputDialog( "Bitte die Tore für Mannschaft " + s0.getMan2().getSpieler() + " eingeben.");
            int e1 = Integer.parseInt(ergebnis1);
            int e2 = Integer.parseInt(ergebnis2);
            if (e1>e2){
                mannschaften.get(s0.getID1()).setPunkte(3);
            }
            else {
                mannschaften.get(s0.getID2()).setPunkte(3);
            }
            mannschaften.get(s0.getID1()).setTore(e1-e2);
            mannschaften.get(s0.getID2()).setTore(e2-e1);
            s0.setErgebnis(e1-e2);
        }

Erklärung:
spiele ist eine Integer Spiele Map, die ich mir generieren lasse um alle Spiele zu haben.
spiele wird nach dem Füllen mit den Spielen um 2 leere Spiele ergänzt, die dafür sorgen, dass die for Schleife halt kein nichtadressierbares Spiel adressiert.
s0.getID1 und s0.getID2 gibt mir die passende MannschaftsID zurück, unter der ich die Mannschaft in der Integer Mannschaften Map (mannschaften) finde.
die Methode setTore habe ich geändert, das der Wert dazuaddiert wird, genauso bei setTore.
setErgebnis für ein Spiel speichert das Ergebnis, welches für die Rangfolge wichtig sein kann.
Wenn fragen zum Code sind bitte bescheid sagen. Ich war sehr sparsam mit Kommentaren.

Na dann baue doch die 2. Variante Stück für Stück zur ersten aus: immer ein Element aus Variante 1 hinzufügen und ausprobieren.
Wenn die NPE wieder kommt weist Du, dass das letzte hinzugefügte Element die Ursache ist.

Da gibt es ein Problem. Ich führe die for-Schleife mit z.B. 4 Mannschaften also 6 Spielen in der spiele Map und die 2 leeren aus. Dabei wirft der schon beim ersten Durchlauf den Fehler. Ich werde natürlich deinen Vorschlag einmal ausprobieren, aber dies kann ich erst morgen machen.

 

[bERt0r]

Sehr schön, das sind typische anfängerfehler: Die Zählervariable in einer For schleife DARF nicht verändert werden.

Angenommen du hast 3 Objekte in der Liste, erster Schleifendurchlauf:

Spiele s0 = spiele.get(i);          //i=0  zugriff auf 0. Element
            Spiele s1 = spiele.get(i++);   //i=1  zugriff auf 1. Element
            Spiele s2 = spiele.get(i+2);   //i=1  zugriff auf 3. Element

2. Schleifendurchlauf:

Spiele s0 = spiele.get(i);         //i=2  zugriff auf 2. Element
            Spiele s1 = spiele.get(i++);  //i=3  zugriff auf 3. Element
            Spiele s2 = spiele.get(i+2);  //i=3  zugriff auf 5. Element

Da die Liste nur 3 Einträge (das 0., das 1. und das 2.) hat, wird hier null zurückgeliefert und deine Exception fliegt. Wenn du i++ schreibst, erhöht das die Variable selber um 1, das wird gespeichert und wirkt sich auf deine Zählvariable aus.

 

[lobnews]

Ah ok. So ein dummer Fehler. Darauf bin ich gar nicht gekommen. Aber müsste das Programm denn bei dem Vorhin beschriebenen Fall, also 4 Teams also 6 Spiele + 2 null Spiele macht 8 Einträge in spiele.
Also müsste einfach nur theoretisch die Schleife mindestens einmal durchlaufen, dies tut sie aber nicht einmal. Oder irre ich mich in meiner Vermutung? Dann bitte ich um korrektur.