Thread vs Listener

[dotlens]

Hallo allerseits

Ich bin an einem Bugfix eines bestehenden Programms wo Threads mit Listener gemischt werden. Das sieht so aus:

	private class ProgramPanelUpdateActionListener extends AbstractAction {
		
		private Thread thread = null;
		
		public void actionPerformed(ActionEvent e) {
			// Only create a new thread and start it, if the previous one ended
			if (thread == null) {
				thread = new Thread() {
					
					@Override
					public void run() {
						doSomething();

						// Set the thread to null to show outside that this thread has ended
						ProgramPanelUpdateActionListener.this.thread = null;
					}
				};
				thread.start();
			}
		}
	}

Irgendwie finde ich das aber nicht sinnvoll. Die Listeners arbeiten doch sowieso nicht im gleichen Thread wie das GUI oder? und andere Berechnungen werden zu diesem Zeitpunkt nicht gemacht. Wieso soll also bei einem Event immer ein Thread erstellt werden, derr dann wieder auf null gesetzt wird?!?

Was haltet ihr davon?

Danke und Gruss

 

[Wildcard]

Natürlich ist das sinvoll. Die Listener werden vom EDT aufgerufen.
Dauert die Aktion die der Listener anschließend startet länger, würde die gesamte GUI während dieser Zeit blockieren wenn man keinen eigenen Thread startet.
An dieser Stelle keine Threads zu verwenden ist der Hauptgrund warum oft (fälschlicherweise) angenommen wird das Swing langsam sei.

 

[bygones]

hat nicht arenal da letztens einen so wunderbaren link von dem VideoTutorial von son nem Guru gepostet ?

bzgl FoxTrot / SwingUtilities usw....

 

[Wildcard]

Der gute Ben...
http://www.javalobby.org/eps/galbraith-swing-2/

 

[dotlens]

dankeschön
hatte geglaubt dass Listeners in einem eigenen Thread laufen

Seh ich mir mal an

 

[byte]

Wobei man da in der Tat noch optimieren kann, indem man nicht jedes mal einen neuen Thread erzeugt sondern z.B. einen CachedThreadPool verwendet.

 

[Linuxhippy]

if (thread == null) {
.....
ProgramPanelUpdateActionListener.this.thread = null;

Oh ein ganz böser Fehler - ein Feld von zwei verschiedenen Threads setzen ohne "volatile"-modifier oder synchronoized-block.
Sun schreibt überall dass man sowas auf keinen fall machen kann - die CPU cached den wert in einem ungünstigen Fall und dann verläuft sich das ganze.

Am besten du machst um das ganze if(thread == null) ...
einen synchronized(this){} herum.

Mfg Clemens