Action-listener

[michi2]

Hi,
ich habe in meinen programm einen Action-listener an dem alle menue-items,etc. angemeldet sind.
Diese werden dan mit lauter if()-Bedingungen unterschieden und behandelt.
In zwischen hab ich dort jedoch über 80 if()-Bedingungen hintereinander!
Wäre es klüger mehere Action-listener zu verwenden???


Thanks
PS: ich habe das JDK 1.4

 

[lin]

Wenn du die If-Bedingungen verschachtelst, könntest du vielleicht was an Performance rausholen...
z.B.

public void actionPerformed(ActionEvent ae) {
    	Object src = ae.getSource();
    	
    	if(src instanceof JButton) {
    
    	}
    	
    	if(src instanceof JMenuItem) {

    	}
    }

 

[Sky]

1.) Also, wenn schon, dann bitte so:

public void actionPerformed(ActionEvent ae) {
  Object src = ae.getSource();
    	
  if(src instanceof JButton) {
    
  } else if(src instanceof JMenuItem) {

  }
}

Ansonsten wird auch für den JButton die if für JMenuItem durchlaufen, was in der Summe eher langsamer als schneller wird.

2.) Noch besser - m. E. aber immer noch nicht gut:

public void actionPerformed(ActionEvent ae) {
  Object src = ae.getSource();
   doIt( src );
}

private void doIt( Object arg ) {
}
private void doIt( JButton arg ) {
}
private void doIt( JMenuItem arg ) {
}

3.) Der Quelltext wird langsam unübersichtlich bei 1.) und 2.). Deshalb versuche mehrere Listener zu schreiben, welche thematisch zusammenpassende Aufgaben erledigen.

 

[AlArenal]

Warum benutzt du keinen parametrisierten Action-Listener und arbeitest mit Switch-Case?

 

[michi2]

Danke!
Aber den vorschlag von 'AlArenal' versteh ich ehrlich gesagt nicht...
Bin noch nicht so weit.

 

[AlArenal]

Na z.B. sowas:

    button_overview_zoom_in.addActionListener(new MyActionListener(MyActionListener.BUTTON_OVERVIEW_ZOOM_IN));

...

    class MyActionListener implements ActionListener {
        public static final int BUTTON_OVERVIEW_ZOOM_IN = 1;
        public static final int BUTTON_OVERVIEW_ZOOM_OUT = 2;
        public static final int BUTTON_PRINT = 3;
        public static final int BUTTON_REFRESH = 4;

        private int type;

        public MyActionListener(int type) {
            this.type = type;
        }

       public void actionPerformed(ActionEvent e) {
            switch (type) {
            
            case BUTTON_OVERVIEW_ZOOM_IN: 
                break;
            
            case BUTTON_OVERVIEW_ZOOM_OUT: 
                break;
            
            case BUTTON_PRINT: 
                break;
            
            case BUTTON_REFRESH: 
                break;

            default:
                log.error("unknown type : " + type); //$NON-NLS-1$

            }
        }
        
    }

Das halte ich für deutlich übersichtlicher und schneller, als irgendwelche verschachtelten if-Klamotten (mit/ohne instnaceof), nur um rauszubekommen, wo der Event nun eigentlich herkam.

Was deine Performance-Sorge bei einem ActionListener angeht, hängt es davon ab wie oft er aufgerufen wird und wie lange seine Abarbeitung dauert, ob es sich lohnt sich da wirklich nen Kopf drüber zu machen. Wenn ich irgendwo auf nen Knop drücke werde ich nicht merken, ob der Rechner das nun in 1 oder 5 hundertstel Sekunden ausführt...

 

[Beni]

Also ich weiss nicht AlArenal, da würde ich doch lieber mehrere ActionListener machen (dann hätte man weniger Code, weil das Typsystem unnötig wäre) :wink:

michi2, ein einzelner ActionListener benötigt nicht viel Speicher, und auch 80 Instanzen werden einen heutigen PC nicht weiter beeindrucken :wink: