JComboBox stores same

[Giuseppa Cefalu]

Hi,

The comboBox below, or variable str stores the table name selected thorugh the dropdown of the combobox twice. System.out.println(str); outputs:
test5
test5

Also; constructor initialization in the related class (public class Dat extends AbstractTableModel) through dtable = new Dat(str); happens twice, and since I use the table name to open the table for reading, the table gets opened twice. I have not idea of what is going on? Could you please read my code?
Thank you.

 public class Combo {
    private Connection conn;
    private Statement stmt;
    private ResultSet rset;
    private JComboBox comboBox;
    private JFrame frame;
    private JPanel panel;
    private JTextField txt;
    private String str;
    private Dat dtable;
    private JTable table;

   public Combo(){

       frame = new JFrame();
       panel = new JPanel();


        try{
            conn = getConnection();
            stmt = conn.createStatement();
            rset = stmt.executeQuery("SELECT *FROM TABLES");
            DefaultComboBoxModel model = new DefaultComboBoxModel();
            while(rset.next()){
                model.addElement(rset.getString(1));
                
            }
            comboBox = new JComboBox(model);
        }catch(Exception e){
            System.out.println(e.getMessage());
        }finally{
            try{
                rset.close();
                stmt.close();
                conn.close();
            }catch(SQLException se){

                System.out.println(se.getMessage());
            }
        }

        txt = new JTextField(10);
        panel.add(comboBox, BorderLayout.NORTH);
        panel.add(txt);
     
        comboBox.addItemListener(new ItemListener(){
            public void itemStateChanged(ItemEvent ie){
                str = (String)comboBox.getSelectedItem();
                txt.setText(str);
                dtable = new Dat(str);
                System.out.println(str);
            }

        });

        table = new JTable(dtable);
        
        frame.add(panel);
        frame.setSize(400, 200);
        frame.setVisible(true);


    }

    public static void main(String[] args){

        Combo combo = new Combo();
        
    }

[Rob Camick]

Use an ActionListener instead of an ItemListener.

An ItemListener has two states, selected/unselected.

[Giuseppa Cefalu]

Thank you! I did that and it works.