ActionListeners for an array of JComboBoxes...

THEOWL

JGO n00b

Posts: 10

Java games rock!

ActionListeners for an array of JComboBoxes...

« on: 2005-04-22 15:50:15 »

Hi,

In my program, each user has a JComboBox. However, the number of users varies depending on how many users are specified at the start of the game.

As a result I decided to make an array of JComboBoxes:
private JComboBox[] characterBox;

Then, later I specify the size of the JComboBox:
characterBox=new JComboBox[size];

I have no problem creating the JComboBoxes or adding them to the JPanel.

The problem occurs when I try to make actionlisteners. The following code is in a for loop that runs the number of users in the game.

characterBox

.addActionListener(

   new ActionListener()
   {
      public void actionPerformed( ActionEvent event )
{

String picNum = (String)characterBox
.getSelectedItem();

}
}
);//END Listener

The problem is that the compiler tells me x must be final. However I can't have it final because I am looping the creating of the Listeners. I know the actionlistener code works but I can't figure out how to create all of the listeners in a for loop. Any thoughts? Thanks.

digitprop

Full Member

Posts: 108

Re: ActionListeners for an array of JComboBoxes...

« Reply #1 on: 2005-04-24 06:49:38 »

I would parameterize the JComboBox into the constructor. Here is an example with an explicit class implementing ActionListener:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15

class MyActionListener implements ActionListener
{
   private JComboBox comboBox;

   public MyActionListener(JComboBox comboBox)
   {
      this.comboBox=comboBox;
   }

   public void actionPerformed(ActionEvent e)
   {
      String picNum=(String)comboBox.getSelectedItem();
      // ... do something ...
   }
}

The problem with your approach is that x is not known at compile-time, and whereas x is in the scope of the executed code at run-time, for the implicit ActionListener implementation it is not (as it will be created at compile-time, not at run-time). The compiler wants it to be final, because then it would be possible to get the actual integer value at compile-time.

Instead of having a reference to the JComboBox for each listener instance, you could also have a class member integer which is assigned the value of x; however, I would prefer the JComboBox as it makes it clear what the stored value is about.

M. Fischer . www.digitprop.com

c_lilian

JGO Ninja

Posts: 643

Java games will probably rock someday...

Re: ActionListeners for an array of JComboBoxes...

« Reply #2 on: 2005-04-24 09:38:41 »

Another (better) way (in my opinion) :

use jcombobox.putClientProperty(key, value) to store user information into each combobox. (for example an integer, a UserInfo object...) and retrieve it with getClientProperty(key)

write only one ActionListener, like this :

ActionListener comboListener = new ActionListener(){
public void actionPerformed(ActionEvent e){
JComboBox cbx = (JComboBox) e.getSource();
UserInfo ui = (UserInfo) cbx.getClientProperty("user.info");
// do something depending on the user info
}
};

add this listener to every user combos.

Lilian

Former java games developer...

Games published by our own members! Go get 'em!

gjbeyl

JGO n00b

Posts: 3

Java games rock!

Re: ActionListeners for an array of JComboBoxes...

« Reply #3 on: 2005-05-10 19:13:09 »

You can cheat the compiler. Grin

In the for loop:

final JComboBox finalCB = characterBox;
finalCB.addActionListener(new ActionListener()  
{
   public void actionPerformed( ActionEvent event )
   {          
      String picNum = (String)finalCB.getSelectedItem();  
   }
});[/pre]

Pages: [1]

		ActionListeners for an array of JComboBoxes... (Read 1341 times)
0 Members and 2 Guests are viewing this topic.