Mehrfacherfassung in Swing GUI

[didi577]

Hallo,

ich baue mir gerade eineSwing Oberfläche auf der Daten erfasst werden können. Es müssen ein paar Mindestangaben (jcombobox) erfasst sein damit gespeichert werden kann. Das funktioniert auch beim ersten Datensatz. Sobald ich den zweiten, dritten etc. Datensatz erfasse schlägt die Mindestprüfung (methode) nicht mehr zu und es können leere Datensätze gespeichert werden.

Wo kann der Fehler liegen?

 

[Robat]

Poste bitte deinen Code in [code=Java]...[/code] Tags .. hier will keiner raten.
Ich verweise auch gerne auf Wie stellt man Fragen richtig?

 

[JStein52]

schlägt die Mindestprüfung (methode) nicht mehr zu

Lass dir einfach deine Variablen die bei dieser Mindestprüfung beteiligt sind mal ausgeben. Ich denke dann wird man den Fehler schnell sehen

 

[didi577]

es ist jetzt nicht nur so dass leere Datensätze gespeichert werden sondern auch mehrere Datensätze. Ich klicke einmal auf Speichern und habe den Datensatz u.U. 6x in der DB also 6 Zeilen mit den selben Daten.
ohne die minimum Methode passiert das nicht, ich will mit dieser erreichen dass der Speichern Button erst funktioniert wenn die Mindestangaben erfasst sind, hier der Code:

private void minimum() {
     
     if (boxKurs.getSelectedIndex() >0 && boxDozent.getSelectedIndex() >0 && boxOrt.getSelectedIndex() > 0) {
       btnSpeichern.addActionListener(new SpeichernAction());
       
     }
   }

hier der code zum prüfen der Comboboxen:

private class BoxKursAction implements ActionListener {
     
     @Override
     public void actionPerformed(ActionEvent e) {
       
       if (boxKurs.getSelectedIndex() > 0) {
       } minimum();
     }}
   
   private class BoxDozentAction implements ActionListener {
     
     @Override
     public void actionPerformed(ActionEvent e) {
       
       if (boxDozent.getSelectedIndex() > 0) {
       } minimum();
     }}
   
   private class BoxOrtAction implements ActionListener {
     
     @Override
     public void actionPerformed(ActionEvent e) {
       
       if (boxOrt.getSelectedIndex() > 0) {
       } minimum();
     }}

0 der erste Eintrag in der Combobox und der darf nicht ausgewählt sein

 

[Meniskusschaden]

Scheint so, als ob du dem Speichern-Button immer wieder neue ActionListener hinzufügst. Wenn man dann einmal speichern drückt, werden die natürlich auch alle ausgeführt.

 

[didi577]

private void minimum() {
     
     if (boxKurs.getSelectedIndex() >0 && boxDozent.getSelectedIndex() >0 && boxOrt.getSelectedIndex() > 0) {
       btnSpeichern.addActionListener(new SpeichernAction());
   
     }
   }

diese Methode rufe ich im Speichern Button auf. Kann es sein dass ich wie oben beschrieben bei Änderungen in den JComboboxen damit immer den Actionlistener aufrufe?

 

[Meniskusschaden]

diese Methode rufe ich im Speichern Button auf.

Nein, die rufst du nicht im Speichern-Button auf.

Du hast noch drei andere ActionListener. Ich unterstelle mal, dass jeder davon bei genau einer ComboBox als ActionListener hinzugefügt wurde. Dann wird jedes Mal, wenn bei einer ComboBox etwas ausgewählt wurde und die Bedingung zutrifft, minimum() aufgerufen.

minimum() fügt dann dem Speichern-Button eine neue Instanz des ActionListeners SpeichernAction hinzu, und zwar tut sie das jedesmal wenn sie aufgerufen wird und die Bedingungen zutreffen. Wenn dann der Speichern-Button betätigt wird, feuert jede der SpeichernAction-Instanzen.

Also angenommen in zwei ComboBoxen ist bereits je ein Element selektiert und du selektierst nun auch eins in der dritten. Dann wird minimum() erstmals einen SpeichernAction-ActionListener zum Speichern-Button hinzufügen. Ab jetzt passiert also überhaupt etwas, wenn man Speichern drückt.

