de desbloqueo de ReentrantLock sin IllegalMonitorStateException

Question

tengo una pieza de código (simplificado):

if(reentrantLockObject.isLocked()) { 
     reentrantLockObject.unlock(); 
} 

donde reentrantLockObject es java.util.concurrent.locks.ReentrantLock. A veces recibo IllegalMonitorStateException. Se ve que el bloqueo se liberó entre la llamada de verificación y la de desbloqueo(). ¿Cómo puedo evitar esta excepción?

Answer 1

isLocked devuelve si cualquier hilo mantiene el bloqueo. Creo que quieres isHeldByCurrentThread:

if (reentrantLockObject.isHeldByCurrentThread()) { 
    reentrantLockObject.unlock(); 
} 

Dicho esto, isHeldByCurrentThread se documenta ser principalmente para fines de diagnóstico - que sería inusual para este trozo de código que es el enfoque correcto. ¿Puedes explicar por qué crees que lo necesitas?

Answer 2

Necesita tener el candado para poder desbloquearlo. reentrantLockObject.isLocked() solo es verdadero si algún hilo posee el bloqueo, no necesariamente usted.

reentrantLockObject.lock(); 
    try{ 

     // do stuff 
    }finally{ 
     reentrantLockObject.unlock(); 
    } 

Aquí el hilo posee la cerradura para que puedan desbloquearlo.

Answer 3

ReentrantLock lanza esta excepción según esta lógica:

if (Thread.currentThread() != getExclusiveOwnerThread()) { 
    throw new IllegalMonitorStateException(); 
} 

Así que la solución es comprobar si el mismo hilo es desbloquear:

if (reentrantLockObject.isHeldByCurrentThread()) { 
    reentrantLockObject.unlock(); 
}