Action before actionPerformed!

[jambusa]

Hallo zusammen,

ich habe eine recht simple Anforderung, aber komme bisher zu keiner zufriedenstellenden Lösung und hoffe mir hier einen Ansatz holen zu können. Folgende vereinfachte Situation:

Ich habe eine Applikation mit mehreren Buttons. Diese sind setFocusable( false ); eingestellt. Aus diesem Grund habe ich meine eigene Buttonklasse geschrieben, die sich standardmäßig nicht fokussieren lässt (natürlich kann die noch mehr, aber für diese Situation unerheblich).

Im Konstruktor dieser Klasse gebe ich jeweils eine Action mit. Nun habe ich festgestellt, dass bevor der Button seine actionPerformed ausführt noch immer eine andere bestimmte Action ausgeführt werden muss.

Ein FocusListener kommt nicht in Frage, da dieser Button kein Fokus erhalten kann und einen weiteren ActionListener anmelden sollte ich auch vermeiden, da ich meine gelesen zu haben, dass man sich auf die Reihenfolge nicht verlassen kann. Gibt es noch andere Optionen?

Hier ein zweckmäßiges Example:

public class ActionBeforeActionExample extends JFrame
{
  private ColorButton blue;
  private ColorButton green;
  private ColorButton red;

  public ActionBeforeActionExample()
  {
    setDefaultCloseOperation( EXIT_ON_CLOSE );
    setBounds( 200, 200, 300, 300 );
    setLayout( new FlowLayout() );
    initComponents();
    setVisible( true );
  }


  private void initComponents()
  {
    blue = new ColorButton( new ColorBlueAction() );
    blue.setText( "blue" );
    green = new ColorButton( new ColorGreenAction() );
    green.setText( "green" );
    red = new ColorButton( new ColorRedAction() );
    red.setText( "red" );

    add( blue );
    add( green );
    add( red );
  }

  private class ColorButton extends JButton
  {
    public ColorButton( Action action )
    {
      super( action );
      setFocusable( false );
    }
  }

  private class ColorBlueAction extends AbstractAction
  {
    @Override
    public void actionPerformed( ActionEvent e )
    {
      System.out.println( "setColor: -> Blue" );
    }
  }

  private class ColorRedAction extends AbstractAction
  {
    @Override
    public void actionPerformed( ActionEvent e )
    {
      System.out.println( "setColor: -> Red" );
    }
  }

  private class ColorGreenAction extends AbstractAction
  {
    @Override
    public void actionPerformed( ActionEvent e )
    {
      System.out.println( "setColor: -> Green" );
    }
  }

  private class ResetOldColorAction extends AbstractAction
  {
    @Override
    public void actionPerformed( ActionEvent e )
    {
      System.out.println( "reset Color!" );
    }
  }

  public static void main( String[] args )
  {
    new ActionBeforeActionExample();
  }
}

Wie implementier ich am geschicktesten, dass immer bevor eine Farbe initialisiert wird die ResetOldColorAction aufgerufen wird, egal welchen Button ich anklicke? Natürlich wäre eine Lösung ResetOldColorAction in allen Actions jeweils aufrufen zu lassen, aber das würd ich gern vermeiden, da es in der richtigen Applikation zahlreiche Actions sind, die dann überarbeitet werden müssten.

Hoffe ihr habt eine schöne objektorientierte Idee!

Vielen Dank,
MfG,
Jambusa

 

[Michael...]

einen weiteren ActionListener anmelden sollte ich auch vermeiden, da ich meine gelesen zu haben, dass man sich auf die Reihenfolge nicht verlassen kann.

Das war mir jetzt nicht bewusst - bin allerdings noch nie in die Verlegenheit gekommen.
1. Möglichkeit: Das was in ResetOldColorAction ausgeführt wird - sofern noch nicht geschehen - in eine Methode auslagern und diese dann in den anderen Actions aufrufen
2. Möglichkeit: Alle betroffenen Actions von ResetOldColorAction erben lassen und dann in deren actionPerformed(..) zuerst super.actionPerformed() aufrufen.

 

[Marco13]

Ohne da jetzt 100% in die Tiefe nachgedacht zu haben, was spontanes: Sowas wie eine

  private class ChainedAction extends AbstractAction
  {
    private Action actions[];

    public ChainedAction(Action ...actions)
    {
        this.actions = actions.clone(); // null-check etc...
    }

    @Override
    public void actionPerformed( ActionEvent e )
    {
      for (Action action : actions) action.actionPerformed(e);
    }
  }

Das kann man dann auf unterschiedliche Arten verwenden, z.B.

 private class ColorButton extends JButton
  {
    public ColorButton( Action action )
    {
      super( new ChainedAction(new ResetColorAction(), action) );
      setFocusable( false );
    }
  }

oder so...

 

[jambusa]

Hey, dank euch beiden für die Vorschläge. Entschieden hab ich mich für die Implementierung von marco13. Sehr gute Idee, genau dass was ich gesucht habe. Ärgere mich, dass ich nicht selber drauf gekommen bin.

Vielen Dank,
Jambusa

 

[Marco13]

Ggf. müßte/sollte bei diesen Buttons dann noch setAction passend überschrieben werden, damit das Verhalten konsistent ist...