Benötigen Sie semaphore.relase() wenn semaphore.acquire() InterruptedException bekommt?

Question

Aus der Java java.util.concurrent.Semaphore docs war es mir nicht ganz klar, was passiert, wenn semaphore.acquire() den Thread blockiert und später durch eine InterruptedException unterbrochen wird. Wurde der Semaphorwert verringert und muss der Semaphor freigegeben werden?

Derzeit bin ich mit Code wie folgt:

try { 
    // use semaphore to limit number of parallel threads 
    semaphore.acquire(); 
    doMyWork(); 
} 
finally { 
    semaphore.release(); 
} 

Oder soll ich rufe eher nicht Release(), wenn ein InterruptedException während acquire auftritt()?

Answer 1

Aufruf release() wenn eine InterruptedException während acquire() auftritt?

Sie sollten nicht. Wenn .acquire() unterbrochen wird, wird das Semaphor nicht erworben, also sollte es wahrscheinlich nicht freigegeben werden.

Code sollte

// use semaphore to limit number of parallel threads 
semaphore.acquire(); 
try { 
    doMyWork(); 
} 
finally { 
    semaphore.release(); 
} 

Answer 2

werden, wenn der Faden vor acquire Methodenaufruf unterbrochen wird, oder während des Wartens auf eine Genehmigung zum Erwerb der InterruptedException geworfen werden und wird keine Genehmigung halten werden, so dass keine Notwendigkeit zu lösen. Nur wenn Sie sicher sind, dass eine Genehmigung erworben wurde (nach dem Aufruf der Methode acquire), müssen Sie die Genehmigung freigeben. Also besser zu erwerben, bevor Sie Try-Block beginnt, so etwas wie:

sem.acquire(); 
try{ 
    doMyWork(); 
}finally{ 
    sem.release(); 
} 

Answer 3

nos die akzeptierte Antwort teilweise richtig ist, mit der Ausnahme semaphore.acquire() auch wirft InterruptedException. Also, um 100% korrekt zu sein, würde der Code wie folgt aussehen:

try { 
    semaphore.acquire(); 
    try { 
     doMyWork(); 
    } catch (InterruptedException e) { 
     // do something, if you wish 
    } finally { 
     semaphore.release(); 
    } 
} catch (InterruptedException e) { 
    // do something, if you wish 
}