Swing JComboBox mit Datenbank füllen

[schina02]

Guten Abend die Herren,

ich beschäftige mich seit Neustem mit Datenbanken (MySQL) und wollte die doch mal mit einem schönen Programm bearbeiten

Jetzt habe ich folgendes Problem:

Ich möchte eine ComboBox erstellen, die nur die Daten aus der bestimmten Tabelle enthalten. Also man einfach nur anklickt, anstatt das ganze Wort einzutippen.

Die Suche hier und bei Google habe ich schon benutzt, aber es hilft mir nicht wirklich.

Status quo:

Die Combobox ist erstellt.

private JComboBox comCombobox = new JComboBox();

und eine Verbindung zur Datenbank ist hergestellt.

            final String URL = "jdbc:mysql://localhost/bibliothek";
            final String USER = "root";
            final String PASSWORD = sPass;
            
            mds.setURL(URL);
            mds.setUser(USER);
            mds.setPassword(PASSWORD);
            con = mds.getConnection(USER, PASSWORD);
            
            stm = con.createStatement(ResultSet.TYPE_SCROLL_INSENSITIVE, ResultSet.CONCUR_READ_ONLY);
            sql = "select vorname, name from student";
            stm.executeQuery(sql);
            rs = stm.getResultSet();

Kommen nun alle Befehle in den ActionPerformed der Combobox und welche sind das?

Der Sinn ist ja, die Werte aus der Tabelle rauszuholen und für jede Zeile in der Tabelle eine neue Zeile in der Combobox anzulegen.

 

[vanny]

Stichwort ComboBoxModel/ListModel

 

[turtle]

Das Stichwort von vanny ist schon voll treffend.

Ich möchte das ergänzen und sagen, dass Du nachdem Du den ResultSet aus der DB bekommen hast, Du über dieses iterieren und die Werte Dir zum Beispiel in einem Array merkst. Dann kannst Du beispielsweise
über

DefaultComboBoxModel(Object[] items)

ein DefaultComboboxModel erzeugen und als Model der Combobox zuweisen, also ungefähr so wie im folgenden Pseudocode:

JComboBox<String> cbxName = ...;
String[] werte = new String[] {"Eins", "Zwei"}; // Solch ein Array solltest über das ResultSet erzeugen
ComboBoxModel<String> cbxModel = new DefaultComboBoxModel<String>(werte);
cbxName.setModel(cbxModel);

 

[schina02]

was spricht denn gegen eine while schleife?
er soll solange das item aus der spalte holen bis es nicht mehr geht...

zb so:

while (result.next) {
combobox.addItem("spaltenname"); }

so würde die combobox auch dynamisch bleiben oder nicht

 

[Attila]

JComboBox cb = new JComboBox();
//...
while (rs.next()){
    cb.addItem(rs.getSring(1));
}

 

[turtle]

was spricht denn gegen eine while schleife?

Nichts, Du musst halt "nur" über das Resultset laufen und alle Werte zu einer ComboBox addieren. Wenn Du da immer nur Strings reinpackst, um so einfacher.

Aber manchmal hat man Date, Color und was weiss ich und da reicht oft das DefaultComboboxModel, welches automatisch erzeugt wird wenn Du keins angibst, nicht aus.

 

[schina02]

okay erstmal vielen dank an Attila und turtle!

sieht jetzt so aus:

    Statement stmArt;
    String sqlArt;
    ResultSet rsArt;
    
    private void comArt_actionPerformed(ActionEvent e) {
        
        try {
            stmArt = con.createStatement(ResultSet.TYPE_SCROLL_INSENSITIVE, ResultSet.CONCUR_READ_ONLY);
            sqlArt = "select art_lang from art";
            stmArt.executeQuery(sqlArt);
            rsArt = stmArt.getResultSet();

            while (rsArt.next()) {
                comArt.addItem(rsArt.getString(1));
            }
        }
        catch (Exception ex) {
            ex.printStackTrace();
        }
    }

dazu ist noch zu sagen, dass es alles Strings sind und es immer bleiben werden, trotzdem danke für den Hinweis turtle

zeigt keinen fehler an, funktioniert einfach nur nich...wofür steht eigentlich die 1 bei comArt.addItem(rsArt.getString(1)); ?

 

[schina02]

ach die 1 in comArt.addItem(rsArt.getString(1)); steht für die spalte, alles klar.

irgendwas muss da noch fehlen, aber ich komm einfach nicht drauf was...

 

[Camino]

