SwingWorker verschwindet nicht

[Iron Monkey]

Hi!

Warum verschwindet der SwingWorker nicht, wenn die Methode "done()" zum Schluss bereits durchgeführt ist? ( siehe Bild )
Ich habe hier im Forum auch erfahren, dass der SwingWorker nur einmal durchführen kann! Wenn ja, was mache ich jetzt nun?

Hier ist mein Code:

Wenn ich das Fenster mit "JFrame" öffne, dann wird der SwingWorker in der Methode "processWindowEvent" unter WindowEvent.WINDOW_OPENED ausgeführt und sobald der SwingWorker vollständig fertig ist, verschwindet der im Prozess nicht! (siehe Bild) Ich verwende zum ersten Mal mit SwingWorker von Java 6! Was habe ich nun falsch gemacht?

new SwingWorker<Void, String>()
			{
				private ArrayList<ArrayList<Object>> listTermine = null;

				// Job wird in eigenem Thread ausgeführt.
				// Hier dürfen keine Manipulationen an Swing-Komponenten stattfinden.
				@SuppressWarnings("unchecked")
				@Override
				protected Void doInBackground() throws Exception
				{
					MomentDialogWithConsole moment = new MomentDialogWithConsole( parent, false, mcd_RLH );
					
					// Bediener-Gruppe aus der DB holen und anschliessend in die JList füllen!
					moment.setText( "Bediener-Gruppe holen" );
					Vector<String> listBedienerGruppe = ByteConverter.readBedienerGruppeFromDB( parent, mcd_RLH, mcd_RLH.USERID, true );

					if( listBedienerGruppe != null )
					{
						oldBedienerGruppe = (Vector<String>) listBedienerGruppe.clone();
						
						for( String elemente : listBedienerGruppe )
							publish( elemente );
						
						listBedienerGruppe.add( mcd_RLH.USERID );

                                                .....
					}
					
					moment.setText( "Termine holen" );
					if( listBedienerGruppe != null )
					{
						for( int i = 0; i < listBedienerGruppe.size(); i++ )
						{
							if( isCancelled() )
								break;
							
							// Andere Aufgabe
						}
					}
					else
					{
						// Dem Server sagen:"Gib mir Daten von der DB"
					}
					
					moment.close();
					moment = null;

					if( listBedienerGruppe != null )
					{
						listBedienerGruppe.clear();
						listBedienerGruppe = null;
					}
					
					return null;
				}

	            // Durch publish() veröffentliche Zwischenergebnisse behandeln.
	            // process() wird innerhalb des EDT aufgerufen.
				// Hier können Manipulationen an GUI-Elementen sicher vorgenommen werden.
				@Override
				protected void process( List<String> chunks )
				{
					for (Object element : chunks)
						getModelBedienerUebernahme().addElement(element);
				}
				
				// Worker hat seinen Job beendet. done() wird innerhalb des EDT aufgerufen.
				// Hier können Manipulationen an GUI-Elementen sicher vorgenommen werden.
				@Override
				protected void done()
				{
					if( !isCancelled() )
					{
						....
					}
				}

			}.execute();

Gruß
Iron Monkey

 

[Marco13]

Falls es nur um den Thread geht: Die SwingWorker-Threads werden in einem Pool verwaltet, d.h. wenn man z.B. 2 SwingWorker startet, werden 2 Threads im Pool erzeugt (boah, ein Pool wär jetzt cool), und wenn sie Fertig sind, und man erzeugt danach nochmal 3 SwingWOrker, werden die 2 Thread vom Anfang "recyclet" und nur EIN neuer zusätzlich erstellt.
Diese Pool-Threads sterben nach einer gewissen Zeit, wenn sie nicht benutzt werden (eine Minute vielleicht? Könnte man im Quelltext nachsehen...)

 

[Iron Monkey]

OK, danke erstmal für die Info!

Es wird immer nur 1 Swingworker nach jedem Öffnen des Fensters gestartet.

So, und wenn ich das Fenster öffne, dann wird der Swingworker gestartet. Beim ersten Mal klappt es immer super: Daten über Server aus der DB holen und danach komplett in die Tabelle füllen!

Wenn ich aber das Fenster schliesse und das wieder öffne, dann wird der gleiche SwingWorker wieder gestartet. Beim 2. Mal wird das Holen der Daten über Server aus der DB auf einmal blockiert! Da dachte ich, es hat was mit dem SwingWorker zu tun. Es passiert immer wieder ab dem 2. Mal! Das ist genau das, was ich nicht verstehe! Der Code in der Methode "doInBackground" wird beim 2. Mal nicht vollständig durchgeführt, obwohl der gleiche Code beim 1. Mal super gut funktioniert. Das verstehe ich nicht!

Gruß
Iron Monkey

 

[Marco13]

Ja, in dem Code ist ziemlich viel "..." weggelassen, und bei einer DB-Verbindung wird man wohl kein KSKB basteln können, aber ... vielleicht kannst du das ganze ja mal im Debugger starten, um zu sehen, wo genau er hängt... Keine Fehlermeldung und so?

 

[SlaterB]

> Der Code in der Methode "doInBackground" wird beim 2. Mal nicht vollständig durchgeführt

jeder kompliziere Code ist von Natur aus schlecht für irgendwelche Aussagen,
baue in die Methode allein
> System.out.println("Methode läuft");
wenn das zweimal klappt, dann besteht beim Swingworker kein Problem,

