为什么我的 cyclicBarrier 为空？

Question

我正在学习 ciclycbarrier 并且正在尝试创建一个小应用程序。我的应用程序的构造函数如下：

public FileDownloader(String line, int maxDownload){
    this.position = 0;
    this.line = line;
    this.maxDownload = maxDownload;
    this.urls = new ArrayList<String>();
    this.ths = new ArrayList<Thread>();
    this.exm = new Semaphore(1);
    this.GenerateURLS();
    final CyclicBarrier cb = new CyclicBarrier(this.maxDownload, new Runnable(){
        @Override
        public void run(){


            System.out.println("All download are finished");
            //Mergear cuando se termina
            //Borrar cuando se termina
        }

    });

    for(int i=0; i<this.maxDownload;i++){
        ths.add(new Thread(new Task(this.cb),"Hilo"+i));
    }
    for(Thread th: ths){
        th.start();
    }

}

在构造函数中，我创建了我的 Cyclicbarrier，设置了一个 maxDownload 数量和一个新的运行nable。在那之后，y 创建我所有的线程设置一个任务（设置循环屏障。任务实现运行nable）。我的任务代码如下：

class Task implements Runnable{
    private CyclicBarrier barrier;
    public static int position;
    public Task(CyclicBarrier cb){
        this.barrier = cb;

    }

    public void run(){
        try {

            FileDownloader.DownloadManager();
            this.barrier.await();
        } catch (InterruptedException | BrokenBarrierException e) {
            // TODO Auto-generated catch block
            e.printStackTrace();
            System.out.println(e.getStackTrace());
        }
    }
}

但问题是当方法 DownloadFile（在我的任务运行内）结束时，是时候执行 cb.await，我有下一个错误：

Exception in thread "Hilo1" java.lang.NullPointerException
    at Prac2.Task.run(FileDownloader.java:23)
    at java.lang.Thread.run(Thread.java:745)

调试我可以看到我的任务中的 cyclicbarrier(barrier) 总是空的，但 cb 不是。

可能是什么问题？

Answer 1

因为您在 FileDownloader 构造函数中创建了一个本地 cb 变量，但是您将未初始化的 this.cb 传递给 Task 构造函数。

Answer 2

仔细查看您的代码。

您创建局部变量 cb。

 final CyclicBarrier cb = new CyclicBarrier(this.maxDownload, new Runnable(){
    @Override
    public void run(){
        System.out.println("All download are finished");
        //Mergear cuando se termina
        //Borrar cuando se termina
    }

});

但是这里你访问的是class级变量。

for(int i=0; i<this.maxDownload;i++){
    ths.add(new Thread(new Task(this.cb),"Hilo"+i));
}

我的意思是：

this.cb

留心。

为什么我的 cyclicBarrier 为空？

Why my cyclicBarrier is null?

java

multithreading

concurrency

java.util.concurrent

cyclicbarrier