Un thread Java peut-il être arrêté en toute sécurité en retournant dans la méthode run()?

Question

Je sais que Thread.stop() et d'autres fonctions sont obsolètes, et je veux empêcher un thread de s'exécuter si une certaine condition est remplie. Voici une version simplifiée de ce que je fais maintenant:

public class SomeThread extends Thread 
{ 
    private static boolean running = true; 

    public static void shutdown() 
    { 
     running = false; 
    } 

    public void run() 
    { 
     while(running) 
     { 
      // do something cool 
     } 

     return; // <- this is what I'm wondering about 
    } 
} 

Lorsqu'un autre classe appelle la méthode shutdown() de SomeThread, les sorties de boucle while, puis la méthode retourne run(). Est-ce sûr? La principale raison pour laquelle j'utilise une boucle while au lieu de simplement laisser exécuter run() est parce qu'il y a quelques choses que je ne veux faire qu'une seule fois (avant la boucle while), et je ne suis vraiment que utiliser le thread pour la concurrence. Je suis sûr que c'est une bastardisation de ce que les Threads étaient destinés, mais je m'interroge principalement sur la déclaration de retour.

Answer 1

Le retour est bon, le reste de votre code est incertain.

Le membre running doit être soit volatile, soit vous avez besoin d'une synchronisation autour des lectures et écritures sur cette variable.

(vu shutdown et running comme statique est inhabituel.)

Answer 2

utiliser le mécanisme d'interruption intégré de fils à la place d'un champ

public class SomeThread extends Thread { 
    public void run() { 
     while(!isInterrupted()) { 
      // do something 
     } 
    } 
} 

D'un autre fil, appeler interrupt() sur le fil afin pour l'éteindre. Cela a l'avantage supplémentaire que cela fonctionnera même si votre thread attend sur un moniteur.