Warum das Prog mehrmals das gleiche macht wegen ItemListener

[GAST]

Hallo,
ich habe folgendes Problem, ich habe zwei JcomboBoxen in denen Zeiträume stehen, nun wähle ich in den Boxen meinen Zeitraum und diesen fange ich mit der folgenden Methode ab, nun zur Vereinfachung wähle ich nur bei einem JcomboBox den Zeitraum bei dem anderem ist dieser fest gewählt:

public String jComboBox3Selection()
    {
        jComboBox3.addItemListener(new ItemListener()
        {
            public void itemStateChanged(ItemEvent e)
            {
                String str3 = (String)jComboBox3.getSelectedItem();
                //System.out.println(str3); //Hier funktioniert es noch
                analyser.this.jComboBox3Wert = str3;
                
                if(e.getSource() == jComboBox3 && e.getStateChange() != ItemEvent.SELECTED )
                {                
                    System.out.println(anzahlFailDesZeitraumes());
                    //anzahlFailDesZeitraumes();
                    jLabel13.setText(anzahlFailDesZeitraumes() + " Stück");
                }
            }
        });
        return jComboBox3Wert;
    }

so wenn ich das Prog starte, öffnet dieser mir mein GUI und ich wähle bei der JcomboBox3 den Zeitraum, nun werden sofort mit System.out.println, die Ergebnisse der Methode rausgegeben, und in jLabel die Erbebnisse der Methode dargestellt!
Das Problem ist, dass die Methode nicht einmal durchlaufen wird, sondern zuerst einmal, wenn ich dann anderen Zeitraum wähle, dann drei mal, wenn ich noch einen anderen Zeitraum wähle, dann 9 mal usw.!

Was kann ich dagegen tuen, damit er nur einmal die Methode durchläuft und das Ergebnis in jLabel ausgibt, weil das Programm richtig lange Laufzeit hat!

Wenn es noch nicht ganz verständlich ist, einfach mailen, ich versuche die sache klarer zu beschreiben!
Gruss

 

[Niki]

Wie oft rufst du jComboBox3Selection auf? jedesmal wenn du die Methode aufrufst wird ein zusätzlicher Listener registriert! Den Listener solltest du genau einmal registrieren, am besten im Konstruktor!

 

[Guest]

Wie oft rufst du jComboBox3Selection auf? jedesmal wenn du die Methode aufrufst wird ein zusätzlicher Listener registriert! Den Listener solltest du genau einmal registrieren, am besten im Konstruktor!

Hi,
ich rufe es eigentlich nur einmal auf und zwar, wie schon ober angegeben als Methode!
Diese speichert den Selektierten Wert des JcomboBox3 im jComboBox3Wert String, damit ich diesen weiter verwenden kann!

Wenn ich diesen im Konstruktor registriere, dann kann bestimmt nicht mehr den Wert (jComboBox3Wert) zwischenspeichert, oder?

Gruss

 

[Niki]

Bist du dir ganz sicher, ich bezweifle das die Methode nur einmal aufgerufen wird. Du möchtest ja den Wert der ComboBox über diese Methode ermitteln, oder? Probier mal das combobox.addItemListener gleich nach dem Instanzieren der ComboBox auszulagern. Das Attribut jCombobox3Wert kannst du dir ganz sparen. Mach lieber

public String jComboBox3Selection() {
  return (String)comboBoxe.getSelectedItem();
}

 

[Guest]

Jo Danke Niki, es hat alles geklappt, danke nochmals!!!