noch ein Test:
führe die Methode zweimal direkt aus main oder aus einem ActionListener heraus aus, die GUI darf für den Test ruhig solange blockieren,
wichtige Meldungen testweise sowieso immer mit System.out.println

wenn das auch so zweimal nicht klappt und der SwingWorker unschuldig ist, dann machst du in deinem Code irgendwas fatales,
z.B. hast du
> moment.close();
vielleicht schließt das irgendeine übergeordnete Verbindung, die dann beim zweiten Durchlauf immer noch geschlossen ist

 

[Iron Monkey]

> Der Code in der Methode "doInBackground" wird beim 2. Mal nicht vollständig durchgeführt

jeder kompliziere Code ist von Natur aus schlecht für irgendwelche Aussagen,
baue in die Methode allein
> System.out.println("Methode läuft");
wenn das zweimal klappt, dann besteht beim Swingworker kein Problem,

noch ein Test:
führe die Methode zweimal direkt aus main oder aus einem ActionListener heraus aus, die GUI darf für den Test ruhig solange blockieren,
wichtige Meldungen testweise sowieso immer mit System.out.println

wenn das auch so zweimal nicht klappt und der SwingWorker unschuldig ist, dann machst du in deinem Code irgendwas fatales,
z.B. hast du
> moment.close();
vielleicht schließt das irgendeine übergeordnete Verbindung, die dann beim zweiten Durchlauf immer noch geschlossen ist

Jetzt habe ich gerade beim Debuggen Schritt für Schritt geprüft und stelle fest, dass in der Methode "doInBackground()" beim Vector-Füllen ein heimliches Exception von FutureTask.class geworfen wurde und ab der Zeile wird es sofort blockiert ( also unendliche Schleife )! Das würde bedeuten, dass das Füllen ins Vector NICHT in doInBackground() durchgeführt werden darf. Ist das richtig?



Schau mal hier:

FutureTask.class

        void innerRun() {
            if (!compareAndSetState(0, RUNNING))
                return;
            try {
                runner = Thread.currentThread();
                if (getState() == RUNNING) // recheck after setting thread
                    innerSet(callable.call());
                else
                    releaseShared(0); // cancel
            } catch (Throwable ex) {     // <-------------------------------------- EXCEPTION GEWORFEN !!
                innerSetException(ex);
            }
        }

Dann geht der zu innerSetException und da sehe ich eine unendliche Schleife:

FutureTask$Sync.class

void innerSetException(Throwable t) {
	    for (;;) {
		int s = getState();
		if (s == RAN)
		    return;
                if (s == CANCELLED) {
		    // aggressively release to set runner to null,
		    // in case we are racing with a cancel request
		    // that will try to interrupt runner
                    releaseShared(0);
                    return;
                }
		if (compareAndSetState(s, RAN)) {
                    exception = t;
                    result = null;
                    releaseShared(0);
                    done();
		    return;
                }
	    }
        }

Gruß
Iron Monkey

 

[SlaterB]

tja, in dieser unendlichen Schleife stehen auch ein paar return, die kann also schon abgebrochen werden,
sollte da tatsächlich ein Bug in so einer oft genutzten Klasse vorhanden sein?
schwer zu glauben

> Das würde bedeuten, dass das Füllen ins Vector NICHT in doInBackground() durchgeführt werden darf. Ist das richtig?

so gesehen dürftest du überhaupt keinen Code da haben, denn überall kann doch eine Exception auftreten,
gehe der Exception auf den Grund und entferne sie

allgemein könntest du zur Sicherheit in deinem Code selber ein try/ catch setzen,
damit ja keine Exception zur FutureTask.class durchkommt

 

[ymene]

SwingWorker schlucken gern mal die ein oder andere Exception. Um wirklich rauszufinden, welcher Fehler aufgetreten ist, kann man hierfür die get() Methode des Swingworkers im done()-Block aufrufen. Das rauszufinden hat mich auch schon die ein oder andere schlaflose Nacht gekostet.

Eine gute Erläuterung wie ich fand gibt es von Jonathan Giles.

 

[Iron Monkey]

OK, ich danke euch erstmal!

Ich habe endlich den Fehler gefunden, was von mir auch total blöd war! Ich habe eine statische Vector-Variable deklariert und beim Schliessen lösche ich diese statische Variable mit null! Und beim 2. Aufruf bricht deswegen die Methode "doInBackground" ab und geht dann direkt zu der Methode "done()"! Es war ein NullPointerException!

Es hat sich jetzt erledigt und es funktioniert einwandfrei!

Gruß
Iron Monkey

 

[Iron Monkey]

SwingWorker schlucken gern mal die ein oder andere Exception. Um wirklich rauszufinden, welcher Fehler aufgetreten ist, kann man hierfür die get() Methode des Swingworkers im done()-Block aufrufen. Das rauszufinden hat mich auch schon die ein oder andere schlaflose Nacht gekostet.

Eine gute Erläuterung wie ich fand gibt es von Jonathan Giles.

Ja stimmt, da hast du recht! Dieser Fehler hat für mich auch eine sclaflose Nacht gekostet, aber aus Fehler lernt man was und beim nächsten Mal kommt das nie wieder vor!

Gruß
Iron Monkey

 

[ymene]

Ein schönes Schlusswort: "Beim nächsten Mal kommt das nie wieder vor!"

gefällt mir