Selektierst du dann in irgendeiner ComboBox ein anderes Element, so wird minimum() wieder einen SpeichernAction-ActionListener zum Speichern-Button hinzufügen. Nun gibt es also zwei und wenn man Speichern drückt, werden beide das tun, was dort eben programmiert ist - also vermutlich einen Satz in die DB schreiben. Mit jedem Auswählen eines ComboBox-Elements kommt dann ein weiterer Listener hinzu.

Kann es sein dass ich wie oben beschrieben bei Änderungen in den JComboboxen damit immer den Actionlistener aufrufe?

Ja, dessen actionPerformed-Methode wird jedes Mal vom Swing-Framework aufgerufen, wenn bei der zugehörigen Komponenten das entsprechende Ereignis eintritt. Für SpeichernAction passiert das beim Drücken des Speichern-Buttons und für die drei anderen beim Selektieren eines Elements in der zugehörigen ComboBox.

 

[didi577]

ok Danke für deine Erklärungen zu meinem Problem. Ich dachte mir sowas. Habe alles zurückgedreht und suche mir eine andere Möglichkeit Mindestangaben zu prüfen. Die addActiolistener habe ich aus den Button wieder raus genommen so dass nur eine Instanz beim Programmstart erzeugt wird.

 

[didi577]

Hallo Meniskusschaden ;-),

habe jetzt umgebaut. Ohne actionListener zu nutzen. Ich arbeite jetzt mit setEnabled. Klappt fast gut.

Ein Speichernbutton soll erst aktiv werden wenn drei Comboboxen einen Wert größer null haben. Das klappt, der Button wird dann aktiv. Wenn ich aber vor dem Speichern eine der Comboboxen wieder auf 0 setze wir der Button nicht inaktiv. Das müsste er per Definition aber (nach meinem Verständnis).

die minimum Methode:

private void minimum() {
     
     if (boxKurs.getSelectedIndex() > 0 & boxDozent.getSelectedIndex() > 0 & boxOrt.getSelectedIndex() > 0) {
       btnSpeichern.setEnabled(true);
   
     }
   }

ich rufe minimum() beim Speichern und beim Neu Button auf
die ActionListener der Comboboxen:

private class BoxKursAction implements ActionListener {
     
     @Override
     public void actionPerformed(ActionEvent e) {
       
       if (boxKurs.getSelectedIndex() == 0) {
       } minimum();
       
     }}
   
   private class BoxDozentAction implements ActionListener {
     
     @Override
     public void actionPerformed(ActionEvent e) {
       
       if (boxDozent.getSelectedIndex() == 0) {
       } minimum();
     }}
   
   private class BoxOrtAction implements ActionListener {
     
     @Override
     public void actionPerformed(ActionEvent e) {
       
       if (boxOrt.getSelectedIndex() == 0) {
       } minimum();
     }}

was muss ich ändern?

 

[Meniskusschaden]

Du rufst minimum() nur auf, wenn getSelectedIndex gleich 0 ist, minimum() tut jedoch nur dann etwas, wenn getSelectedIndex größer 0 ist. Das kann doch niemals gleichzeitig zutreffen.

 

[Joose]

Falsch @Meniskusschaden minimum wird immer aufgerufen egal welcher Index ausgewählt wurde.
Die Einrückung stimmt einfach nicht.

 

[Meniskusschaden]

Falsch @Meniskusschaden minimum wird immer aufgerufen egal welcher Index ausgewählt wurde.
Die Einrückung stimmt einfach nicht.

Stimmt, habe ich übersehen.

Wenn ich aber vor dem Speichern eine der Comboboxen wieder auf 0 setze wir der Button nicht inaktiv. Das müsste er per Definition aber (nach meinem Verständnis).

Das passiert aber nicht von selbst. Du musst dafür schon irgendwo btnSpeichern.setEnabled(false) aufrufen.

 

[didi577]

...genau das habe ich jetzt auch getan jetzt funktioniert es. vielen Dank für eure Tipps