我已经将常规的for循环代码转换为java 8流。我尝试了一些,但是我仍然在学习并且用尽了所有想法,请提出想法。可以进一步简化吗?除了使用forEach之外,我无法进行太多更改。
另外,为什么我必须将eid转换为getERecordFromId((String)eid)中的String
Stream <String>eIdsStream = getEidStream();
final HashSet<String> declinedRecords = new HashSet<>();
eIdsStream.forEach (eid -> {
ERecord eRecord = getERecordFromId((String)eid);
if(eRecord.getEHash() != null && Status.DECLINED == eRecord.getStatus()) {
declineRecords.add(eRecord.getEHash());
}
}
最佳答案
由于您使用原始的Stream
变量,因此必须进行强制转换。假设getEidStream()
返回Stream<String>
,则应该将其分配给Stream<String>
变量,或者根本不将其分配给变量。
首先使用forEach
会破坏使用Stream
的目的。
您应该使用filter
和map
转换Stream
以容纳所需的元素,然后收集到Set
。
Set<String> declinedRecords =
getEidStream().map(eid -> getERecordFromId(eid))
.filter(eRecord -> eRecord.getEHash() != null && Status.DECLINED == eRecord.getStatus())
.map(ERecord::getEHash)
.collect(Collectors.toSet());