日期:2014-05-20  浏览次数:20733 次

java 中使用wait与notifyAll的奇怪的问题
Java code

import java.util.ArrayList;
import java.util.List;
import java.util.Queue;
import java.util.concurrent.LinkedBlockingQueue;

public class Producer {
    private final Queue<String> productQueue = new LinkedBlockingQueue<String> ();
    
    private final List<Consumer> consumers = new ArrayList<Consumer> ();
    
    public void addConsumer(Consumer c) {
        this.consumers.add(c);
    }
    
    public void notifyProductGenerated() {
        for(Consumer c : this.consumers) {
            c.notifyProductGenerated();
        }
    }
    
    public static void main(String... args) {
        Producer producer = new Producer();
        final Consumer consumer = new Consumer(producer.productQueue);
        
        producer.addConsumer(consumer);
        
        new Thread(new Runnable() {
            @Override
            public void run() {
                consumer.consume();
            }
        }).start();
        
        for(int i = 0; i < 100; i++) {
            try {
                Thread.sleep(1000 * 1);
            } catch(InterruptedException ex) {
                ex.printStackTrace();
            }
            
            producer.productQueue.add("" + i);
            System.out.println("" + i + " made.");
            
            producer.notifyProductGenerated();
        }
    }
}

class Consumer {
    private Queue<String> productQueue;
    
    public Consumer(Queue<String> productQueue) {
        this.productQueue = productQueue;
    }
    
    public synchronized void notifyProductGenerated() {
        notifyAll();
    }
    
    public synchronized void consume() {
        String product = null;
        
        while(true) {
            product = productQueue.poll();
            
            if(product == null) {
                try {
                    wait();
                    continue;
                } catch (InterruptedException ex) {
                    ex.printStackTrace();
                    return;
                }
            }
            
            try {
                Thread.sleep(1000 * 10);
            } catch(InterruptedException ex) {
                ex.printStackTrace();
                return;
            }
            
            System.out.println("run out of product " + product);
        }
    }
}



以上代码是模拟生产者与消费者。当生产者生成出产品后,将通知消费者,消费都进而消费。
代码中设计成消费者每一秒生产一个产品,而消费者每10秒才能消费一个产品。
所以,我期待的输出应该像这样(输出一):
0 made.
1 made.
2 made.
3 made.
4 made.
5 made.
6 made.
7 made.
8 made.
9 made.
10 made.
run out of product 0
11 made.
12 made.
...

然而输出的结果却是这样(输出二):
0 made.
1 made.
run out of product 0
run out of product 1
2 made.
3 made.
run out of product 2
run out of product 3
4 made.
...

似乎是生产者在生产两个产品后,就在等待消费者消费,待消费者消费完成后再行生产。

请各位大侠帮忙看看,如果要达到输出一的效果,代码中应该如何调整?

谢谢!

------解决方案--------------------
使用wait()与notify()/notifyAll()可以使得多个任务之间彼 此协作。

调用sleep()和yield()的时候锁并没有被释放,而调用wait() 将释放锁。这样另一个任务(线程)可以获得当前对象的锁,从而 进入它的synchronized方法中。可以通过notify()/notifyAll(), 或者时间到期,从wait()中恢复执行。

只能在同步控制方法或同步块中调用wait()、notify()和 notifyAll()。如果在非同步的方法里调用这些方法,在运行时会 抛出IllegalMonitorStateException异常。
------解决方案--------------------
粗看了下,消费者根本不需要再主动等十秒,本身是从动方
再说已经有wait了,只要生产者生产够了10个产品,就通知消费者消费,同时等待
消费完了消费者再通知生产者继续生产
------解决方案--------------------
我的做法:
1 根据线程同步规则,concume()执行时时不释放其同步对象的锁旗标的,所以在其sleep(10000)时间里,main 线程因要执行notifyProductGenerated()这个方法,而在等待池里等待锁旗标,而什么也干不了。(无法生产)。
2 根据上面的分析,关键是在consume等待时,不占有锁旗标,所以对consume()进行修改如下: