Java 8 Lambda - 按位与运算

Question

我目前正在处理使用 Java 8 中的 Lambda 转换使用按位运算的 for 循环的问题。

给定一组复杂的条目，循环需要遍历所有条目并在它们上调用给定的方法（方法返回布尔值）。然后返回结果。

换句话说，我需要在所有条目上调用该方法并存储结果。 这背后的原因是每个条目独立执行复杂的操作，必须执行。最终结果是结果的组合。

代码sn-p：

boolean areAllSuccessful = true;
for (SomeEntry entry : entries) {
     areAllSuccessful = areAllSuccessful & entry.doComplexAction(); // keep calling the method on other entries regardless of the result.
}

return areAllSuccessful;

问题在于 Java 8 中的 Lambda 函数通常会执行短路操作（一旦检测到错误条目，“循环”就会中断并返回错误结果）。

到目前为止，我最好的解决方案是使用 map/filter/count 组合：

return entries
       .stream()
       .map(entry -> entry.doComplexAction())
       .filter(result -> result == false)
       .count() > 0

有没有更聪明/更清洁的方法？

谢谢！

Answer 1

不应该是这样的吗：

boolean areAllSuccessful = entries.stream()
   .map(entry -> entry.isSuccessful())
   .reduce(Boolean.TRUE, Boolean::logicalAnd);

Answer 2

最简洁、最有效的方法是使用带有allMatch()的方法引用

return entries.stream().allMatch(SomeEntry::isSuccessful);

如果您有 1000 个元素，请考虑改用 parallelStream()。

这不会处理每个元素（它在第一个false 返回），所以如果你的isSuccessful() 方法有副作用，这是一个坏名字，你应该重命名它或重构代码来执行副作用在process()（或类似）方法中，并让isSuccessful() 返回结果，如果process() 尚未首先被调用，则抛出IllegalStateException。

如果你不重构，一些开发者（包括你）会打电话给isSuccessful()，而没有意识到它“做事”，这可能很糟糕。

Answer 3

如果你打算使用count()，你真的不需要map()：

return !(entries
        .stream()
        .filter(entry -> !entry.isSuccessful())
        .count() > 0);

---

如果方法isSuccessful()没有副作用，你只需要知道如果所有输入都成功，你可以使用allMatch()：

return entries
        .stream()
        .allMatch(entry -> entry.isSuccessful());

这确实是一个短路操作，一旦找到isSuccessful() 是false 的条目就会返回false，除非必要，否则不会消耗整个流。但是你已经评论了isSuccesful() 的真正意思是“做一些复杂的动作然后然后告诉我他们是否成功”，所以它不适用。