Mach mal anstatt
[JAVA=10]
stmArt.executeQuery(sqlArt);
rsArt = stmArt.getResultSet();
[/code]
folgendes

rsArt = stmArt.executeQuery(sqlArt);

Das Statement gibt dir nämlich mit dieser Methode das ResultSet zurück.

 

[schina02]

danke, aber das bringt leider auch nichts...

wo könnte noch ein fehler liegen?

ist actionPerformed für eine ComboBox richtig?

Connecten macht er auch vernünftig, da ich im gleichen Frame mit anderen Daten aus der DB arbeiten kann...

 

[Camino]

wo könnte noch ein fehler liegen?

Entweder du zeigst uns mal deinen ganzen Code, also das mit der Erstellung der ComboBox, wo der ActionListener aufgerufen wird und wo die ComboBox gefüllt wird. Oder du schreibst dir zwischendurch immer mal Ausgaben zum testen auf die Konsole und schaust, welche Werte vorhanden sind. Zum Beispiel in der Schleife, bei der du das ResultSet durchgehst, einfach mal schauen, was das in die ComboBox überhaupt reingeschrieben wird, usw.

Und als Fehlermeldung "funktioniert einfach nur nich" oder "das bringt leider auch nichts" sind nicht sehr aussagekräftig. Was passiert denn? Die ComboBox wird angezeigt und es ist aber nix drin?

 

[schina02]

okay camino, dann mach ich das mal genau so...

erstellung der combobox:

private JComboBox comArt = new JComboBox();

actionlistener und füllen der combobox:

    private void comArt_actionPerformed(ActionEvent e) {

        try {
  
            stmArt = con.createStatement(ResultSet.TYPE_SCROLL_INSENSITIVE, ResultSet.CONCUR_READ_ONLY);
            sqlArt = "select art_lang from art";
            rsArt = stmArt.executeQuery(sqlArt);

             while (rsArt.next()) {
                comArt.addItem(rsArt.getString(1));     
            } 
        } catch (Exception ex) {
            ex.printStackTrace();
        }
    }

die combobox erscheint und beim draufklicken wird eine leere zeile darunter erzeugt...

 

[Camino]

Na ja, das ist ja das, was du vorher auch schon gezeigt hattest. Ich dachte da eigentlich eher an etwas mehr Code, also eher den zusammenhängenden Code von der Erzeugung der Combobox bis zur actionPerformed-Methode, damit man genauer sehen kann, was wo und wann erstellt wird oder auf was zugreift, und wo bzw. wie der ActionListener ausgelöst wird. Die kleinen Ausschnitte, die du jetzt gepostet hast, zeigen das leider nicht.

Mach doch mal so etwas in der Schleife mit dem ResultSet:

while (rsArt.next()) {
     System.out.println("ResultSet-Ausgabe: " + rsArt.getString(1));
     comArt.addItem(rsArt.getString(1));     
}

Damit lässt du dir die Werte auf der Konsole ausgeben und siehst schonmal, ob da überhaupt was von der Abfrage zurückkommt, oder ob da nicht evtl. schon das Problem liegt.

 

[schina02]

Damit lässt du dir die Werte auf der Konsole ausgeben und siehst schonmal, ob da überhaupt was von der Abfrage zurückkommt, oder ob da nicht evtl. schon das Problem liegt.

super tipp, danke!

also hab einen button draus gebastelt, der das in die console schmeißt.

1. klick:
ResultSet-Ausgabe: Bungalow

2. klick:
ResultSet-Ausgabe: Bungalow
ResultSet-Ausgabe: Bungalow
ResultSet-Ausgabe: Einfamilienhaus
ResultSet-Ausgabe: Mehrfamilienhaus


das heißt beim 1. klick erzeugt er die erste zeile und beim 2. dann alle 3 zeilen

da is ja schon mal was faul dran...

 

[schina02]

und bevor ich jetzt wieder zu wenig als javacode poste, hau ich dir jetzt mal alles außer imports rein, Camino

public class frmInsert extends JFrame {


    MysqlDataSource mds = new MysqlDataSource();
    Connection con = null;
    Statement stm = null;
    ResultSet rs = null;
    String sql = null;


    private JPanel panelCenter = new JPanel();
    private JMenuBar menuBar = new JMenuBar();
    private JMenu menuFile = new JMenu();
    private JMenuItem menuFileExit = new JMenuItem();
    private JLabel statusBar = new JLabel();
    private ImageIcon imageOpen = new ImageIcon(frmInsert.class.getResource("openfile.gif"));
    private ImageIcon imageClose = new ImageIcon(frmInsert.class.getResource("closefile.gif"));
    private ImageIcon imageHelp = new ImageIcon(frmInsert.class.getResource("help.gif"));
    private JButton btnConnect = new JButton();
    private JTextArea txtArt = new JTextArea();
    private JTextArea txtZimmer = new JTextArea();


