Java reenrantlock 解锁抛出 java.lang.IllegalMonitorStateException

Question

线程在执行前需要获取多个锁，所以我把所有的锁放到一个列表中，并迭代tryLock()它们，如果tryLock()成功，我将获取到的锁添加到一个名为acquiredLocks的列表中，所以当任何一个锁'tryLock () 失败，我可以解锁已经获取的锁，但是当回滚发生时，它会通过 ReenrantLock 抛出 java.lang.IllegalMonitorStateException，因为当前线程没有持有锁。 而且我检查了acquiredLocks列表，其中的一些锁确实被另一个线程获取了，但是我在将它添加到acquiredLocks列表之前调用了tryLock（）。

private final List<Lock> locks = new CopyOnWriteArrayList<>();
private final List<Lock> acquiredLocks = new CopyOnWriteArrayList<>();


/**
  * try to acquire all locks, if any lock is not acquired, roll back and release all already acquired 
  * locks
  */
private boolean tryAllLocks() {
        for (Lock lock : locks) {
            if (lock.tryLock()) {
                acquiredLocks.add(lock);
            } else {
                acquiredLocks.forEach(Lock::unlock);
                return false;
            }
        }
        return true;
}

 public PriceGenerationAggregate call() throws Exception {
        try {
            if (TenorConst.isSpotTenor(marketPriceCache.getTenor())) {
                this.priceGenerationAggregate = new SpotRootPriceGenerationAggregate(marketPriceCache, synchronizeForwardSpotPrice());
            } else {
                this.priceGenerationAggregate = new ForwardRootPriceGenerationAggregate(marketPriceCache);
            }

            for (GenerationProduct generationProduct : priceGenerationAggregate.getAllSubscribedPriceForPriceGeneration().getAllGenerationProducts()) {
                Lock lock = ProductLockCache.getLock(generationProduct.getCcyPair(), generationProduct.getTenor(), generationProduct.getVolume());
                locks.add(lock);
            }
            // try all the locks before the price generation
            while (!tryAllLocks()) {
                Thread.sleep(100);
            }

            generatePrices();

        } catch (Exception e) {
            e.printStackTrace();
        } finally {
            releaseAllLocks();
        }

        return priceGenerationAggregate;
    }

Answer 1

您忘记在解锁后清除已获取锁的列表。 解锁后，它们不再被获取，其他线程可以锁定它们：正如您提到的，您看到来自acquiredLocks的锁被其他线程持有。

添加一个acquiredLocks.clear(); 解锁后的声明（这仅在acquiredLocks为每个线程都有一个新的列表实例的假设下有效）。

private boolean tryAllLocks() {
    for (Lock lock : locks) {
        if (lock.tryLock()) {
            acquiredLocks.add(lock);
        } else {
            acquiredLocks.forEach(Lock::unlock);
            acquiredLocks.clear(); // <-- added
            return false;
        }
    }
    return true;
}