CPU杀手

先看下面一段代码，有没有发现问题

 

			for (;;) {
				if (taskManager.size() > 0) {
					TransData td;
					try {
						logger.debug("等待获取任务.........");
						td = (TransData) taskManager.take();
						logger.debug("已获取一个任务.........");
						threadPool.execute(new SocketTaskThread( "SocketTaskThread-" + ato.getAndIncrement(), td,processManager));
					} catch (InterruptedException e) {
						// TODO Auto-generated catch block
						e.printStackTrace();
						logger.error(e.getMessage(),e);
					}
				}
			}

 上面当taskManager.size() == 0时，以上代码就会不断循环，CPU的一个线程就会被用光了。

 

正确的用法是把if (taskManager.size() > 0) {}条件删除，

其实td = (TransData) taskManager.take();就会等待数据返回，

当taskManager.size() == 0时会一直等待，当taskManager.size() > 0时，就会激活，并执行下面的代码。