Threads behindern sich gegenseitig

[djui]

Hi!

Ich hab ein großes Problem.
Hab eine main-Methode, die zwei Klassen instanziert. Beide instanzierten Klassen erben von der Klasse Thread. Anschließend rufe ich die .start()-Methode beider Instanzen auf:

	XXXThread xxxThread = new XXXThread ();
	YYYThread yyyThread = new YYYThread ();

	xxxThread.start();
	yyyThread.start();

Soweit, so gut. In der run-Methode beider Threads (die ja von .start() aufgerufen wird) mach ich nichts anderes, als eine Klasse Generator zu instanzieren (in jedem der beiden Threads). Anschließend wird eine Methode der Instanz aufgerufen. Damit will ich erreichen, dass der Inhalt der aufgerufenen Methode parallel ausgeführt wird. Also 1x/Thread.

Der Inhalt der Methode, die ausgeführt wird, soll folgendes machen: DB-Abfragen machen, verarbeiten und das "Ergebnis" auf ein GUI schreiben. Ein Thread schreibt auf eine JTextArea, der andere Thread auf die andere JTextArea.

Nun mein Problem: Wird die Methode, die parallel ausgeführt wird ausgeführt, und das wird auch parallel ausgeführt (debug-Ausgaben) passiert teilweise (und genau das ist das Problem, dass es nicht IMMER so ist) folgendes: An einer JTextArea wird der Inhalt angezeigt, der eigentlich auf der anderen angezeigt werden soll. Es wird also bei einen Thread das falsche ausgeführt. Wie gesagt ist es leider so, dass das nur selten vorkommt.
Lt. Debug-Ausgaben wird zwar Thread 1 ausgeführt, die Select-Ergebnisse der DB gehören jedoch zur Query, die ich in Thread 2 absetze.

Habt ihr irgendeine Ahnung, was da falsch sein kann? Behindern sich die Threads da irgendwie gegenseitig? Hab ich die Threads falsch angelegt bzw. ausgeführt? Gibt es evtl. ein Problem mit der Methode, die parallel ausgeführt werden soll (also rufe ich sie falsch auf)? Das Problem ist, dass ich mich mit Threads nicht so gut auskenne. Hab zwar nachgelesen, find aber nichts besonderes, das meinen Fehler beheben könnte!

Vielleicht habt ihr ja einen Tipp!

Vielen Dank im Voraus!

 

[Tobias]

Das ist eine klassische DataRace-Situation. Thread 1 belegt Variablen, wird dabei unterbrochen (das kannst du NICHT beeinflußen), Thread 2 liest jetzt die unvollstädig belegten Variablen - falsche Ausgaben sind das Ergebnis.

synchronized, wait(), notify() und ab Java 5.0 java.util.concurrent.* sind deine Freunde zur Lösung des Problems. Ausserdem solltest du ein vernünftiges Lehrbuch zu Rate ziehen - z.B. die Insel.

mpG
Tobias

 

[Campino]

Ich denke du hast dich irgendwo vertippt und z.B. TextArea1 statt TextArea2 geschrieben, überprüf das mal.

 

[Tobias]

@Campino: Dann müßte das immer und nicht nur manchmal passieren.

mpG
Tobias

 

[djui]

Ja Race-Conditions, Deadlocks, Mutual Exclusions etc. kenn ich ja wohl. Also Probleme mit Threads allgemein.

Aber das kann ja irgendwie nicht der Grund sein meiner Meinung nach, weil Thread 1 ja eine eigene Instanz des Generators verwendet, genauso wie Thread 2. Also jeder Thread verwendet seine eigene Instanz!

Da kann das ja fast nicht sein, dass da ein Thread eine falsche/andere Instanz einer Klasse nimmt, oder? Oder blick ich da jetzt nicht durch?

Vielen Dank! Antworten sind wirklich super!

PS: Falsch benannt hab ich das nicht bzw. falsch aufgerufen. Der Grund wurde schon von Tobias genannt!

 

[sparrow]

Wenn es SQL-Anfragen sind, kann es vielleicht sein, dass beide auf das gleiche Connection-Object zugreifen?
Das führt dann natürlich zu fehlern, daher immer für jeden Thread ein eigenes Connection-Object erstellen.

 

[Tobias]

djui, wir haben null Code von dir gekriegt, daher können wir auch nur ins Blaue raten: Machst du Connection-Pooling? Benutzt du statische Variablen? Singleton-Klassen?

mpG
Tobias

 

[Campino]

Ich dachte daran, dass es in den Threads vllt. mehrere Stellen gibt wo in die Textfelder geschrieben wird, so ala if(...){ textfeld1.setText(...)} else if(...){textfeld1.setText(...)} und der Tippfehler ist nur in einem der "Arme" des if-Konstrukts, dann tritt der Fehler nicht immer auf. Währe halt das erste was ich prüfen würde...

 

[Tobias]

Jo, wäre so gesehen eine Möglichkeit.

mpG
Tobias

 

[Beni]

Hat zwar nur indirekt mit dem Problem hier zu tun, aber Swing (bzw. AWT) ist nicht threadsicher!
Wenn man nicht aus dem EventDispatcher-Thread auf eine JTextArea schreibt, kann einem das Ding irgendwann um die Ohren fliegen.
Am einfachsten kann man die Schreibbefehle in ein eigenes "Runnable" kapseln, und damit die Methode "EventQueue.invokeLater" aufrufen. Oder man benutzt den "SwingWorker".