De thread en Java: Comment verrouiller un objet?

La Fonction suivante est en cours d'exécution dans son propre thread:

private void doSendData()
{
    try {

           //writeToFile(); //just a temporary location of a call
           InetAddress serverAddr = InetAddress.getByName(serverAddress);
           serverAddr.wait(60000);
           //Log.d("TCP", "C: Connecting...");
           Socket socket = new Socket(serverAddr, portNumber);
           socket.setSoTimeout(3000);

               try {
                //Log.d("TCP", "C: Sending: '" + message + "'");
                PrintWriter out = new PrintWriter( new BufferedWriter( new OutputStreamWriter(socket.getOutputStream())),true);
                String message = packData();
                out.println(message);
                Log.d("TCP", "C: Sent.");
                Log.d("TCP", "C: Done.");
                connectionAvailable = true;

             } catch(Exception e) {
                 Log.e("TCP", "S: Error", e);
                 connectionAvailable = false;

               } finally {
                  socket.close();
                  announceNetworkAvailability(connectionAvailable);
                }

         } catch (Exception e) {
              Log.e("TCP", "C: Error", e);
              announceNetworkAvailability(connectionAvailable);
         }
}

Lors de l'exécution atteint la ligne serverAddr.wait(60000) il déclenche une Exception:

java.lang.IllegalMonitorStateException: object not locked by thread before wait()

Personne ne sait comment verrouiller un objet ou une fonction afin d'éviter la simultanéité?
J'ai essayé d'ajouter un Verrou de l'objet:

private final Lock lock = new ReentrantLock();

et la ligne

boolean locked = lock.tryLock();

au début de la fonction, mais ça ne fonctionne pas.

  • Que voulez-vous faire? L'ATTENTE n'a aucun sens...
InformationsquelleAutor Niko Gamulin | 2009-03-19
  1. 64

    Pour faire un appel à wait() sur un objet, vous devez maintenir la synchronisation de verrou sur l'objet (si le verrou est réellement libérée tandis que le thread est en attente):

    synchronized (serverAddr) {
  serverAddr.wait();
}

    Je dois avouer que pourquoi vous êtes désireux de le faire me déroute dans ce cas...

    • Désolé, juste pour être clair: je comprends pourquoi vous devez acquérir le verrou sur l'objet avant de le en attente. Ce que je ne comprends pas, c'est le logique de ce que l'affiche originale est en train d'essayer de le faire...!!!
    • Oh, alors mon commentaire est rédigé 🙂
    • +1 pour "le verrou est réellement libérés tandis que le thread est en attente", m'a aidé à me fixe un autre problème et je n'étais pas au courant (oui, je suis nouveau sur java)
    • Je suppose que l'OP est de faire en sorte qu'une seule connexion à cette adresse est faite à un moment donné. Si c'est le cas, je prédis qu'il ne fonctionne toujours pas, car chaque thread voit un autre 'serverAddr' objet, bien que j'ai peut-être tort sur ce point.
    • Neil Coffey me dire comment puis-je résoudre stackoverflow.com/q/31966790/4260932
    InformationsquelleAutor Neil Coffey
  2. 21

    Peut-être la méthode que vous cherchez est Fil de discussion.sleep(long)? Cette méthode attendre (comme dans l'arrêt de l'exécution du thread) pour le temps spécifié en millisecondes, avant de reprendre.

    objet.attendre(longtemps) (qui est ce que vous utilisez) fait quelque chose d'entièrement différent. Il attend un autre objet à partir d'un autre thread pour l'informer de la (ie: envoyer une sorte de message de réveil), et attendre au plus le nombre de millisecondes spécifié. Étant donné que le code que vous avez posté, je doute fortement de ce est ce que vous voulez vraiment.

    Si Thread.sleep() n'est pas ce que vous voulez, alors vous devriez utiliser la synchronisation de bloc, comme mentionné par d'autres affiches.

    InformationsquelleAutor LordOfThePigs
  3. 11

    J'ai toujours grincer des dents quand je vois ce genre de code. Faites-vous une faveur et regarder le java.util.concurrent paquet.

    InformationsquelleAutor Apocalisp
  4. 1

    Pour éviter ce message d'erreur, utilisez le mot-clé synchronized:

    synchronized(serverAddr){
  serverAddr.wait(60000);
}
    InformationsquelleAutor krosenvold
  5. 1

    Le ci-dessus sont correctes. Vous pouvez utiliser en parallèle, un bloc de code. Ou vous pouvez créer ce qu'ils appellent un mutex. Un mutex peut effectivement être n'importe quel objet. Beaucoup de gens simplement utiliser de l'Objet lui-même comme un mutex. Ensuite, vous pouvez verrouiller le mutex. Tous les threads qui veulent obtenir l'accès doit attendre que le thread en tenant le mutex pour le libérer.

    Il a également été suggéré par Apocalisp. Je recommande également que vous regardez à la java.util.simultanées paquet.

    InformationsquelleAutor uriDium
  6. 0

    En général, quand vous avez le programme multi-threadé en Java, vous devez verrouiller la variable partagée par l'aide de la synchronisation (mot-clé) puis à tout moment juste un thread peut accéder à la mémoire partagée.

    InformationsquelleAutor Khash Nejad
  7. 0

    Code ci-dessous devrait fonctionner.

         private final ReentrantLock lock = new ReentrantLock();

     lock.lock();  //block until condition holds
     try {
        serverAddr.wait(60000);
     } finally {
       lock.unlock()
     }
   }

    Référence à cette documentation page pour plus de détails. 

    public void lock()

    Acquiert le verrou.

    Acquiert la serrure si elle n'est pas tenue par un autre thread et retourne immédiatement, le réglage du verrouillage compter jusqu'à un.

    Si le thread actuel est déjà titulaire de la serrure, puis la cale compteur est incrémenté par un et la méthode retourne immédiatement.

    Si le verrou est détenu par un autre thread le thread courant est désactivée pour fil le cadre de la planification et reste inactif jusqu'à ce que la serrure a été acquis, au moment où le verrouillage comte est un

    Référer à cette question SE connaître les avantages de lock sur synchronization:

    La synchronisation vs Verrouillage

    InformationsquelleAutor Ravindra babu