JComboBox MVC

Carregue todos os dados num list:

public static List<String> AtualizaComboBoxVariedade() { List<String> dados = new ArrayList<String>(); ResultSet rs = null; try { rs = stmt.executeQuery("SELECT DISTINCT variedade FROM Vinho;"); while (rs.next()) { dados.add(rs.getString("variedade")); } } catch (Exception ex) { throw new RuntimeException("Problemas no BD", ex); } finally { if (rs != null) rs.close(); } return dados; }

No caso do JComboBox, pode ser mais fácil carregar num (argh) Vector. Mas o código é o mesmo, é só trocar List e ArrayList por Vector.

Deu certo. Só que ele coloco tudo como um item só no combobox ficou assim: [Item1, Item 2] Tem como colocar cada dado como sendo um item no combobox? Assim:
Item 1
Item 2…

Sendo que eu to adicionando dessa maneira:

Cvariedade.addItem(ControleVinho.AtualizaComboBoxVariedade());

Faz um foreach:

for (String item : ControleVinho.AtualizaComboBoxVariedade()) { Cvariedade.addItem(item); }

Cara, seu código está repleto de mas práticas. Métodos estáticos, não respeita a code convention, manter o statement aberto, não tem as classes de negócio. Isso está muito, muito, muito, longe de sequer chegar perto de uma arquitetura MVC.

Valeu… vou tentar ajustar o código!