Question

我正在练习书＃34; Java SE 8 for the Really Impatient＆＃34;作者：Cay S. Horstmann。

写一个方法
public static <T> CompletableFuture<T> repeat(
         Supplier<T> action, Predicate<T> until)
异步重复该操作，直到它产生一个值被until函数接受，该函数也应该运行异步。使用读取的函数进行测试来自控制台的java.net.PasswordAuthentication和一个函数通过睡眠一秒钟来模拟有效性检查检查密码是否为＆＃34;秘密＆＃34;。

我想出了以下代码，但随机密码生成策略似乎让我失望了。所有线程都不断选择相同的密码，这看起来很奇怪。

public static <T> CompletableFuture<T> repeat(final Supplier<T> action, final Predicate<T> until) {
    final CompletableFuture<T> futureAction = supplyAsync(action);
    final boolean isMatchFound = futureAction.thenApplyAsync(until::test).join();

    final T suppliedValue = getSuppliedValue(futureAction);

    if (isMatchFound) {
        LOGGER.info("Got a match for value {}.", suppliedValue);

        return futureAction;
    }

    return repeat(() -> suppliedValue, until);
}

private static <T> T getSuppliedValue(final CompletableFuture<T> futureAction) {
    try {
        return futureAction.get();
    } catch (InterruptedException | ExecutionException e) {
        LOGGER.error(e.getMessage());
    }

    return null;
}

测试用例：

@Test
public void testRepeat() {
    Supplier<PasswordAuthentication> action = () -> {
        final String[] passwordVault = new String[] { "password", "secret",
            "secretPassword" };

        final int len = passwordVault.length;

        return new PasswordAuthentication("mickeyMouse",
            passwordVault[ThreadLocalRandom.current().nextInt(len)].toCharArray());
    };
    @SuppressWarnings("static-access")
    Predicate<PasswordAuthentication> until = passwordAuth -> {
        try {
            currentThread().sleep(1000);
        } catch (InterruptedException e) {
            fail(e.getMessage());
        }

        final String password = String.valueOf(passwordAuth.getPassword());

        LOGGER.info("Received password: {}.", password);

        return password.equals("secret");
    };

    repeat(action, until);
}

一次运行，看看有多奇怪地选择相同的密码：

2015-01-09 15：41：33.350 [Thread-1] [INFO] n.a.j.j.c.PracticeQuestionsCh6Test - 收到的密码： secretPassword的。 2015-01-09 15：41：34.371 [Thread-3] [INFO] n.a.j.j.c.PracticeQuestionsCh6Test - 收到的密码： secretPassword的。 2015-01-09 15：41：35.442 [Thread-5] [INFO] n.a.j.j.c.PracticeQuestionsCh6Test - 收到的密码： secretPassword的。 2015-01-09 15：41：36.443 [Thread-7] [INFO] n.a.j.j.c.PracticeQuestionsCh6Test - 收到的密码： secretPassword的。 2015-01-09 15：41：37.451 [Thread-9] [INFO] n.a.j.j.c.PracticeQuestionsCh6Test - 收到的密码： secretPassword的。

Answer 1

我认为你正走在一条不必要的复杂道路上。由于供应 - 测试 - 供应 - 测试 - 供应 - 测试序列是连续的，因此您只需要初始supplyAsync。在async中没有理由做异步。

这是一个简单的实现：

public static <T> CompletableFuture<T> repeat(final Supplier<T> action,
        final Predicate<T> until) {

    return CompletableFuture.supplyAsync(() -> 
        Stream.generate(action)
                .filter(until)
                .findFirst()
                .get()
    );
}

Answer 2

我发现了这个错误。我创建的供应商每次都返回相同的值。以下是更正后的代码。但有一个问题：为什么编译器会强制执行转换？

public static <T> CompletableFuture<T> repeat(final Supplier<T> action,
        final Predicate<T> until) {
    final CompletableFuture<T> futureAction = supplyAsync(action);

    @SuppressWarnings("unchecked")
    CompletableFuture<T> future = (CompletableFuture<T>) futureAction
        .thenApplyAsync(until::test).thenApply(
            isMatchFound -> isMatchFound ? futureAction : repeat(action, until));

    future.join();

    return future;

}

Java 8 CompletedFuture和ThreadLocalRandom

2 个答案: