ActionListener as nested class, anonymous class etc.

Question

I'm programing my own text editor and im trying to decide in what way to implement an ActionListener for JMenuItems.
I've got 3 possible ideas of how to implement it.
First way is to implement the ActionListener in the same class as the JMenu and use a switch statement or a fairly long if-else statement.
Second way is to create nested classes for each ActoinEvent.
public class OuterClass {
     //Some random code here...
     private class ActionClass implements ActionListener{
          public void actionPerformed(ActionEvent e) {
               //Random code.
}And final way is creating anonymous classes adding ActionListeners for each JMenuItem.
menuItem.addActionListener(new AbstractAction(){
public void actionPerformed(ActionEvent e) {
});But i can't decide on wich of these are the moste correct and accepted way.
Could someone point me to the right direction?
Edited by: Idono on Jun 3, 2010 7:36 PM

Answer 1

the only time you would do the first one would be if you wanted several ActionListeners to do the EXACT SAME THING.
Then you just write the "actionClass" one time, and have each Component use it.
private class ActionClass implements ActionListener{
          public void actionPerformed(ActionEvent e) {
               //Random code.
menuItem.addActionListener(new ActionClass());
menuItem1.addActionListener(new ActionClass());
menuItem2.addActionListener(new ActionClass());
menuItem3.addActionListener(new ActionClass());But (as the other person mentioned) usually you use anonymous classes because each component has different actions.
menuItem.addActionListener(new AbstractAction(){
public void actionPerformed(ActionEvent e) { ... }
menuItem1.addActionListener(new AbstractAction(){
public void actionPerformed(ActionEvent e) { ... }
menuItem2.addActionListener(new AbstractAction(){
public void actionPerformed(ActionEvent e) { ... }
});