起因:同事代码手动创建线程
对同事的一段代码进行了优化
部署后,规约扫描报错“不能手动创建线程池”。
3.【强制】 线程资源必须通过线程池提供,不允许在应用中自行显式创建线程。
说明:使用线程池的好处是减少在创建和销毁线程上所花的时间以及系统资源的开销,解决资源不足的问题。如果不使用线程池,有可能造成系统创建大量同类线程而导致消耗完内存或者“过度切换”的问题。
修改1:使用Executors创建线程池
ExecutorService threadPool = Executors.newCachedThreadPool(); //创建大小可伸缩的线程池
try {
for (int i = 0; i < 30; i++) {
//使用线程池来创建线程
threadPool.execute(()->{
System.out.println(Thread.currentThread().getName()+"ok");
});
}
} catch (Exception e) {
e.printStackTrace();
} finally {
threadPool.shutdown(); //线程池使用完毕后需要关闭
}
部署后,规约扫描报错“不能使用Executors创建线程池”。
4.【强制】 线程池不允许使用Executors去创建,而是通过ThreadPoolExecutor的方式,这样的处理方式让写的同学更加明确线程池的运行规则,规避资源耗尽的风险。
说明:Executors返回的线程池对象的弊端如下:
1)FixedThreadPool和SingleThreadPool:
允许的请求队列长度为Integer.MAX_VALUE,可能会堆积大量的请求,从而导致OOM。
2)CachedThreadPool
允许的创建线程数量为Integer.MAX_VALUE,可能会创建大量的线程,从而导致OOM。
修改2:改用ThreadPoolExecutor创建线程池
ThreadFactory namedThreadFactory = new ThreadFactoryBuilder().setNameFormat("demo-pool-%d").build();
ThreadPoolExecutor threadPool = new ThreadPoolExecutor(
2,
size,
3,
TimeUnit.SECONDS,
new ArrayBlockingQueue<>(3),
namedThreadFactory,
new ThreadPoolExecutor.DiscardOldestPolicy()
);
List<FutureTask<Map<String, String>>> futureTasks = new ArrayList<>();
try {
for (int i = 0; i < size; i++) {
final int finalI = i+1;
long realInterval = interval != null ? interval * finalI : 10;
final FutureTask<Map<String, String>> task = new FutureTask<>(() ->
selfDefineFunction());
futureTasks.add(task);
threadPool.submit(task);
}
} catch (Exception e) {
e.printStackTrace();
} finally {
threadPool.shutdown(); //线程池使用完毕后需要关闭
}
futureTasks.forEach(task -> {
try {
Map<String, ShakeResultDTO> oneResult = task.get();
matchResultMap.putAll(oneResult);
} catch (InterruptedException | ExecutionException e) {
e.printStackTrace();
}
});
小结
1、手动创建-->Executors-->ThreadPoolExecutor
2、返回值submit、不带返回值execute