einen Thread beim Schlafen überraschen und abschießen

[Generic1]

Hallo,

ich habe unteres Programm, in dem ein Thread etwas macht, in diesem Fall etwas auf die Konsole ausgeben. Nach jeder Ausgabe wartet dieser Thread 1sec. Ein zweiter Thread schießt den ersten Thread ab. Meine Frage wäre jetzt, wie kann ich den ersten Thread sicher abschießen, ohne eine Exception zu bekommen. Geht das irgendwie in der Richtung, dass ich vor dem Abschießen den Thread wieder aufwecke und dann in die ewigen Jagtgründe schicke oder muss ich mit der Exception java.lang.InterruptedException: sleep interrupted leben?
Besten Dank.

package threadsleeptest;

public class Main {

    public Main() {
        final Runnable runnableRUN = new Runnable() {
            public void run() {
                while(true) {
                    System.out.println("Ich laufe im RUN");
                    try {
                        Thread.sleep(1000);
                        }
                    catch(Exception e) {
                        e.printStackTrace();
                        }
                    }
                }
            };
        final Thread thRUN = new Thread(runnableRUN);
        thRUN.start();

        final Runnable runnableINTERRUPT = new Runnable() {
            public void run() {
                while(true) {
                    thRUN.interrupt();
                    }
                }
            };
        final Thread thINTERRUPT = new Thread(runnableINTERRUPT);
        thINTERRUPT.start();
        }

    public static void main(String[] args) {
        final Main main = new Main();
        }
}

 

[Gast2]

Wenn du den Thread sauber beenden willst wäre es ein Möglichkeit die while Bedingung umzusetzen:

    [...]
            boolean stopped = false;
            public void run() {
                while(!stopped) {
                    System.out.println("Ich laufe im RUN");
                    try {
                        Thread.sleep(1000);
                        }
                    catch(Exception e) {
                        e.printStackTrace();
                        }
                    }
                }
            };

public synchronized void stopThread(){
    this.stopped = true;
}

Dann über stopThread den Thread anhalten.

 

[FArt]

@fassy: beliebter Fehler - das Auslesen deines Flags ist nicht synchronisert... das Flag sollte volatile deklariert sein.

@Generic1: der beste Weg einen Thread zu unterbrechen ist mit interrupt... und eine Exception ist nichts schlimmes, schon gar nicht eine InterruptedException. Und "interrupt" verwendet bereits ein Flag, wieso also ein extra Flag bauen, vor allem wenn es die meisten Leute nicht richtig machen und diese Flag keine blockierten Threads unterbricht?

 

[Gast2]

Das Flag ist gar nicht deklariert wie du vielleicht gesehn hast, aber ich gebe dir Recht - es soll volatile sein.