Question

考虑我们有一个预定的执行者服务：

ScheduledExecutorService threadPool = Executors.newScheduledThreadPool(...);

对于某些逻辑，我们想重试任务执行。以下方法对我来说似乎很有趣，但我不明白为什么：

threadPool.submit(new Runnable() {
    @Override
    public void run() {
        // ...
        if (needToBeScheduled()) {
           threadPool.schedule(this, delay, TimeUnit.MINUTES);
        } else if (needToBeResubmitted()) {
           threadPool.submit(this);
        }
    }
});

我看到的一个明显问题是此代码无法转换为lambda：

threadPool.submit(()-> {
    // ...
    if (needToBeScheduled()) {
        threadPool.schedule(this, delay, TimeUnit.MINUTES);
    } else if (needToBeResubmitted()) {
        threadPool.submit(this);
    }
});

^^这不会编译，因为我们不能从lambda引用this。虽然可以通过引入一个生成这样一个实例的方法来解决它，而不是this。

但这只是我看到的一个缺点。这里还有什么可以导致任何问题吗？也许有更合适的方法？将此逻辑移至ThreadPoolExecutor.afterExecute()（这会导致类型转换......）？

假设该对象是无状态的，即Runnable实例中没有对象变量。

P.S。该怎么做（重新安排任务或重新提交或什么都不做）的逻辑基于从数据库（或任何外部源）检索到的一些信息。所以Runnable仍然是无国籍的，但它会根据其工作的一些结果来计算结果。

Answer 1

老实说，我不喜欢任务（一个简单的独立工作单元）决定是否应该将自己置于服务中并直接与ExecutorService交互的方法。我相信// ...是任务应该执行的唯一部分。

我会转换Runnable中的Callable<Boolean>：

Callable<Boolean> task = () -> {
    // ...
    return needToBeScheduled; // or sth more complex with several boolean fields
};

我肯定会将该逻辑移到任务之外（例如，转换为服务方法）：

Future<Boolean> future = threadPool.submit(task);

try {
    boolean needToBeScheduled = future.get();

    if (needToBeScheduled) {
        threadPool.schedule(task, delay, TimeUnit.MINUTES);
    }
} catch (InterruptedException | ExecutionException e) {
    e.printStackTrace();
}

通过更复杂的东西，我的意思是一个包含2个boolean字段的类。 <{1}} s使得事情变得懒惰。

Supplier<Boolean>

通过对上述示例进行一些更改，我们得到了：

final class TaskResult {
    private final Supplier<Boolean> needToBeScheduled;
    private final Supplier<Boolean> needToBeResubmitted;

    private TaskResult(Supplier<Boolean> needToBeScheduled, Supplier<Boolean> needToBeResubmitted) {
        this.needToBeScheduled = needToBeScheduled;
        this.needToBeResubmitted = needToBeResubmitted;
    }

    public static TaskResult of(Supplier<Boolean> needToBeScheduled, Supplier<Boolean> needToBeResubmitted) {
        return new TaskResult(needToBeScheduled, needToBeResubmitted);
    }

    public boolean needToBeScheduled() {
        return needToBeScheduled != null && needToBeScheduled.get();
    }

    public boolean needToBeResubmitted() {
        return needToBeResubmitted != null && needToBeResubmitted.get();
    }
}

从任务本身重新提交/安排任务 - 这是一个好习惯吗？

1 个答案: