我正在阅读有关 Java 流的内容,并在阅读过程中发现新事物。我发现的新事物之一是 peek()
函数。我在 peek 上读到的几乎所有内容都说它应该用于调试您的 Streams。
如果我有一个 Stream,其中每个帐户都有一个用户名、密码字段以及一个 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());
现在,据我所知,这完全符合预期。它;
- 获取帐户列表
- 尝试登录每个帐户
- 过滤掉任何未登录的帐户
- 将登录帐户收集到新列表中
这样做的缺点是什么?我有什么理由不应该继续吗?最后,如果不是这个解决方案那又是什么?
原始版本使用 .filter() 方法如下;
.filter(account -> {
account.login();
return account.loggedIn();
})
原文由 Adam.J 发布,翻译遵循 CC BY-SA 4.0 许可协议
从中得出的关键结论是:
不要以无意的方式使用 API,即使它可以实现您的近期目标。 这种方法将来可能会失效,未来的维护者也不清楚。
将其分解为多个操作并没有什么坏处,因为它们是不同的操作。以不明确和意外的方式使用 API 是 有害的,如果在未来的 Java 版本中修改了这种特定行为,则可能会产生后果。
在此操作上使用
forEach
将使维护者清楚地知道accounts
的每个元素都有 预期 的副作用,并且您正在执行一些可以改变它的操作。从某种意义上说,它也更传统
peek
是一个中间操作,在终端操作运行之前不会对整个集合进行操作,但是forEach
确实是一个终端操作。这样,您可以围绕代码的行为和流程提出强有力的论据,而不是询问有关peek
是否与forEach
在此上下文中的行为相同的问题。