Java Stream中搭配forEach使用if语句是否属于不推荐的写法？

写法合法性与推荐性说明 #

你写的forEach(p -> {if...语法完全合法，可以正常运行，不属于语法错误，但确实不符合Java流式API的最佳实践，不推荐在生产代码中使用。

不推荐的核心原因 #

• 违背流式API的设计理念：流式API是声明式编程的典型实现，核心思想是将「筛选、转换、消费」等不同逻辑拆分为独立的流式算子，逻辑边界清晰易读。你把筛选逻辑内嵌到终端操作forEach的代码块中，相当于把过程式代码塞进了流式结构里，失去了流式编程的优势。
• 可维护性差：如果后续需要新增筛选条件、调整判断逻辑，forEach内的代码块会越来越臃肿，嵌套层级变高后可读性会急剧下降。
• 调试成本更高：使用独立的filter算子时，可以很方便地在筛选步骤后插入peek算子打印中间结果，或者单独给筛选逻辑打断点排查问题，内嵌到forEach里就无法单独调试筛选逻辑。
• 你的原有代码中重复调用了两次p.getRoom()，虽然大部分场景下getter没有副作用不会出问题，但属于不必要的重复调用，代码严谨性不足。

更合适的实现方案 #

方案1：拆分筛选与消费逻辑（最通用，推荐） #

用filter算子提前筛选符合条件的灯具，再执行关闭操作，逻辑分层清晰，和原有代码逻辑完全等价：

import java.util.Objects;

public void turnOffLampsInRooms(Set<Room> rooms) {
    lamps.stream()
         .filter(lamp -> Objects.nonNull(lamp.getRoom()))
         .filter(lamp -> rooms.contains(lamp.getRoom()))
         .forEach(Lamp::turnOff);
}

方案2：反向遍历优化（适用于rooms集合远小于lamps集合的场景） #

如果目标房间集合的规模远小于灯具总集合，反向遍历rooms再关闭房间内的灯具，性能会更好：

import java.util.Objects;
import java.util.Collection;

public void turnOffLampsInRooms(Set<Room> rooms) {
    rooms.stream()
         .filter(Objects::nonNull)
         .map(Room::getLamps)
         .flatMap(Collection::stream)
         .forEach(Lamp::turnOff);
}

注意该方案的前提是Room类提供getLamps()方法可以获取房间内的所有灯具实例。

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