comportement avec plusieurs threads Peculiar/variables volatiles/boucles/conditionals (Java)

Question

Voici un extrait d'un serveur Web que je suis en train de construire ...

// ...

threadPool = Executors.newCachedThreadPool(); 
while (true) 
    if(this.isOn) { 
       try { // listen for incoming connection 
       this.clientSocket = serverSocket.accept(); 
     } catch (IOException e) { 
        System.err.println("LOG: >> Accept failed! "); 
        System.exit(1); 
        } 
     // as soon as a connection is established send the socket 
     // with a handler/processor to the thread pool for execution 

        threadPool.execute(new ClientRequestProcessor(clientSocket)); 
     } 

// ...

Veuillez noter que la variable isOn est un booléen VOLATILE.

Si j'allume le si dans un certain temps ... ce code fonctionne ... mais comme il est pas. Puis-je demander pourquoi? D'un point de vue logique, les deux devraient fonctionner, même si je teste ce drapeau dans un si ... ai-je raté quelque chose ?!

[Modification postérieure:] En ne fonctionnant pas, je veux dire ... un navigateur (par exemple firefox) ne peut pas se connecter, en fait il continue à essayer mais il finira par s'arrêter. Encore une fois, si je change ça si (isOn) en un temps (isOn) cela fonctionne comme un charme.

Toutes les suggestions/idées sont plus que bienvenues !!!

P.S. J'ai besoin de ce combo "while (true) if/while (drapeau de test) {...}" parce que le serveur peut être démarré/arrêté à partir d'une interface graphique ... donc le niveau supérieur alors que (vrai) est nécessaire peut revérifier si je suis sur (et donc écouter les connexions) ou si je suis éteint (et ne se soucient pas vraiment des connexions entrantes). Les aiguilles indiquant que les gestionnaires d'événements de l'interface graphique peuvent modifier le drapeau à tout moment.

Answer 1

Une meilleure solution consiste à fermer le socket du serveur lorsque vous voulez qu'il s'arrête et en lancer un nouveau dans un nouveau thread lorsque vous voulez le démarrer. De cette façon, vous rejetez les nouvelles connexions et ne consommez pas de CPU lorsque vous ne faites rien.

Lorsque isOn == true et que vous le désactivez, il n'acceptera que les connexions après la nouvelle connexion suivante. (Peut être n'importe quand plus tard) De plus, toute nouvelle connexion client attendra simplement que l'acceptation soit appelée (ou éventuellement expirée) Vous pouvez avoir jusqu'à 50 connexions en attente d'être acceptées par défaut.

Lorsque isOn == false, votre thread sera occupé, consommant un CPU. Je vous suggère de mettre un peu de retard comme Thread.sleep (250). Cela va couper le processeur de façon spectaculaire, mais ne retardera pas trop le démarrage.

BTW:

• si vous obtenez une exception, vous devez vous connecter/imprimer. Sinon, quand cela échouera, vous ne saurez pas pourquoi.
• Si l'acceptation échoue, il se peut que le processus soit en dehors des fichiers, vous ne voulez donc pas qu'il meure simplement, en supprimant toutes les connexions existantes.

Answer 2

Si vous avez le while (true) et si if (this.isOn) la boucle while n'a aucun moyen d'arrêter. Que se passe-t-il lorsque le drapeau isOn est mis à false. La boucle while ne s'arrête jamais car elle est essentiellement faite pour être infinie. Branchez une autre instruction pour la faire sauter et cela devrait fonctionner comme prévu.

Si vous supprimez l'instruction if et que vous la faites juste while (this.isOn), lorsque l'indicateur isOn est mis à false, la boucle se termine. Pas de boucle infinie.

Ce sont mes pensées au premier coup d'œil ...

Answer 3

Mon hypothèse serait basée sur votre là boucle serrée. Est-il possible que vous ayez plusieurs instances du programme en cours d'exécution sur votre serveur? La version if (isOn) ne s'arrêtera pas si vous définissez isOn sur false, mais bouclez simplement votre CPU.