使用 ScheduledExecutorService 执行具有多个线程的 Runnable lambda

Question

我有一个方法 start() 可以创建一个带有 lambda 的 Runnable。在该方法中，我启动了一个使用该 Runnable 的 ScheduledExecutorService。我认为如果只使用 1 个线程来执行任务，我不会有问题，但是如果我启动多个线程并在内部传递相同的 Runnable 会发生什么。示例代码如下：

public class MessageProcessor { 
    private final ServiceA serviceA;
    private final ServiceB serviceB;
    private final ScheduledExecutorService executor;

    public MessageProcessor() {
        this.executor = Executors.newScheduledThreadPool(1);
        this.serviceA = new ServiceA();
        this.serviceB = new ServiceB();
    }

    public void start() {

        Runnable messageProcessingTask = () -> {
            try {
                List<Message> messages = serviceA.receiveMessages();
                messages.forEach(m -> {
                    boolean success = serviceB.doSomething(m);
                    if (success) serviceB.deleteMessage(m);
                    else LOG.error("failed to process the message bla bla...");
                });
            }
            catch (Exception e) {
                e.printStackTrace();
            }
        };

        executor.scheduleWithFixedDelay(messageProcessingTask, 0, 1, TimeUnit.SECONDS);
    }

    public void stop() {
        executor.shutdown();
    }
}

如果将代码更改为使用 2 个线程会发生什么：

    public class MessageProcessor { 

        .....

        public MessageProcessor() {
            this.executor = Executors.newScheduledThreadPool(2);
            this.serviceA = new ServiceA();
            this.serviceB = new ServiceB();
        }

        public void start() {

            Runnable messageProcessingTask = () -> {
                try {
                    List<Message> messages = serviceA.receiveMessages();
                    messages.forEach(m -> {
                        boolean success = serviceB.doSomething(m);
                        if (success) serviceB.deleteMessage(m);
                        else LOG.error("failed to process the message bla bla...");
                    });
                }
                catch (Exception e) {
                    e.printStackTrace();
                }
            };

            executor.scheduleWithFixedDelay(messageProcessingTask, 0, 1, TimeUnit.SECONDS);
            executor.scheduleWithFixedDelay(messageProcessingTask, 0, 1, TimeUnit.SECONDS);
        }

        ....
    }

1. 这种方法是不好的做法吗？
2. 如果此代码导致错误，我该如何生成它们？
3. 如果这是一种不好的方法，最佳做法是什么？

提前致谢。

Answer 1

如果将代码更改为使用 2 个线程会发生什么

嗯，这两个任务将以固定的延迟同时执行。他们都将尝试从服务 A 接收消息，使用服务 B 做某事并删除他们收到的每条消息，然后在一秒钟后再次执行。

是不是你想要的，serviceA 和 serviceB 是否能够处理并发调用都是未知的，所以由你来决定。