Question

I have a list of Map.Entry<String,Integer>s that I am looping through, and for each one, making a JLabel/JSpinner representing that particular entry.

How can I make it so that when the ChangeListener fires on the JSpinner, it updates that entry to reflect the new value?

My code looks like

for (Map.Entry<String,Integer> entry : graph.getData()) {
    SpinnerNumberModel model = new SpinnerNumberModel(
      entry.getValue(),(Integer)0,(Integer)100,(Integer)5);
    JSpinner spinner = new JSpinner(model);

    JLabel label = new JLabel(entry.getKey());

    spinner.addChangeListener(new ChangeListener() {
        public void stateChanged(ChangeEvent e) {
            entry.setValue((Integer)((JSpinner)e.getSource()).getValue());
        }
    });

    // more stuff
}

However, this does not work, because entry either needs to be final or object-level, and it is neither.

Is there an easier way, like C#’s dataSource that directly binds the object to the spinner? Or how can I make this work?

Answer 1

There are several options if you want to stick with individual JLabel and JSpinner. Use @trashgod’s answer if the map could get large.

1. WRONG Per @Suresh Kumar’s comment: final Map.Entry<String,Integer> entry
2. WRONG Add spinner.putClientProperty("MyEntryKey", entry), then in your ChangeListener get the entry with spinner.getClientProperty.
3. Add spinner.putClientProperty("MyEntryKey", entry.getKey()), then in your ChangeListener graph.put(spinner.getClientProperty(), ...).
4. From @Michael’s answer (not sure if this is ethical): Add final String key = entry.getKey(), then in your ChangeListener graph.put(key, ...).

5. Let your enclosing class implement ChangeListener to avoid one ChangeListener per JSpinner:

   public void stateChanged(ChangeEvent e) {
   JSpinner spinner = (JSpinner)e.getSource();
   graph.put((String)spinner.getClientProperty(“MyEntryKey”), (Integer)spinner.getValue());
   }

BTW graph.getData() looks a bit odd. If graph implements Map, you should use graph.entrySet().