在Java流中peek真的只用于调试吗？

Question

我正在阅读有关Java流的内容，并在此过程中发现了一些新的东西。我发现的一个新东西是peek()函数。我在peek上看到的几乎所有东西都说它应该用来调试流。

如果我有一个流，其中每个帐户都有用户名、密码字段和login（）和loggedIn（）方法。

我也有

Consumer<Account> login = account -> account.login();

而且

Predicate<Account> loggedIn = account -> account.loggedIn();

为什么会这么糟？

List<Account> accounts; //assume it's been setup
List<Account> loggedInAccount = 
accounts.stream()
    .peek(login)
    .filter(loggedIn)
    .collect(Collectors.toList());

null

做这样的事情有什么坏处？有什么理由不让我继续吗？最后，如果不是这个解决方案，那又是什么呢？

它的原始版本使用了.filter（）方法,如下所示；

.filter(account -> {
        account.login();
        return account.loggedIn();
    })

Answer 1

关键在于：

不要以无意的方式使用API，即使它完成了您的眼前目标。那种方法在未来可能会破裂，对于未来的维护者来说也是不清楚的。

将此分解到多个操作中没有坏处，因为它们是不同的操作。以不明确和无意的方式使用API是有害的，如果在Java的未来版本中修改这种特定的行为，可能会产生后果。

它也更传统，因为peek是一个中间操作，在终端操作运行之前不会对整个集合进行操作，但foreach确实是一个终端操作。这样，您就可以围绕代码的行为和流程进行有力的论证，而不是询问peek是否会与foreach在此上下文中的行为相同。

accounts.forEach(a -> a.login());
List<Account> loggedInAccounts = accounts.stream()
                                         .filter(Account::loggedIn)
                                         .collect(Collectors.toList());