ItemListener mit NullPointerException

[MrSnake]

Ich habe meinem Programm eine ComboBox (cboSavedTeams) hinzugefügt deren Elemente aus einer separaten Datei geladen werden - dies funktioniert und die Elemente werden korrekt angezeigt.

Jetzt versuche ich der ComboBox einen ItemListener hinzuzufügen um nach der Auswahl eines Elementes diverse Anzeigen zu verändern. Das Problem ist aber, an der Stelle wo ich der ComboBox den ItemListener hinzufüge - fliege ich direkt mit NullPointerException raus...

Hier der Code in kurzform:

public class GemsOfWar extends JFrame implements ItemListener {

    public GemsOfWar() {
        //Teamliste mit Einheit 1-4 füllen
        for(int i=0; i<4; i++) {
            lmSelectedTeam.addElement("Einheit " + (i+1));
        }
        cboSavedTeams.addItemListener(this);
        initComponents();
        setLocationRelativeTo(null);
        fillTroopList();  //Einheitentabelle beim Start des Programms füllen
        fillSavedTeamComboBox();  //ComboBox mit gespeicherten Teams füllen  
    }
}

@Override
public void itemStateChanged(ItemEvent e) {
    if(e.getSource() == cboSavedTeams) {
        System.out.println("Aktion");
    }
}

Laut Debugger komme ich durch die kurze for-Schleife die mein ListModel füllt und in der Zeile cboSavedTeams.addItemListener(this); werde ich rausgeschmissen. Aus der DebuggerConsole werde ich leider auch nicht schlau - die Zeile die den Fehler verursacht kann ich noch erkennen, der Rest bezieht sich meines Erachtens auf andere Dateien die ich nicht geschrieben habe...

Hat jemand eine Idee woran das liegt?

 

[mrBrown]

Wird denn cboSavedTeams irgendwo initialisiert? zumindest in deinem Beispiel nicht, also ist es da noch null.

Ansonsten ist es immer schlecht, 'this' aus dem Constructor weiterzugeben, 'this' ist da schließlich noch nicht gänzlich initialisiert.

 

[MrSnake]

Na klar... Mensch bin ich blöd!

Ich habe cboSavedTeams.addItemListener(this); jetzt ans Ende vom Konstruktor gestellt - die Elemente werden ja erst bei initComponents initialisiert^^

Ich sollte langsam mal eine Pause machen wenn ich selbst das nicht mehr alleine sehe

Danke mrBrown - Du hast mir den richtigen Weg gezeigt!

 

[mrBrown]

Der Aufruf mir 'this' sollte am besten gar nicht im Constructor stattfinden, sondern nach Rückkehr aus diesem aufgerufen werden