Java Stream中groupingBy方法使用报错及问题排查

错误原因 #

你遇到的编译错误核心是对Collectors.groupingBy的用法理解有误：
Collectors.groupingBy的第二个参数要求是Collector类型，它的作用是处理分组后的元素集合（比如将分组内的元素收集为List、拼接字符串等）。但你传入的Lambda表达式是一个Function（把Rental对象直接转成日期字符串），完全不符合方法的参数签名，因此Eclipse报错。

需求匹配的正确实现 #

根据你描述的需求（返回SortedMap<String, String>，键为读者ID，值为"DD-MM-YYYY DD-MM-YYYY"格式的租赁日期字符串），分两种场景给出解决方案：

场景1：每个读者仅对应一条租赁记录 #

如果业务上保证同一个读者不会多次租赁同一本书，或者你只需要保留读者的最新租赁记录，使用Collectors.toMap直接映射键值对，并指定返回SortedMap：

public SortedMap<String, String> getRentals(String bookID) throws LibException {
    Book b = this.idsColl.get(bookID);
    if (b == null) {
        throw new LibException("指定ID的书籍不存在");
    }

    return b.getRentalsList().stream()
            .collect(Collectors.toMap(
                    Rental::getReaderId,
                    r -> r.getStartDate() + " " + r.getEndDate(),
                    // 当同一读者有多个租赁记录时，这里定义合并规则，示例为保留最新的记录
                    (existingRecord, newRecord) -> newRecord,
                    // 指定返回SortedMap的实现类TreeMap
                    TreeMap::new
            ));
}

场景2：每个读者对应多条租赁记录（拼接所有日期） #

如果需要把同一读者的所有租赁日期拼接成一个字符串，此时才需要使用groupingBy配合下游收集器：

public SortedMap<String, String> getRentals(String bookID) throws LibException {
    Book b = this.idsColl.get(bookID);
    if (b == null) {
        throw new LibException("指定ID的书籍不存在");
    }

    return b.getRentalsList().stream()
            .collect(Collectors.groupingBy(
                    Rental::getReaderId,
                    // 指定分组后的Map为SortedMap
                    TreeMap::new,
                    // 将分组内的每个Rental转成日期字符串，再用逗号拼接
                    Collectors.mapping(
                            r -> r.getStartDate() + " " + r.getEndDate(),
                            Collectors.joining(", ")
                    )
            ));
}

额外注意事项 #

1. 必须增加书籍空值判断，避免NullPointerException
2. 原代码末尾返回null不符合需求，需返回收集后的SortedMap

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