Question

我有一个标准的生产者消费者问题。生产者将数据放入消费者使用的堆栈（缓冲区）中。

我想有很多生产者和消费者。

问题是我想让最后一位活着的制片人能够致电b.stop()

for(int i = 0; i < 10; i++){
        try{
    //      sleep((int)(Math.random() * 1));                
        }catch(Exception e){e.printStackTrace();}
        b.put((int) (Math.random()* 10));
        System.out.println("i = " + i);
    }
    b.stop();

然后我致电b.stop()，将running中的Buffer字段更改为false并notifiesAll()

结束然后我得到：

i = 9 // number of iteration this is 10th iteration
Consumer 2.: no data to take. I wait.  Memory: 0
Consumer 1.: no data to take. I wait.  Memory: 0
Consumer 3.: no data to take. I wait.  Memory: 0

他们应该死了，所以我做了方法stop（）但它没有用。

代码正在运行，请检查

import java.util.Stack;


public class Buffer {
private static int SIZE = 4;
private int i;//number of elements in buffer
public Stack<Integer> stack;
private volatile boolean running;
    public Buffer() {
        stack = new Stack<>();
        running = true;
        i = 0;
    }
    synchronized public void put(int val){
        while (i >= SIZE) {
            try {
                System.out.println("Buffer full, producer waits");
                wait();
            } catch (InterruptedException exc) {
                exc.printStackTrace();
            }
        }   
        stack.push(val);//txt = s;
        i++;
        System.out.println("Producer inserted " + val + " memory: " + i);
        if(i - 1 == 0)
            notifyAll();
        System.out.println(stack);
    }

    public synchronized Integer get(Consumer c) {
        while (i == 0) {
            try {
                System.out.println(c + ": no data to take. I wait.  Memory: " + i);
                wait();
            } catch (InterruptedException exc) {
                exc.printStackTrace();
            }
        }   
        if(running){
            int data = stack.pop();
            i--;    
            System.out.println(c+  ": I took: " + data +" memory: " +  i);
            System.out.println(stack);
            if(i + 1 == SIZE){//if the buffer was full so the producer is waiting
                notifyAll();
                System.out.println(c +  "I notified producer about it");
        }
        return data;}
        else 
            return null;
    }

    public boolean isEmpty(){
        return i == 0;
    }
    public synchronized void stop(){//I THOUGH THIS WOULD FIX IT~!!!!!!!!!!!!!!
        running = false;
        notifyAll();
    }
    public boolean isRunning(){
        return running;
    }

}

public class Producer extends Thread {
private Buffer b;
    public Producer(Buffer b) {
        this.b = b;
    }

    public void run(){
        for(int i = 0; i < 10; i++){
            try{
        //      sleep((int)(Math.random() * 1));                
            }catch(Exception e){e.printStackTrace();}
            b.put((int) (Math.random()* 10));
            System.out.println("i = " + i);
        }
        b.stop();
    }

}

public class Consumer extends Thread {
    Buffer b;
    int nr;
    static int NR = 0;

    public Consumer(Buffer b) {
        this.b = b;
        nr = ++NR;
    }

    public void run() {
        Integer i = b.get(this);
        while (i != null) {
            System.out.println(nr + " I received : " + i);
            i = b.get(this);
        }
        System.out.println("Consumer " + nr + " is dead");
    }

    public String toString() {
        return "Consumer " + nr + ".";
}

}

public class Main {

    public static void main(String[] args) {

        Buffer b = new Buffer();
        Producer p = new Producer(b);
        Consumer c1 = new Consumer(b);
        Consumer c2 = new Consumer(b);
        Consumer c3 = new Consumer(b);  
        p.start();
        c1.start();c2.start();c3.start();

    }

}

Answer 1

你必须意识到你的线程可能在两个位置中的任何一个位置等待：

在wait循环中使用i == 0 - 在这种情况下，notifyall会将所有这些内容踢出去。但是，如果i仍为0，他们会再次直接等待。
等待对象的独占访问（即等待synchronized方法） - 在这种情况下（如果您修复了上面的问题1并且锁将被释放），他们将直接进入{{1} } loop。

我建议您将while (i == 0)循环更改为while ( i == 0 )。应该解决您的问题。由于您的while ( running && i == 0 )标志是（正确）running，所以应该整齐地退出。

Answer 2

在stop方法中，您将running设置为false，但您的while循环正在运行i == 0。将i设置为不等于零的值，它应该修复它。

顺便说一句，我不明白为什么你有一个running变量和一个单独的i变量，它实际上是保持一个线程运行的变量。

Answer 3

我会重新考虑你的设计。课程应该有一套连贯的责任;让一个类负责消耗队列中的对象，同时还负责关闭其他消费者，这似乎是你想要分离的东西。

Answer 4

回答只让最后一个生活的制作人能够调用b.stop（）。

您应该向AtomicInteger添加Buffer，其中包含生成器的数量，并在其构造函数中调用每个生成器调用b.start()（使其递增）。

这样你可以在b.stop()中减少它，并且只有当它变为零时才应running设置为false。

许多制片人和许多消费者。让最后一个生产者活着杀死消费者

4 个答案: