Java Streams用法咨询：按年龄分组获取员工姓名的实现是否合理？

嘿，你的这段代码功能上完全正确，确实能实现按年龄分组员工并打印对应姓名的需求，但说实话，嵌套的forEach循环确实有点冗余，我们可以用Stream API的特性让代码更简洁优雅~

原代码的小问题 #

你先通过groupingBy把员工按年龄分组得到Map<Integer, List<Employee>>，然后又嵌套两层forEach来遍历打印。这样不仅多了一次循环，还容易出现最后一个姓名后多余逗号的小问题（比如输出会是Alice, Bob, ）。

优化后的实现 #

我们可以在collect阶段就把每个分组的员工姓名提取出来，直接得到Map<Integer, List<String>>，再结合String.join来简化打印逻辑：

employeeList.stream()
    .collect(Collectors.groupingBy(
        Employee::getAge, // 按年龄分组的分类器
        Collectors.mapping(Employee::getFirstName, Collectors.toList()) // 把每个员工转成姓名并收集为列表
    ))
    .forEach((age, names) -> 
        System.out.printf("In the age %d the following people are present %s%n", 
            age, String.join(", ", names))
    );

优化点说明 #

• 减少显式循环：把提取姓名的逻辑整合到分组收集的过程中，避免后续再遍历员工列表
• 避免多余逗号：String.join会自动用分隔符连接元素，不会在末尾多出来一个逗号
• 更符合Stream设计理念：Stream就是要让数据处理形成流水线，一步到位完成分组和转换，代码逻辑更连贯易读

当然，原代码的逻辑是没问题的，只是优化后会更简洁高效哦~

内容的提问来源于stack exchange，提问作者Vinnie