Java 8代码迁移遇困：传统代码转流式写法失败求助

我正尝试将一段Java代码转换为符合Java 8规范的写法，参考了若干转换指南但始终无法成功。请问我哪里操作有误？

原代码 #

private List<ResponseVo> populateResponse(String userId, String type) {

    List<Wrapper> recordsList = logRepo.getEntities(type, userId);
    List<ResponseVo> responseVoList = new ArrayList<>();

    if (null != userId) {
        for (Wrapper record : recordsList) {
            if (record.getSortKey().contains(userId)) {
                ResponseVo responseVo = buildResultsResponse(record);
                responseVoList.add(responseVo);
            }
        }
    } else {
        for (Wrapper record : recordsList) {
            ResponseVo testLogResponseVo = buildResultsResponse(record);
            responseVoList.add(responseVo); // 此处存在变量名错误，应添加testLogResponseVo
        }
    }
    return responseVoList;
}

我的尝试代码 #

recordsList.stream()
    .filter(record -> record.getSortKey().contains(userId))
    .collect(Collectors.toList());

问题分析 #

• 缺少对象转换步骤：原代码核心逻辑是把Wrapper通过buildResultsResponse()转为ResponseVo，但你的Stream代码只做了过滤和收集，完全漏掉了map()转换环节，最终得到的是Wrapper列表而非需要的ResponseVo列表。
• 未覆盖null分支逻辑：原代码在userId为null时会转换所有记录，不做过滤，但你的尝试只实现了userId非null时的过滤逻辑，没处理else分支场景。
• 原代码的笔误：原else分支里的responseVoList.add(responseVo);是错误的，应该添加的是前面定义的testLogResponseVo，转换时需同步修正。

正确的Java 8 Stream写法 #

private List<ResponseVo> populateResponse(String userId, String type) {
    List<Wrapper> recordsList = logRepo.getEntities(type, userId);
    
    return recordsList.stream()
        // 用一行逻辑覆盖原if-else分支：userId为null时不过滤，否则检查包含关系
        .filter(record -> userId == null || record.getSortKey().contains(userId))
        // 将Wrapper转换为ResponseVo
        .map(this::buildResultsResponse)
        // 收集为目标列表
        .collect(Collectors.toList());
}

这个写法既完整复现了原代码的业务逻辑，又符合Java 8 Stream的简洁规范，同时修正了原代码中的变量名错误。

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