I am currently replacing my anonymous ActionListeners
new ActionListener() {
@Override
public void actionPerformed(final ActionEvent event) {
First of all you should provide public methods for all actionPerformed used in your actions (createPerson, removePerson, etc.). All these action methods should be in one class (I call it PersonController). Than you need to define your AbstractPersonAction:
public class AbstractPersonAction extends AbstractAction {
private PersonController controller;
public AbstractPersonAction(PersonController aController) {
controller = aController;
}
protected PersonController getContrller() {
return controller;
}
}
Now you can extract all your actions into separate classes.
public class CreatePersonAction extends AbstractPersonAction {
public CreatePersonAction(PersonController aController) {
super(controller);
}
public void actionPerformed(ActionEvent ae) {
getController().createPerson();
}
}
These actions can be a part of an outer class or be placed in separate "actions" package.
You can keep your actions in a separate package to isolate them. Sometimes, it is useful to keep them in one class, especially if actions are related or have a common parent, for example:
public class SomeActions {
static class SomeActionX extends AbstractAction {
@Override
public void actionPerformed(ActionEvent e) {
}
}
static class SomeActionY extends AbstractAction {
@Override
public void actionPerformed(ActionEvent e) {
}
}
static class SomeActionZ extends AbstractAction {
@Override
public void actionPerformed(ActionEvent e) {
}
}
}
Then to access them:
JButton button = new JButton();
button.setAction(new SomeActions.SomeActionX());
I'm just feeling the strain of converting ~60 ActionListeners into separate classes.
Only you can decide if 60
is minimal. This example uses four instances of a single class. StyledEditorKit
, seen here, is a good example if grouping as a series of static factory methods. The example cited here uses nested classes. JHotDraw
, cited here, generates suitable actions dynamically.