    private JTextArea txtWohnflaeche = new JTextArea();
    private JTextArea txtGrundflaeche = new JTextArea();
    private JButton btnInsert = new JButton();


    private BorderLayout borderLayout1 = new BorderLayout();
    private JScrollPane jScrollPane1 = new JScrollPane();
    private JTextArea txtLagebeschreibung = new JTextArea();
    private JLabel lblEuro = new JLabel();
    private JTextArea txtOrt = new JTextArea();
    private JTextArea txtPreis = new JTextArea();
    private JLabel lblm2wohn = new JLabel();
    private JLabel jLabel1 = new JLabel();
    private JPasswordField pasPasswort = new JPasswordField();

    private JComboBox comArt = new JComboBox();
    private JButton btnTestArt = new JButton();

    public frmInsert() {


        try {
            jbInit();
        } catch (Exception e) {
            e.printStackTrace();
        }
    }

    private void jbInit() throws Exception {
        this.setJMenuBar(menuBar);
        this.getContentPane().setLayout(borderLayout1);
        panelCenter.setLayout(null);
        this.setSize(new Dimension(575, 481));
        this.setTitle("Der ImmoCheck");

        menuFile.setText("File");
        menuFileExit.setText("Schließen (alles)");
        menuFileExit.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent ae) {
                fileExit_ActionPerformed(ae);
            }
        });
        statusBar.setText("");
        btnConnect.setText("CONNECT");
        btnConnect.setBounds(new Rectangle(210, 20, 165, 30));
        btnConnect.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent e) {
                btnConnect_actionPerformed(e);
            }
        });
        txtArt.setBounds(new Rectangle(140, 65, 150, 45));
        txtArt.setBorder(BorderFactory.createTitledBorder("Art"));
        txtArt.setEditable(false);
        txtZimmer.setBounds(new Rectangle(10, 145, 60, 40));
        txtZimmer.setBorder(BorderFactory.createTitledBorder("Zimmer"));

        txtZimmer.setEditable(false);
        txtWohnflaeche.setBounds(new Rectangle(10, 205, 75, 45));
        txtWohnflaeche.setBorder(BorderFactory.createTitledBorder("Wohnfläche"));
        txtWohnflaeche.setEditable(false);
        txtGrundflaeche.setBounds(new Rectangle(10, 270, 75, 45));
        txtGrundflaeche.setBorder(BorderFactory.createTitledBorder("Grundfläche"));
        txtGrundflaeche.setEditable(false);
        btnInsert.setText("HINZUFÜGEN!");
        btnInsert.setBounds(new Rectangle(140, 375, 265, 20));
        btnInsert.setEnabled(false);
        btnInsert.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent e) {
                btnInsert_actionPerformed(e);
            }
        });
        jScrollPane1.setBounds(new Rectangle(140, 115, 410, 245));
        txtLagebeschreibung.setLineWrap(true);
        txtLagebeschreibung.setBorder(BorderFactory.createTitledBorder("Lagebeschreibung"));
        txtLagebeschreibung.setEditable(false);
        lblEuro.setText("€");
        lblEuro.setBounds(new Rectangle(505, 395, 35, 15));
        txtOrt.setBounds(new Rectangle(320, 65, 140, 45));
        txtOrt.setBorder(BorderFactory.createTitledBorder("Ort"));
        txtOrt.setEditable(false);
        txtPreis.setBounds(new Rectangle(425, 370, 80, 40));
        txtPreis.setBorder(BorderFactory.createTitledBorder("Preis"));
        txtPreis.setEditable(false);
        lblm2wohn.setText("m²");
        lblm2wohn.setBounds(new Rectangle(85, 235, 34, 14));
        jLabel1.setText("m²");
        jLabel1.setBounds(new Rectangle(85, 300, 34, 14));
        pasPasswort.setBounds(new Rectangle(15, 15, 175, 45));
        pasPasswort.setBorder(BorderFactory.createTitledBorder("Passwort"));

        comArt.setBounds(new Rectangle(400, 25, 140, 20));

        comArt.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent e) {
                comArt_actionPerformed(e);
            }
        });
        btnTestArt.setText("TESTE");
        btnTestArt.setBounds(new Rectangle(40, 360, 75, 21));
        btnTestArt.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent e) {
                jButton1_actionPerformed(e);
            }
        });
        menuFile.add(menuFileExit);
        menuBar.add(menuFile);
        this.getContentPane().add(statusBar, BorderLayout.NORTH);
        panelCenter.add(btnTestArt, null);
        panelCenter.add(comArt, null);

        panelCenter.add(pasPasswort, null);
        panelCenter.add(jLabel1, null);
        panelCenter.add(lblm2wohn, null);
        panelCenter.add(txtPreis, null);
        panelCenter.add(txtOrt, null);
        panelCenter.add(lblEuro, null);
        jScrollPane1.getViewport().add(txtLagebeschreibung, null);
        panelCenter.add(jScrollPane1, null);
        panelCenter.add(btnInsert, null);
        panelCenter.add(txtGrundflaeche, null);
        panelCenter.add(txtWohnflaeche, null);
        panelCenter.add(txtZimmer, null);
        panelCenter.add(txtArt, null);
        panelCenter.add(btnConnect, null);
        this.getContentPane().add(panelCenter, BorderLayout.CENTER);
    }


    void fileExit_ActionPerformed(ActionEvent e) {
        try {
            if (rs != null)
                rs.close();
            if (stm != null)
                stm.close();
            if (con != null)
                con.close();
        }

        catch (Exception ex) {
            ex.printStackTrace();
            System.out.println("Beim Disconnecten/Schließen des Fensters gab es ein Problem");
        }

        System.exit(0);
    }


    void helpAbout_ActionPerformed(ActionEvent e) {
        JOptionPane.showMessageDialog(this, new frmImmoCheck_AboutBoxPanel1(), "About", JOptionPane.PLAIN_MESSAGE);
    }

    private void btnConnect_actionPerformed(ActionEvent e) {


        try {
            String sPass = pasPasswort.getText();
            if (sPass != null) {
                final String URL = "jdbc:mysql://localhost/immobilien";
                final String USER = "root";
                final String PASSWORD = sPass;

                mds.setURL(URL);
                mds.setUser(USER);
                mds.setPassword(PASSWORD);
                con = mds.getConnection(USER, PASSWORD);

                stm = con.createStatement(ResultSet.TYPE_SCROLL_INSENSITIVE, ResultSet.CONCUR_READ_ONLY);
                sql = "select id, art, ort, zimmer, lagebeschreibung, wohnflaeche, grundflaeche, preis from sortiment";
                stm.executeQuery(sql);
                rs = stm.getResultSet();

                if (con != null)
                    JOptionPane.showMessageDialog(null, "Sie sind nun mit der Datenbank verbunden.");

                else
                    JOptionPane.showMessageDialog(null,
                                                  "Sie sind nicht mit der Datenbank verbunden. Überprüfen Sie die Verbindung.");

                btnInsert.setEnabled(true);
                txtArt.setEditable(true);
                txtGrundflaeche.setEditable(true);
                txtLagebeschreibung.setEditable(true);
                txtOrt.setEditable(true);
                txtPreis.setEditable(true);
                txtWohnflaeche.setEditable(true);
                txtZimmer.setEditable(true);
                pasPasswort.setEditable(false);
                pasPasswort.setText("");


            }
        }

        catch (Exception ex) {
            JOptionPane.showMessageDialog(null, "Falsches Passwort! Bitte versuchen Sie es erneut!");
            ex.getStackTrace();
        }
    }


    Statement stmArt;
    String sqlArt;
    ResultSet rsArt;

    private void comArt_actionPerformed(ActionEvent e) {

        try {

            stmArt = con.createStatement(ResultSet.TYPE_SCROLL_INSENSITIVE, ResultSet.CONCUR_READ_ONLY);
            sqlArt = "select art_lang from art";
            rsArt = stmArt.executeQuery(sqlArt);

            while (rsArt.next()) {
                comArt.addItem(rsArt.getString(1));

            }
        } catch (Exception ex) {
            ex.printStackTrace();
        }


    }

    private void btnInsert_actionPerformed(ActionEvent e) {

        try {

            String insert =
                "INSERT INTO sortiment (art, ort, zimmer, lagebeschreibung, wohnflaeche, grundflaeche, preis) " +
                "VALUES (?, ?, ?, ?, ? ,?, ?)";
            PreparedStatement ps = con.prepareStatement(insert);

            ps.setString(1, txtArt.getText());
            ps.setString(2, txtOrt.getText());
            ps.setInt(3, Integer.parseInt(txtZimmer.getText()));
            ps.setString(4, txtLagebeschreibung.getText());
            ps.setInt(5, Integer.parseInt(txtWohnflaeche.getText()));
            ps.setInt(6, Integer.parseInt(txtGrundflaeche.getText()));
            ps.setInt(7, Integer.parseInt(txtPreis.getText()));

            int n = ps.executeUpdate();
            if (n == 1) {
                JOptionPane.showMessageDialog(null, "Okay, ein Datensatz wurde hinzugefügt.");
                rs = stm.executeQuery(sql);

                txtArt.setText("");
                txtOrt.setText("");
                txtZimmer.setText("");
                txtWohnflaeche.setText("");
                txtGrundflaeche.setText("");
                txtLagebeschreibung.setText("");
                txtPreis.setText("");
            }

        } catch (Exception ex) {
            JOptionPane.showMessageDialog(null,
                                          "Da gab es wohl ein Problem beim Hinzufügen einer Immobilie. Bitte überprüfen Sie Eingaben.");
            ex.printStackTrace();
        }

    }

    private void jButton1_actionPerformed(ActionEvent e) {

        try {

            stmArt = con.createStatement(ResultSet.TYPE_SCROLL_INSENSITIVE, ResultSet.CONCUR_READ_ONLY);
            sqlArt = "select art_lang from art";
            rsArt = stmArt.executeQuery(sqlArt);

            while (rsArt.next()) {
                System.out.println("ResultSet-Ausgabe: " + rsArt.getString(1));
                comArt.addItem(rsArt.getString(1));
            }

        } catch (Exception ex) {
            ex.printStackTrace();
        }


    }
}

