使用Java Streams的map函数时遭遇"insert "";" to complete BlockStatements"错误

错误原因 #

你这段代码的核心问题是误用了map方法：map需要的是一个能返回新值的函数（Function<T,R>），但你调用的x.setStatus(Status.APPROVED)是void类型的setter方法，没有返回值，完全不符合map的要求。编译器因此抛出语法错误，提示补充分号来完成语句，本质是lambda表达式的类型不匹配导致的语法解析异常。

如果其他Stream方法也出现这个错误，大概率是同样的问题：lambda表达式的逻辑不符合对应Stream方法的函数式接口要求（比如把Consumer用在了需要Function的地方，或是lambda写法有语法疏漏）。

解决方案 #

方案1：修改原集合元素（用forEach）

如果你只是想给原列表里的每个Review对象设置状态，直接用forEach才是正确姿势：

// 直接修改原列表
reviewList.forEach(x -> x.setStatus(Status.APPROVED));

// 若需要新列表，先复制原列表再修改
List<Review> tempList = new ArrayList<>(reviewList);
tempList.forEach(x -> x.setStatus(Status.APPROVED));

方案2：生成新对象（用map，适合不可变场景）

如果不想修改原对象，而是要生成一批状态为APPROVED的新Review对象，就在map里创建新对象并返回，同时别忘了用collect把Stream转换为List：

List<Review> tempList = reviewList.stream()
    .map(x -> {
        // 假设Review有拷贝构造器或builder
        Review newReview = new Review(x);
        newReview.setStatus(Status.APPROVED);
        return newReview;
    })
    .collect(Collectors.toList());

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