Laden und leeren von Comboboxen

[didi577]

Hallo,

ich habe in einer Auswertung 4 Comboboxen:
Box1
Box2
Box3
Box4
Sie sind voneinander abhängig. Wenn ich Box1 auswähle laden sich die anderen Boxen nachfolgend mit Werten aus einer DB. Wenn Box4 gefüllt ist startet eine Auswertug Methode.
Wenn ich die Auswahl in Box1 ändere sollen sich die anderen Boxen entsprechend anpassen. Damit die neuen Daten der Boxen 2 - 4 nicht "addiert" werden lösche ich vor dem Laden die Boxen 2 - 4 mit "box.removeAllItems();". Das klappt wunderbar.
Seitdem die Auswertungsmethode nach Box4 aufgerufen wird, und ich dann Box1 ändere, kommt es bei der Box2 zu einer NPE, alternativ auch bei den folgenden Boxen. Ich vermute es liegt an der remove Methode. Das war durch auskommentieren erkennbar.

Gibt es Alternativen zu "box.removeAllItems();" ? Kann ich das besser lösen?

 

[Robat]

Gibt es Alternativen zu "box.removeAllItems();"

Ich wüsste nicht warum du hier eine andere Methode benutzen solltest.

Lass uns doch mal an deinem Code teilhaben - den Fehler kann man bestimmt finden
(zusätzlich bitte gleich den Stacktrace mit posten)

Gruß
Robert

 

[didi577]

das sind die Listener aller 4 Boxen

private class BoxDozentAusAction implements ActionListener {
       
       @Override
       public void actionPerformed(ActionEvent e) {
   
           if (boxDozent.getSelectedIndex()>=0) {
               boxKurs.setEnabled(true);
               boxKurs.removeAllItems();
               boxOrt.removeAllItems(); //leert beim Wechsel des Dozenten die Combobox Kurs
               boxDatum.removeAllItems();
               ladenBoxKurs();
           }
           
       }
   }
private class BoxKursAusAction implements ActionListener {
       
       @Override
       public void actionPerformed(ActionEvent e) {
           
           if (boxKurs.getSelectedIndex()>=0) {
               boxOrt.setEnabled(true);
               boxOrt.removeAllItems();
               boxDatum.removeAllItems();
               ladenBoxOrt();
           }    
       }
   }
   
   private class BoxOrtAusAction implements ActionListener {
       
       @Override
       public void actionPerformed(ActionEvent e) {
           
           if (boxOrt.getSelectedIndex()>=0) {
               boxDatum.setEnabled(true);
               boxDatum.removeAllItems();
               ladenBoxDatum();
           }
           
       }
   }
   
   private class BoxDatumAusAction implements ActionListener {
       
       @Override
       public void actionPerformed(ActionEvent e) {
           
           ladenAuswertung();
       }
   }

}

hier ist dann "ladenAuswertung()" nach Box4:

public void ladenAuswertung() {
       
       Connection con = null;
       try {
           String dozent = boxDozentM.getSelectedItem().toString();
           String kurs = boxKursM.getSelectedItem().toString();
           String ort = boxOrtM.getSelectedItem().toString();
           String datum = boxDatumM.getSelectedItem().toString();
           con = DriverManager.getConnection("jdbc:mysql://localhost:3306/schulaus?user=root&password=#Patrick1");
           PreparedStatement pst = con.prepareStatement("select land, a1, a2, a3, a4, b1, b2, b3, c1, c1_txt, c2, c3, c4, c5, d1, d2, d3, d4_txt, d5_txt from fragen where dozent=? AND kurs=? AND ort=? AND datum=?");
           pst.setString(1, dozent);
           pst.setString(2, kurs);
           pst.setString(3, ort);
           pst.setString(4, datum);
           ResultSet rs1 = pst.executeQuery();
           while (rs1.next()) {
               txtTest.setText(rs1.getString(10));
               System.out.println(rs1.getString(10));
           }
       
       }    catch (SQLException e) {
               System.out.println(e);
               e.printStackTrace();
           }       
   }

 

[Robat]

Okay nur noch mal um sicher zu gehen, dass ich dein Problem richtig verstanden habe:

Je nach dem, was bspw. in Combobox1 ausgewählt ist werden die anderen Boxen neu geladen. Also wenn ich in der 1. Box das 2. Item auswähle werden in der 4. Box nur die letzten beiden Items angezeigt.

War nämlich kurz verwirrt, dass du da einen ActionListener verwendest

 

[didi577]

Ich möchte Daten einer DB auswerten. Die 4 Boxen sind die ersten 4 DB Felder je Datensatz. Es gibt noch 4 Methoden zum Laden der Boxen.

 

[didi577]

damit lade ich die erste Box:

public void ladenBoxDozent() {

       Connection con = null;
       try {
           con = DriverManager.getConnection("jdbc:mysql://localhost:3306/schulaus?user=root&password=#Patrick1");
           PreparedStatement pst = con.prepareStatement("select DISTINCT dozent from fragen ORDER BY dozent");
           ResultSet rs = pst.executeQuery();
           while(rs.next()) {
               boxDozentM.addElement(rs.getString(1));   
           }
                   
           
           } catch (SQLException e) {
               System.out.println("ERROR" + e);
               e.printStackTrace();
           }
       }

 

[Robat]

Ich glaub ein ItemListener ist eher etwas für dich.
Die Daten einer Combobox können einfach in abhängigkeit einer anderen geändert werden.
Hier mal ein kleines Beispiel:

~quick'n'dirty~

    private JComboBox jcb1;
    private JComboBox jcb2;
   
    private String[] obj1 = { "StringSet1", "StringSet2" };
    private String[] obj2 = { "Foo", "Bar"} ;
    public ComboTest()
    {
       jcb1 = new JComboBox(new DefaultComboBoxModel(obj1));
       jcb2 = new JComboBox(new DefaultComboBoxModel(obj1));
       
       jcb1.addItemListener(new ItemListener() {
           @Override
           public void itemStateChanged(ItemEvent e) {
               if(e.getStateChange() == ItemEvent.SELECTED)
               {
                   JComboBox c = (JComboBox) e.getSource();
                   if(c.getSelectedItem().equals("StringSet1"))
                   {
                       jcb2.setModel(new DefaultComboBoxModel(obj1));
                   }
                   else
                   {
                       jcb2.setModel(new DefaultComboBoxModel(obj2));
                   }
               }
           }
       });
       
       add(jcb1);
       add(jcb2);
    }

 

[didi577]

ok, schonmal Danke für deine Arbeit!!!
ich habe parallel mal vor die ladenAuswertung() mal eine if Bedingung gebaut:

private class BoxDatumAusAction implements ActionListener {
       
       @Override
       public void actionPerformed(ActionEvent e) {
           
           if (boxDozent.getSelectedIndex()>=0 & boxKurs.getSelectedIndex()>=0 & boxOrt.getSelectedIndex()>=0 & boxDatum.getSelectedIndex()>=0) {
           
           ladenAuswertung();
           }
       }
   }

das scheint es auch zu tun, so rufter die Methode ert nach erneutem Laden auf

ist das auch eine Alternative?

 

[didi577]

deinen Code werde ich mir morgen in Ruhe ansehen und probieren

ich habe in den Boxen zwischen 10 und 50 Werte

ich werden probieren was hier am efektivsten ist