Question

我试图创建一个测试，我试图强制竞争条件（或至少增加其发生的可能性）并且我使用了CountDownLatch。

问题是我在java.lang.IllegalMonitorStateException获得CountDownLatch.wait()。我肯定在滥用CountDownLatch，我肯定不会以聪明的方式创建这个测试。

这个简单的代码再现了我的想法和我的问题（我也有gist）：

import java.util.*;
import java.util.concurrent.*;

public class Example {

    private static BusinessLogic logic;

    public static void main(String[] args) {
        final Integer NUMBER_OF_PARALLEL_THREADS = 10;
        CountDownLatch latch = new CountDownLatch(NUMBER_OF_PARALLEL_THREADS);
        logic = new BusinessLogic();

        // trying to force the race condition
        List<Thread> threads = new ArrayList<Thread>(NUMBER_OF_PARALLEL_THREADS);
        for (int i=0; i<NUMBER_OF_PARALLEL_THREADS; i++) {
            Thread worker = new Thread(new WorkerRunnable(latch));
            threads.add(worker);
            worker.start();
        }

        for (int i = 1; i <= NUMBER_OF_PARALLEL_THREADS; i++) {
            try {
                threads.get(i).wait();
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }
    }

    /**
     * Just a dummy business logic class.
     * I want to "force" a race condition at the method doSomething().
     */
    private static class BusinessLogic {
        public void doSomething() {
            System.out.println("Doing something...");
        }
    }

    /**
     * Worker runnable to use in a Thead
     */
    private static class WorkerRunnable implements Runnable {
        private CountDownLatch latch;

        private WorkerRunnable(CountDownLatch latch) {
            this.latch = latch;
        }

        public void run() {
            try {
                // 1st I want to decrement the latch
                latch.countDown();
                // then I want to wait for every other thread to 
                latch.wait(); // the exception is thrown in this line.
                // hopefully increase the probability of a race condition...
                logic.doSomething();
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }
    }
}

CountDownLatch.wait()的javadoc声明如果当前线程不是对象监视器的所有者，则抛出IllegalMonitorStateException。但我担心我不明白这是什么意思，也无法弄清楚如何重新创建我的代码以避免这种异常。

编辑：通过答案中提供的提示，我已经创建了上述示例的新版本，并且我已存储在this gist中。我现在没有例外。

Answer 1

尝试await()，而不是wait()。

await()将等到锁存器达到零。 wait()与锁定无关，而不是您想要的WorkerRunnable。但是，仅供参考，为了在没有获得异常的情况下调用wait()，您必须拥有对象的监视器，并且要成为所有者，您必须位于该对象的synchronized块中。

Answer 2

您的控制器线程（通常是主/ ui线程）应该进行等待，而工作线程执行倒计时。

你应该从你的主线程开始线程并在那里插入latch.await()电话 - 就在你启动它们之后。每个工作线程应在完成时调用latch.countdown()。

当allthreads调用countdown()时，CountDownLatch将退出主线程中的latch.await()并将执行控制权转移给它（latch.await()之后的代码将开始执行）。

所以基本上你需要在你启动工作线程后立即将await()移动到主程序。

编辑：您还应该删除Thread.wait()调用，因为这是另一个多线程框架 - 等待/通知，它比使用CountDownLatch更低级别（除非您需要它用于模拟。我不要＆＃ 39;完全理解你的测试用例）

Answer 3

尝试调用wait()任何对象时，您必须拥有该对象的监视器。

Object o = new Object();
o.wait();

会导致IllegalMonitorStateException。

您必须在该对象上进行同步才能调用wait()：

Object o = new Object();
synchronized(o) {
    o.wait();
}

使用CowntDownLatch测试强制竞争条件会导致java.lang.IllegalMonitorStateException

3 个答案: