多线程代码中不通知工作线程

时间:2016-03-09 02:11:26

标签: java multithreading

我想用Java编写传统的生产者和消费者多线程程序。生产者线程将向线程安全列表发送消息,直到此列表变满为止。一旦缓冲区已满,它将通知工作线程,缓冲区将被清除。在我的编码中,不通知工作线程。

你知道原因吗?谢谢。

package com;
import java.util.ArrayList;
import java.util.Collections;
import java.util.List;
import java.util.concurrent.locks.Condition;
import java.util.concurrent.locks.ReentrantLock;

public class ThreadTest {

    public static void main(String[] args) {

        final List<String> bufferSafeList = Collections.synchronizedList(new ArrayList<String>());

        final ReentrantLock bufferLock = new ReentrantLock();
        final Condition bufferFull = bufferLock.newCondition();
      //  final Condition bufferEmpty = bufferLock.newCondition();

        Thread producerThread = new Thread(new Runnable() {
            @Override
            public void run() {
                while (true) {
                    try {
                        bufferLock.lock();
                        bufferSafeList.add(System.currentTimeMillis() + "");
                        System.out.println("add to buffer " + bufferSafeList.size());
                        if (bufferSafeList.size() > 100) {
                            System.out.println("send wake up signal");
                            bufferFull.signalAll();

                            //waiting for buff cleared
                            while(!bufferSafeList.isEmpty()){
                                Thread.sleep(1000);
                            }                           
                        }

                         Thread.sleep(1000);
                    } catch(Exception e){
                        e.printStackTrace();
                    }
                    finally {
                        bufferLock.unlock();
                    }
                }
            }
        });
        producerThread.start();

        Thread workerThread = new Thread(new Runnable() {
            @Override
            public void run() {
                while (true) {
                    try {
                        bufferLock.lock();
                        System.out.println("waiting for wakeup signal");
                        bufferFull.await();
                        System.out.println("clear buffer");

                        bufferSafeList.clear();                      
                    } catch (Exception ex) {
                        ex.printStackTrace();
                    } finally {
                        bufferLock.unlock();
                    }
                }
            }
        });
        workerThread.start();
    }

}

2 个答案:

答案 0 :(得分:0)

您最后只有unlock bufferLock,因此workerThread永远无法比尝试获取lock

更进一步

当缓冲区已满unlock时,workerThread可以继续

答案 1 :(得分:0)

如果生成器线程碰巧首先运行,因为它可能会先运行,因为它首先被启动,它可能会首先锁定bufferLock,在这种情况下,它将填充列表并调用bufferFull.signalAll()而消费者仍在等待获得锁定。然后你忙着 - 等待工人清除列表,因为它无法继续,所以它永远不会做。

此外,即使您在发出信号后解锁,也为时已晚。当工人处于await()状态时,信号已经过去了。实际上,即使生产者在准备发出信号之前等待获得锁定,也存在竞争条件。

条件变量的正确用法总是涉及测试在等待之前是否满足预期条件,如果从等待恢复后不满足则循环返回等待一些。您可以通过实现此功能来克服代码中的竞争。