handling jradiobuttonmenuitem select only once

[Mark Lybarger]

i'm fairly new to swing, coming from jee.

i have a group of jradiobuttonmenuitem (s). when i select one, the actionPerformed method is always getting called. how do i only call the actionPerformed when it's first selcted.? right now i'm controling it by disabling the selected item, but that doesn't seem ideal.

    private static JMenuItem getJMenuItem(final String option_1, final JFrame frame) {
        JMenuItem jMenuItem = new JRadioButtonMenuItem(new AbstractAction(option_1) {
            public void actionPerformed(ActionEvent e) {
                JMenuItem item = (JMenuItem) e.getSource();
             
                JOptionPane.showMessageDialog(frame, option_1 + " " + item.isSelected() );
                // enable all 
                JPopupMenu p = (JPopupMenu) item.getParent();
                Component[] components = p.getComponents();
                for (int i=0; i< components.length; i++)
                {
                    JMenuItem j = (JMenuItem) components[i];
                    j.setEnabled(true);
                }
                // disable this option
                item.setEnabled(false);
            }
        });
        return jMenuItem;
    }

[Greg Brannon]

By design (and thankfully consistently), the actionPerformed() method will be called each time the menu item is selected. What the actionPerformed method DOES when called is under your program's control. What you're trying to do is not completely clear, but you could restrict what the actionPerformed() method does by determining if the menu item is already selected by using the isSelected() method and acting accordingly, OR if you mean that the menu item is only to be selected once, then it seems appropriate that it WOULD be disabled after being selected once. There are many other possibilities, but what I've suggested should suffice until you more completely describe what you're trying to do.

[Michael Dunn]

you might be better off using an ItemListener

private static JMenuItem getJMenuItem(final String option_1, final JFrame frame) {
  JMenuItem jMenuItem = new JRadioButtonMenuItem(option_1);
  jMenuItem.addItemListener(new ItemListener(){
    public void itemStateChanged(ItemEvent ie){
      if(ie.getStateChange() == ItemEvent.SELECTED) JOptionPane.showMessageDialog(frame, option_1 + " isSelected");
    }
  });
  return jMenuItem;
}

also, does it need to be static?