Java可重入锁的正确锁定/解锁用法

Question

我有一个名为“H2O问题”的家庭作业，我应该实现一个名为H2OBarrier的类，它有3种方法。

• HReady，氢原子（线程）就绪时称为的方法
• OReady，氧原子（线程）就绪时调用的方法
• makeWater，当2个氢原子和1个氧原子准备就绪时称为方法

我应该使用Java可重入锁和条件来执行此操作。

这是我目前为止的代码，我想知道我是否正确地使用了lock和unlock。

public class H2OBarrier {

int hCount;
int oCount;

Lock lock = new ReentrantLock();
Condition hWait = lock.newCondition();
Condition oWait = lock.newCondition();

public void HReady() {
    lock.lock();
    hCount++;

    try {
        hWait.await();
    } catch (InterruptedException e) {
        e.printStackTrace();
    }

}

public void OReady(){
    lock.lock();
    oCount++;

    try {
        while(hCount<2 && oCount<1){
            oWait.await();
        }   
    } catch (InterruptedException e) {
        e.printStackTrace();
    } finally {
        makeWater();
    }


}

public void makeWater(){
    hCount--;
    hWait.signal();
    lock.unlock();

    hCount--;
    hWait.signal();
    lock.unlock();

    oCount--;
    oWait.signal();
    lock.unlock();

}

} 

我应该在我的 makeWater 方法以外的任何地方调用解锁 （）吗？整个程序的流程对我来说似乎非常合乎逻辑，我只是想确保我正在做的事情总体上看起来是正确的。

Answer 1

您的代码正在产生死锁。想象一下，5个原子首先通过，5个原子进入由wacel（）产生的o队列。现在，如果2个h原子通过并不重要，由于您的代码，所有h原子都将自动等待。