Java流循环使用

Question

对于Java流，我是一个新手。 我是否需要在此循环中每次都重新创建流，或者有更好的方法吗？ 创建一次流并使用.noneMatch两次将导致“流已关闭”异常。

for ( ItemSetNode itemSetNode : itemSetNodeList )
{
  Stream<Id> allUserNodesStream = allUserNodes.stream().map( n -> n.getNodeId() );

  Id nodeId = itemSetNode.getNodeId();
  //if none of the user node ids match the node id, the user is missing the node
  if ( allUserNodesStream.noneMatch( userNode -> userNode.compareTo( nodeId ) == 0 ) )
  {
    isUserMissingNode = true;
    break;
  }
}

谢谢 ！

Answer 1

我建议您列出循环外所有用户ID的列表。 只要确保类Id覆盖equals()函数即可。

List<Id> allUsersIds = allUserNodes.stream().map(n -> n.getNodeId()).collect(Collectors.toList());

for (ItemSetNode itemSetNode : itemSetNodeList)
{
    Id nodeId = itemSetNode.getNodeId();

    if (!allUsersIds.contains(nodeId))
    {
        isUserMissingNode = true;
        break;
    }
}

Answer 2

Stream的终端操作（例如noneMatch()关闭Stream并使它再次不可重用。
如果您需要重用此Stream：

Stream<Id> allUserNodesStream = allUserNodes.stream().map( n -> n.getNodeId() );

只需将其移动到方法中：

public Stream<Id> getAllUserNodesStream(){
   return allUserNodes.stream().map( n -> n.getNodeId());
}

并在需要时调用它来创建它：

if (getAllUserNodesStream().noneMatch( userNode -> userNode.compareTo( nodeId ) == 0 ))  

现在请记住，流在编译后成为字节码中的循环。
多次执行同一循环可能是不希望的。 因此，您应该在实例化同一流多次之前考虑这一点。

---

作为创建多个流以检测与nodeId匹配的替代方法：

if (allUserNodesStream.noneMatch( userNode -> userNode.compareTo( nodeId ) == 0 ) ) {
  isUserMissingNode = true;
  break;
}

使用Set类型的结构，该结构包含allUserNodes所有id ：

if (idsFromUserNodes.contains(nodeId)){
  isUserMissingNode = true;
  break;
}

这将使逻辑更简单，性能更好。
当然，它假定compareTo()与equals() compareTo()是一致的，但强烈建议这样做（尽管不是必需的）。

Answer 3

以下代码应该是等效的，除了布尔值的取反，因此如果缺少节点则为false 。

首先，将所有用户节点ID收集到TreeSet （如果Id实现hashCode()和equals() ，则应使用HashSet ）。 然后，我们对itemSetNodeList进行流itemSetNodeList以查看集合中是否包含所有这些nodeId。

TreeSet<Id> all = allUserNodes
                        .stream()
                        .map(n -> n.getNodeId())
                        .collect(Collectors.toCollection(TreeSet::new));

boolean isAllNodes = itemSetNodeList
                        .stream()
                        .allMatch(n -> all.contains(n.getNodeId()));

有很多方法可以编写等效的代码（至少对于外部用户而言），这使用Set来改进查找，因此我们不需要保持不断地迭代allUserNodes集合。

您要避免在循环中使用流，因为当您执行线性循环并在其中进行线性流操作时，这会将算法转换为O(n²) 。 对于线性流操作和O(log n) TreeSet查找，此方法为O(n log n) 。 使用HashSet它下降为O(n) ，除非您要处理大量元素，否则并不重要。

Answer 4

您还可以执行以下操作：

Set<Id> allUserNodeIds = allUserNodes.stream()
    .map(ItemSetNode::getNodeId)
    .collect(Collectors.toCollection(TreeSet::new));
return itemSetNodeList.stream()
    .anyMatch(n -> !allUserNodeIds.contains(n.getNodeId())); // or firstMatch

甚至：

Collectors.toCollection(() -> new TreeSet<>(new YourComparator()));

Answer 5

它将从itemSetNodeList中获取每个项目，并使用noneMatch（）检查它是否存在。 如果不存在，将返回true。 如果没有找到至少一个项目，则anyMatch会停止搜索并返回false。 如果找到所有项目，我们将返回true。

Stream<Id> allUserNodesStream = allUserNodes.stream().map( n -> n.getNodeId() );
            boolean isUserMissing=itemSetNodeList.stream()
                            .anyMatch(n-> allUserNodes.stream().noneMatch(n));