等待列表中的每个未来完成[关闭]

Question

想改进这个问题吗 通过编辑这篇文章，更新问题，以便用事实和引文来回答。

我调用一个方法，该方法为列表中的每个元素返回一次未来

返回Future的方法是库代码，我无法控制该代码的运行方式，我所拥有的只是Future。

我想等待所有Future完成（成功或失败），然后再继续。

有没有比这更好的方法：

List<Principal> users = new ArrayList<>();
// Fill users
List<Future<UserRecord>> futures = getAllTheFutures(users);
List<UserRecord> results = new ArrayList<>(futures.size());
boolean[] taskCompleted = new boolean[futures.size()];
for (int j = 0; j < taskCompleted.length; j++) {
    taskCompleted[j] = false;
}
do {
    for (int i = 0; i < futures.size(); i++) {
        if (!taskCompleted[i]) {
            try {
                results.add(i, futures.get(i).get(20, TimeUnit.MILLISECONDS));
                taskCompleted[i] = true;
            } catch (TimeoutException e) {
                // Do nothing
            } catch (InterruptedException | ExecutionException e) {
                // Handle appropriately, then...
                taskCompleted[i] = true;
            }
        }
    }
} while (allNotCompleted(taskCompleted));

对于好奇的人：

private boolean allNotCompleted(boolean[] completed) {
    for (boolean b : completed) {
        if (!b)
            return true;
    }
    return false;
}

与这个等待未来列表的答案不同，我无法控制创建未来的代码。

Answer 1

您可以简单地做一个简化列表；从列表中删除成功的响应并迭代直到为空。

List<Principal> users = // fill users
List<Future<UserRecord>> futures = getAllTheFutures(users);
List<UserRecord> results = new ArrayList<>();

for (int i = 0; i < futures.size(); i++) {
        try {
            results.add(futures.get(i).get(<how long you want before your application throws exception>));

        }
        catch (InterruptedException | ExecutionException e) {
            // Handle appropriately, results.add(null) or just leave it out
        }
        catch (TimeoutException timeoutEx) {
            // If the Future retrieval timed out you can handle here
        }

    }
}

由于您的目的是在继续之前收集一组作业，因此在这种情况下，等待线程索引X的返回将导致（大致）最后返回的线程的时间开销。

或者，如果计划在任何线程失败时中止该集中的所有线程，则可以使用Java 8 CompletableFuture

CompletableFuture[] cfs = futures.toArray(new CompletableFuture[futures.size()]);

    return CompletableFuture.allOf(cfs)
            .thenApply(() -> futures.stream()
                                    .map(CompletableFuture::join)
                                    .collect(Collectors.toList())
            );

• Kayaman简化了代码库

Answer 2

您的代码可以简化很多。除非您有问题中未指定的要求，否则可以按照以下方式编写等效版本。

List<Principal> users = // fill users
List<Future<UserRecord>> futures = getAllTheFutures(users);
List<UserRecord> results = new ArrayList<>();

for (int i = 0; i < futures.size(); i++) {
        try {
            results.add(futures.get(i).get());
        } catch (InterruptedException | ExecutionException e) {
            // Handle appropriately, results.add(null) or just leave it out
        }
    }
}