ExecutorService应该在bean的整个生命周期内都是活动的，或者应该具有狭窄的（方法）范围

Question

我有一个方法，它必须处理数千个文档。

正常处理需要时间，因此我使用ExecutorService通过让多个线程并行工作来减少处理时间。

我必须使用这种方法来处理两个长列表。

1. 处理执行器服务的方法1：

现在，我已经在方法中初始化了ExecutorService，并用相同的方法关闭了它。（称为两次）

因此，它会导致创建和关闭ExecutorService两次。

  private List<DocDTO> convertData(args) {
    ExecutorService service = Executors.newFixedThreadPool(threads);

    List<Future<Set<DocDTO>>> futures = new ArrayList<Future<Set<DocDTO>>>();

    for (Pro pro : pros.getPros()) {
        Callable<Set<DocDTO>> callable = new Callable<Set<DocDTO>>() {
            public Set<DocDTO> call() throws Exception {        
                for loop code
            }
        };
        futures.add(service.submit(callable));
    }

    service.shutdown();
    for (Future<Set<DocDTO>> future : futures) {
        if (future.get() != null) {
            list.addAll(future.get());
        }
    }   
}   

我在看下面的链接：

http://programtalk.com/java/executorservice-not-shutting-down/

作者已在类级别创建了ExecutorService。

你能告诉我在方法级或类级创建ExecutorService是否是个好主意吗？

仅供参考，上述代码中的方法一天执行两次

Answer 1

感谢Maleen和GhostCat的宝贵意见，我放弃了使用多线程的计划，原因有两个：

>

当从单线程和多线程运行时，我从相同的方法中得到不同的结果

ExecutorService不适用于惰性关系。因此，将实体的加载从懒惰更改为急切可能会影响其他功能的性能

相反，我优化了加载实体的方式，而不是使用多线程。以前，我在for循环中逐个加载数据库中的所有静态数据，并将其保存在EhCache中。

更改后，我使用findall方法一次加载所有静态数据，并保存在map中，从map in for循环中访问相同的数据。它大大提高了方法的性能。再次感谢你们俩。

Answer 2

首先，我建议您阅读本文档，了解如何使用spring来启用TaskExecutor服务。正如GhostCat正确建议的，它应该在类级别。此外，我建议为它创建一个单独的bean。

前任：

@Bean
public TaskExecutor taskExecutor() {
    ThreadPoolTaskExecutor executor = new ThreadPoolTaskExecutor();
    executor.setCorePoolSize(5);
    executor.setMaxPoolSize(10);
    executor.setQueueCapacity(25);
    return executor;
}

我从这个答案中得到了答案，但你们可以在互联网上找到很多不同的配置。

Answer 3

自然的选择是将此类服务作为一个类字段。

建立这样的服务不是免费的。您花费了“构建”线程池的开销；以及正确关闭它的时间——每次调用该方法。

很多时候，那不是你想要的。你想花一次时间（比如：当保存服务的对象被创建时）。

一个缺点是：现在持有服务的类的“生命周期”很重要。因为您现在可能需要担心显式调用该holder类所拥有的服务。

换句话说：当您不得不担心性能时，您可能会将服务更改为类的一个字段。

但是：您明确声明每天调用此方法两次。那么您绝对不需要担心创建/关闭该服务所产生的开销。即使开销会导致1秒的计算时间——每天2秒也没关系。

因此，鉴于这一要求，我（个人）的建议是：保持代码原样。除非有真正的问题需要解决，否则不要触摸正在运行的系统。