problem using vectors and jcomboboxes

[Paul_White]

I am working on a program that integrates with mysql.

I am querying the database and using a vector to populate a JcomboBox with the results . This is done as the GUI gets built and is working fine. Based on the item chosen in the first JcomboBox I want to popultate the second JcomboBox. I am using an action listener on the first JcomboBox, calling another class, passing the item selected in the first box, doing the query and returning the result as a vector.

As I used a vector in the first case I thought I could just call

box2.addItem(vector2);

My problem is that I get the result of the query displayed as a single line (one entry) enclosed by []

I am puzzled because JComboBox Box1 = new JComboBox(vector1); works to set up a JcomboBox at intitialisation but I cannot add a vector in the same way to an empty box after it has been made visible.

I can input single lines using box2.addItem("new line etc"); but I want to add a list as I have done successfully using a vector like in box 1.

Could someone give me a hint as to what I am doing wrong?

Thank you

Paul

[sunde887]

Box 1 uses a constructor. The addItem(...) is adding the string representation of the vector. You can use a loop to add each item of the vector to the jcombobox.

[JosAH]

As I used a vector in the first case I thought I could just call

box2.addItem(vector2);

My problem is that I get the result of the query displayed as a single line (one entry) enclosed by []

I am puzzled because JComboBox Box1 = new JComboBox(vector1); works to set up a JcomboBox at intitialisation but I cannot add a vector in the same way to an empty box after it has been made visible.

You can call it an inconsistency (or omission) in the API for the JComboBox class but the addItem( ... ) method just adds one single item to the list (as its name implies); you can't add all elements of a Vector to a JComboBox with just one method call. You have to call that method over and over again for each element in the Vector; something like this:

for (Object item: yourVector)
   box2.addItem(item);

kind regards,

Jos

[DarrylBurke]

I'd consider it better style to construct a new model and set it to the combo.

comboBox.setModel(new DefaultComboBoxModel(vector));

db

[DarrylBurke]

The addItem(...) is adding the string representation of the vector.

No, it's adding the vector. The default renderer uses the String returned from toString().

db

[Paul_White]

Guys,

Lots of good advice as always - Thank you all.

Darryl from all the answers given I liked the look of your answer (It seemed to me to be the closest to the way I wanted it coded) and it works like a charm!

Thanks again.