有没有办法把这个for循环改成使用Java流？

Question

Set<String> unique = new HashSet<>();
List<String> duplicates = new ArrayList<>();

for (Animal cat: animals) {
    if (!unique.add(cat.getName())) {
        duplicates.add(cat.getName());
    }
}
return duplicates;

我想知道是否有办法简化这个？我是Java流的新手，我尝试使用Map，但我使用了传统的for循环。

Answer 1

我不知道这是否可以被认为是一种简化，但这里有一种方法可以对流进行简化：

return animals.stream()
        .collect(Collectors.groupingBy(Animal::getName))
        .values()
        .stream()
        .flatMap(group -> group.stream().skip(1))
        .map(Animal::getName)
        .collect(Collectors.toList());

Answer 2

是否要根据动物名称提取动物列表的重复字符串名称？尽管您的代码不涉及第一个找到的重复项，并返回重复项位于 n-1 计数中的 List，但这里是：

Set<String> set = new HashSet<>();
List<String> names = animals.stream()
                            .map(cat -> cat.getName())      // Names to collect and compare
                            .filter(name -> !set.add(name)) // Collect duplicates
                            .collect(Collectors.toList());  // To the List

此解决方案基于＜code＞for循环

这里有一个关于流的替代工作方式——API文档——但是有点复杂:

List<String> names = animals.stream()
    .collect(Collectors.groupingBy(
         Animal::getName, 
         Collectors.counting()))            // Collects to Map <name, count>
    .entrySet()                             // Gets the entries
    .stream()                               // Stream of the entries
    .filter(entry -> entry.getValue() > 1)  // Filters the duplicates
    .map(entry -> Collections.nCopies(      // Creates n-1 copies of the key as same as the
        entry.getValue().intValue() - 1,    //   OP's sample consumes the first duplication
        entry.getKey()))                    //   and includes the remainin ones
    .flatMap(List::stream)                  // Flattens the structure
    .collect(Collectors.toList());          // Results in the List

两种方式都来自输入：

List<Animal> animals = Arrays.asList(
    new Animal("A"), new Animal("A"), new Animal("A"), 
    new Animal("B"), new Animal("B"), new Animal("C"));

以下输出（无序）：

[A， B， A]

Answer 3

我想知道是否有办法简化这一点？我

流方式可能不是您的需求所需要的，您的代码实际上很简单。

流允许从输入（流）传递

a-

实际上你的代码不能依赖这样的逻辑，因为返回的< code >列表

List<String> duplicates =     
    animals.stream()
           .collect(Collectors.groupingBy(Animal::getName))
           .entrySet().stream()
           .filter(e -> e.getValue().size() > 1)
           .map(Entry::getKey)
           .collect(Collectors.toList());

您希望按出现的顺序返回每个重复出现的List。
这意味着您不映射Stream

a-

因为如果结果中没有添加动物名称，就需要过滤掉元素...但是流并不是用来递增地填充结果的。所以你被困住了。

你可以写这个，但正如所说的，这确实不是一个简化，而且它仍然没有应用相同的逻辑，因为dup名称的顺序与代码中的顺序不同：

List<String> duplicates = 
          animals.stream()
                 .collect(Collectors.groupingBy(Animal::getName, LinkedHashMap::new, Collectors.toList()))
                 .values().stream()
                 .flatMap(l-> l.stream().skip(1)) 
                 .map(Animal::getName)
                 .collect(Collectors.toList());