mit dem button packt er es perfekt in die Combobox.

kann es also sein, dass ich einfach den falschen listener frü die combobox benutze?

benutze momentan comArt_actionPerformed

 

[Camino]

Da könnte z.B. der Fehler liegen:
[JAVA=224]
Statement stmArt;
String sqlArt;
ResultSet rsArt;

private void comArt_actionPerformed(ActionEvent e) {

try {

stmArt = con.createStatement(ResultSet.TYPE_SCROLL_INSENSITIVE, ResultSet.CONCUR_READ_ONLY);
sqlArt = "select art_lang from art";
rsArt = stmArt.executeQuery(sqlArt);

while (rsArt.next()) {
comArt.addItem(rsArt.getString(1));

}
} catch (Exception ex) {
ex.printStackTrace();
}

}
[/code]

Du solltest die Variablen sqlArt, stmArt und rsArt in den Methodenblock von comArt_actionPerformed reinpacken, also dass diese nur dort verfügbar sind.

Und normalerweise geht man bei Datenbankabfragen so vor:
Man holt sich die Verbindung zur DB (Connection), erstellt das Statement und dieses liefert einem das ResultSet. Danach sollten Connection, Statement und ResultSet auch wieder mit close() geschlossen werden, und zwar am besten in der umgekehrten Reihenfolge, wie beim Öffnen/Erstellen. Also zuerst ResultSet, dann Statement und dann Connection schliessen.

 

