nullpointerexception?

8ljdwjyq  于 2021-06-27  发布在  Java
关注(0)|答案(1)|浏览(345)

我正在编写一段涉及线程的代码,并设置了以下for循环来示例化它们:

for (Buyer buyer : buyers) {
    isPrime = (rand.nextInt(10) + 1 < 3);
    buyer = new Buyer(warehouse,
                      isPrime,
                      "buyer" + ++i);
    buyer.start();
    System.out.println("started buyer: " + i);
}

之后,我将示例化一些其他线程,然后再次加入这些买家线程,等待程序完成:

System.out.println("Joining buyers. ");
for (Buyer buyer : buyers) {
    buyer.join();
    System.out.println("Joining thread: " + buyer.getName());
}

当我运行我的程序时,我在下面一行得到1个nullpointerexception: buyer.join(); 所有线程都完成了执行,但似乎没有一个线程希望在该点加入。这是怎么回事?
以下是买家线程的代码:

import java.util.Random;

public class Buyer extends Thread{
    private int packsBought = 0;
    private boolean isPrime;
    private Random rand;
    private Warehouse warehouse;

    public Buyer(Warehouse warehouse, Boolean isPrime, String name) {
        super(name);
        this.isPrime = isPrime;
        this.rand = new Random();
        this.warehouse = warehouse;
    }

    public void run() {
        while(this.packsBought < 10) {
            try {
                Thread.sleep(this.rand.nextInt(49) + 1);
            } catch (InterruptedException ex) {

            }
            Order order = new Order(this.rand.nextInt(3)+ 1, 
                                    this, 
                                    this.isPrime);
            this.warehouse.placeOrder(order);
        }
        System.out.println("Thread: " + super.getName() + " has finished.");
    }

    public int getPacksBought() {
        return this.packsBought;
    }

    public void setPacksBought(int packsBought) {
        this.packsBought = packsBought;
    }

    public boolean isPrime() {
        return isPrime;
    }  
}
2sbarzqh

2sbarzqh1#

问题在于:

for (Buyer buyer : buyers) {
    isPrime = (rand.nextInt(10) + 1 < 3);
    buyer = new Buyer(warehouse,          // <--- this is wrong 
                      isPrime,
                      "buyer" + ++i);
    buyer.start();
    System.out.println("started buyer: " + i);
}

您并没有真正初始化列表中的元素 buyers . 这是:

buyer = new Buyer(warehouse,
                          isPrime,
                          "buyer" + ++i);

不会更改保存在列表上的引用 buyers . 是的,您将创建并启动一组线程。但在:

System.out.println("Joining buyers. ");
for (Buyer buyer : buyers) {
    buyer.join();
    System.out.println("Joining thread: " + buyer.getName());
}

您没有对已创建并启动的线程(即买家)调用join。为了让你得到一个npe

buyer.join();

是因为你已经用 null ,认为可以在循环中初始化:

for (Buyer buyer : buyers) {
        isPrime = (rand.nextInt(10) + 1 < 3);
        buyer = new Buyer(warehouse,
                          isPrime,
                          "buyer" + ++i);
        buyer.start();
        System.out.println("started buyer: " + i);
    }

相关问题