0

我正在尝试编写多线程程序的一部分，其中固定线程池中的每个线程都尝试从队列中获取对象，如果队列为空，则线程等待。

我遇到的问题是程序使用的内存不断增加。

public class Ex3 {

public static LinkedBlockingQueue<Integer> myLBQ = new LinkedBlockingQueue<Integer>(10);

public static void main(String argc[]) throws Exception {
    ExecutorService executor = Executors.newFixedThreadPool(3);
    myLBQ.add(new Integer(1));
    for (;;) {
        executor.execute(new MyHandler(myLBQ));
    }
}
}

class MyHandler implements Runnable {

LinkedBlockingQueue<Integer> myLBQ;

MyHandler(LinkedBlockingQueue<Integer> myLBQ) {
    this.myLBQ = myLBQ;
}

public void run() {
    try {
        myLBQ.take();
    } catch (InterruptedException e) {
        e.printStackTrace();
    }
}
}

我不明白为什么 executor.execute 在线程应该等待将项目添加到队列时继续触发。如何修改我的代码以反映这一点？

4

2 回答 2

1

这会尽可能快地向执行器添加任务。

for (;;) {
    executor.execute(new MyHandler(myLBQ));
}

这将消耗大约 200 MB 每秒。它与是否有任务要执行没有任何关系。

如果您不想这样做，我建议您将循环移动到可运行对象并仅添加一个。这将导致它永远等待任务。

更好的方法是使用 ExecutorService 的内置队列来排队任务。

ExecutorService executor = Executors.newFixedThreadPool(3);
final int taskId = 1;
executor.submit(new Runnable() {
    @Override
    public void run() {
        doSomething(taskId);
    }
});

executor.shutdown();

这做同样的事情，但更简单的恕我直言。

于 2011-07-11T15:11:57.540 回答
0

这是因为您正在创建大量 MyHandler 实例并将它们插入到执行程序的内部队列中。

那个无限的for循环是相当卑鄙的。

于 2011-07-11T15:11:11.623 回答