[Camino]

mit dem button packt er es perfekt in die Combobox.

kann es also sein, dass ich einfach den falschen listener frü die combobox benutze?

benutze momentan comArt_actionPerformed

Das ist auch ein bisschen unverständlich. Du hast einen ActionListener auf die ComboBox gelegt, mit dem du diese ComboBox mit Daten aus der Datenbank füllen möchtest? Normalerweise reagiert dieser ActionListener ja dann, wenn du dort etwas mit der Maus anklickst. Bin mir jetzt auch nicht ganz sicher, ob der ActionListener dort auch schon ausgelöst wird, wenn über eine andere Methode dort etwas hinzugefügt wird.

Wenn du auf die Auswahl in deiner ComboBox reagieren möchtest, dann brauchst du dafür einen ItemListener, der mitkriegt, welches Item in der ComboBox du ausgewählt hast und dann dementsprechend mit dem Wert etwas macht.

Wenn du die ComboBox mit Daten aus der Datenbank füllen möchtest, kannst du das entweder über einen Button machen (so wie du das schon gemacht hast), dann nimmst du aber auch diese Methode, welche der ActionListener des Buttons aufruft. Oder du füllst die ComboBox einfach nachdem diese erstellt wurde.

Und wie gesagt, Connection holen, Statement erstellen, ResultSet abfragen und danach diese alle wieder schliessen.