How to populate data in a JComboBox?

Question

I have created a GUI and have a database located externally in which I fetch data from. I am using the GUI builder in NetBeans to do this. Does anyone know of a simple way t

Answer 1

EDIT: This is your basic mistake.. you're calling the method in ActionPerformed !!

classConstructor(){
setDiscountNames("Fixed", jComboBox1); // call this method here.. This will work.

}

If the values are printing correctly then try this..

List<String> strings = new ArrayList<String>();
while(rs.next()){

     strings.add(rs.getString("Name"));  // Confirm if "Name" is valid


    }
cbox.addItem(strings);

Answer 2

When trying to add elements dynamically to a combo box, use MutableComboBoxModel.addElement

JComboBox box = new JComboBox(new DefaultComboBoxModel());
....
MutableComboBoxModel model = (DefaultComboBoxModel)box.getModel();
while (rs.next()) {
    model.addElement(rs.getString("Name"));
}

To remove all elements you could also do

((DefaultComboBoxModel)box.getModel).removeAllElements();

Using the model methods will fire the necessary changes to update the ui

Answer 3

It might be that your SELECT query returns no results. To verify this, you can add logging statements inside the while (rs.next()) loop. My SQL knowledge is a bit rusty, but I remember using ' (single quotes) for string literals, whereas you use " (double quotes) in your statement.

Besides that, I see several things which could cause problems:

• The SQL code for PreparedStatement should not be created by string concatenation. Instead, use ? for parameter values which will be substituted into the statement, e.g.

  stmt = con.prepareStatement("SELECT Name FROM Discount WHERE Type = ?");
  stmt.setString(1, type); // first (and only) parameter
  rs = stmt.executeQuery();
  

• It is strongly recommended that you explicitly close JDBC resources after being done with them. For this, you need to add a finally block after the catch (SQLException ...), e.g.

  } finally {
       try {
           if (rs != null) rs.close();
       } catch (SQLException ignore) {}
  
       try {
           if (stmt != null) stmt.close();
       } catch (SQLException ignore) {}
  
       try {
           if (conn != null) conn.close();
       } catch (SQLException ignore) {}
  }
  

  or, preferrably, use the try-with-resources statement (if you are using Java 7 and higher):

  try (Connection con = DriverManager.getConnection(...)) {
      // ...
      try (PreparedStatement stmt = con.prepareStatement("SELECT Name FROM Discount WHERE Type = ?")) {
          // ...
          try (ResultSet rs = stmt.executeQuery()) {
              while (rs.next()) {
                  // processing
              }
          }
      }
  } catch (SQLException) {
      Logger....;
  } // notice no finally block; resources are closed automatically