简化java流以查找重复的属性

Question

我有一个users列表，我想找到所有重名的用户：

var allNames = users
              .stream()
              .map(u -> u.getName()).collect(Collectors.toList());

var duplicateNames = allNames
                .stream()
                .filter(i -> Collections.frequency(allNames, i) > 1)
                .collect(Collectors.toSet());

我可以改进/简化上述解决方案吗？

例如，实际上我创建了一个包含所有名称的列表，然后对其进行过滤。如何遍历列表以查找其重复名称而不创建附加列表allNames？

Answer 1

一种解决方案是

var duplicate = users.stream()
    .collect(Collectors.toMap(User::getName, u -> false, (x,y) -> true))
    .entrySet().stream()
    .filter(Map.Entry::getValue)
    .map(Map.Entry::getKey)
    .collect(Collectors.toSet());

这会创建一个中间体Map<String,Boolean>来记录哪个名称出现不止一次。您可以使用该keySet()地图的 ，而不是收集到新的Set：

var duplicate = users.stream()
    .collect(Collectors.collectingAndThen(
        Collectors.toMap(User::getName, u -> false, (x,y) -> true, HashMap::new),
            m -> {
                m.values().removeIf(dup -> !dup);
                return m.keySet();
            }));

循环解决方案可以简单得多：

HashSet<String> seen = new HashSet<>(), duplicate = new HashSet<>();
for(User u: users)
    if(!seen.add(u.getName())) duplicate.add(u.getName());