当我在CyclicBarrier回调中调用join时,应用程序挂起

时间:2015-08-10 07:20:22

标签: java multithreading concurrency cyclicbarrier

我在启动线程时有以下方法测试:

public static void main(String[] args) throws InterruptedException {
        List<Thread> threads = new ArrayList<>();
        final CyclicBarrier cyclicBarrier = new CyclicBarrier(1);
        Thread thread = new Thread(new CallbackThread(cyclicBarrier, threads));
        threads.add(thread);
        thread.start();          
    }

CallBack线程如下所示:

class CallbackThread implements Runnable {
    CyclicBarrier cyclicBarrier;
    List<Thread> threads;

    CallbackThread(CyclicBarrier cyclicBarrier, List<Thread> threads) {
        this.cyclicBarrier = cyclicBarrier;
        this.threads = threads;
    }

    @Override
    public void run() {
        try {
            cyclicBarrier.await();
        } catch (InterruptedException e) {
            e.printStackTrace();  //To change body of catch statement use File | Settings | File Templates.
        } catch (BrokenBarrierException e) {
            e.printStackTrace();  //To change body of catch statement use File | Settings | File Templates.
        }
        System.out.println("Threads started");
        for (Thread thread1 : threads) {
            try {
                thread1.join();
            } catch (InterruptedException e) {
                e.printStackTrace();  //To change body of catch statement use File | Settings | File Templates.
            }
        }
        System.out.println("Threads finished");

    }
}

当我运行应用程序时,我看到以下输出:

Threads started

并且应用程序挂起。

我不明白为什么。

如果要将连接逻辑替换为主方法 - 所有商品都很好。

public static void main(String[] args) throws InterruptedException {
        List<Thread> threads = new ArrayList<>();
        final CyclicBarrier cyclicBarrier = new CyclicBarrier(1);
        Thread thread = new Thread(new CallbackThread(cyclicBarrier, threads));
        threads.add(thread);
        thread.start();
        for (Thread thread1 : threads) {
            try {
                thread1.join();
            } catch (InterruptedException e) {
                e.printStackTrace();  //To change body of catch statement use File | Settings | File Templates.
            }
        }
        System.out.println("Threads finished");

    }

你能解释一下这个区别  恕我直言,它应该工作相同。

2 个答案:

答案 0 :(得分:2)

第一个示例中的代码在其自己的线程上调用join。您将它添加到列表中,线程遍历列表并加入列表中的每个线程。

答案 1 :(得分:1)

您正在加入 self 。这就是程序永远不会结束的原因。

当您从主线程调用join()时,主线程正在尝试加入它创建的线程CallableThread。所以这是正确的。

但是当你加入CallableThread时,你传递的threads[]包含对自身的引用。所以它正在加入,而不会结束。