问题陈述是: -
每个线程使用1到1000之间的唯一ID,程序必须运行60分钟或更长时间,因此在60分钟内,所有ID都可能完成,所以我需要再次重用这些ID,
我知道有几种方法可以做到这一点,下面是我在StackOverflow的帮助下编写的,但是当我尝试运行它时,我发现,经过几分钟的运行后,这个程序变得很慢而且它需要花费大量时间在控制台上打印ID。而且我有时会得到OutOfMemory Error。有没有更好的方法来解决这类问题?
class IdPool {
private final LinkedList<Integer> availableExistingIds = new LinkedList<Integer>();
public IdPool() {
for (int i = 1; i <= 1000; i++) {
availableExistingIds.add(i);
}
}
public synchronized Integer getExistingId() {
return availableExistingIds.removeFirst();
}
public synchronized void releaseExistingId(Integer id) {
availableExistingIds.add(id);
}
}
class ThreadNewTask implements Runnable {
private IdPool idPool;
public ThreadNewTask(IdPool idPool) {
this.idPool = idPool;
}
public void run() {
Integer id = idPool.getExistingId();
someMethod(id);
idPool.releaseExistingId(id);
}
private void someMethod(Integer id) {
System.out.println("Task: " +id);
}
}
public class TestingPool {
public static void main(String[] args) throws InterruptedException {
int size = 10;
int durationOfRun = 60;
IdPool idPool = new IdPool();
// create thread pool with given size
// create thread pool with given size
ExecutorService service = new ThreadPoolExecutor(size, size, 500L, TimeUnit.MILLISECONDS, new ArrayBlockingQueue<Runnable>(10), new ThreadPoolExecutor.CallerRunsPolicy());
// queue some tasks
long startTime = System.currentTimeMillis();
long endTime = startTime + (durationOfRun * 60 * 1000L);
// Running it for 60 minutes
while(System.currentTimeMillis() <= endTime) {
service.submit(new ThreadNewTask(idPool));
}
// wait for termination
service.shutdown();
service.awaitTermination(Long.MAX_VALUE, TimeUnit.DAYS);
}
}
答案 0 :(得分:4)
我在前一个问题中已经向您解释过,您的代码向执行程序提交了数百万个任务,因为它在60分钟内循环提交任务,没有等待。
目前还不清楚你的最终目标是什么,但是你正在填补任务队列,直到你再也没有任何可用内存为止。既然你没有解释你的程序的目标,很难给你任何解决方案。
但您可以做的第一件事是限制执行程序的任务队列的大小。这会在每次队列满时强制主线程阻塞。