我正在使用Stream过滤器findAny.orElse,但它没有像我期望的那样工作,所以我认为我不理解它是如何工作的.
在这里我的代码
在这里我的代码
return Stream.of(getobjectAttributeValue(product,matchCriteria.getFieldName())) .map(o -> isIngredientRestricted(matchCriteria,(List<String>) o)) .filter(Boolean::valueOf) .findAny().orElse(isCommercialHierarchyInfoRestricted(product,matchCriteria));
基本上我所期望的是,如果第一个地图发出布尔值false,那么它将被过滤,因此findAny将找不到任何可选项,因此将调用orElse.但即使在过滤器中有一个true,也会调用isCommercialHierarchyInfoRestricted.
知道我做错了什么吗?
解决方法
作为
Tagir explained,使用orElse(表达式)总是在调用方法orElse之前导致表达式的评估,而您必须使用orElseGet(() – > expression)来推迟表达式的求值.
但是,这是不必要的Stream API使用.如果要评估单个项目,则不需要创建单个元素流,只是为了之后调用findAny.您可以在第一时间创建一个Optional:
return Optional.of(getobjectAttributeValue(product,matchCriteria.getFieldName())) .map(o -> isIngredientRestricted(matchCriteria,(List<String>)o)) .filter(b -> b) .orElseGet(() -> isCommercialHierarchyInfoRestricted(product,matchCriteria));
但是,与同等的普通Java语言构造相比,即使这是一个不必要的复杂问题:
return isIngredientRestricted(matchCriteria,(List<String>)getobjectAttributeValue(product,matchCriteria.getFieldName())) || isCommercialHierarchyInfoRestricted(product,matchCriteria);
这完全相同,无需额外的API或lambda表达式. ||如果第一个表达式的计算结果为true,则运算符还保证不会计算第二个表达式.