while-Schleife bricht nicht ab

[MK234]

Hallo,

ich schreibe ein Programm, in welchem ein Thread gestartet wird und das Programm in eine while-Schleife läuft, diese prüft bei jedem Durchlauf den boolean-Wert einer Variablen "simuLaeuft",
wird dieser Wert durch den Thread auf false gesetzt, soll die Schleife abbrechen, leider tut sie das nicht.


der Programmausschnitt der Klasse Zeichenflaeche mit der while-Schleife und in starten des Threads gestartet wird

                                th = new Simulation(this);
                                th.start();
                                simuLaeuft = true;
                                while(simuLaeuft) {                                                                       
                                    //do nothing
                                }                           
                                                                
                                th.interrupt();

run-Methode des gestarteten Threads

 public void run() {
        while(!isInterrupted()) {
            try {
                   sleep(2000);
                   zeichenfl.simuLaeuft = false;
              
            } catch (InterruptedException e) {
                interrupt();
               
            }
        }
    }

Woran liegt das?

 

[faetzminator]

Wurde [c]simuLaeuft[/c] als [c]volatile[/c] deklariert? Dann sollte es funktionieren.

 

[MK234]

Nein simuLaeuft wurde nicht als volatile deklariert.

Wenn ich

volatile boolean simuLaeuft = false;

statt

boolean simuLaeuft = false;

nehme bricht die Schleife leider auch nicht ab.

 

[MK234]

sorry, doch sie bricht mit volatile ab.

Danke, hab wieder was dazu gelernt

 

[xehpuk]

Das hier ist jetzt aber nicht der Original-Code, oder?

while(simuLaeuft) {                                                                       
    //do nothing
}

 

[MK234]

doch vorerst schon, aber er ist weiter in Bearbeitung.

 

[faetzminator]

In solch einer CPU-Zeit fressenden Schleife solltest zu zumindest ein Mal pro Durchlauf ein [c]Thread.yield()[/c] machen, um den Thread "verschnaufen" zu lassen bzw. andere Threads an die Reihe zu bringen.

 

[xehpuk]

Wie bereits [thread=129677]hier[/thread] beschrieben, ist

Thread.yield()

nutzlos.
Wie der Code momentan aussieht, brauchst du den zweiten Thread gar nicht.

 

[faetzminator]

@xehpuk: Klar, denn normalerweise ist so etwas ein Designfehler Aber nutzlos kann man nicht sagen, denn ein Programm mit [c]while(true) {}[/c] frisst viel mehr CPU-Zeit (logischerweise die gesamte verfügbare). Aber willst du echt in einem Anfängerthema den Leuten erklären, wie man ein komplexes Programm richtig designed?

 

[xehpuk]

Nun ja, lieber den TE zum richtigen Code leiten, statt ihm einen "korrekten" Workaround zu geben.
Und deinen Workaround würde ich nicht mal als korrekt bezeichnen. :bae: