Java流式操作中如何避免用peek()修改集合元素？

你可以通过以下几种方式避免使用peek()来实现需求，每种方案各有适用场景：

方案一：收集后遍历修改（最直观） #

先完成过滤、排序和收集操作，再对结果列表进行遍历统一设置属性。这种方式代码可读性强、逻辑清晰，适合大多数常规场景：

List<MyDto> myDtoList = service.getDtos()
    .filter(...)
    .sorted(...)
    .collect(Collectors.toList());

// 收集完成后统一修改属性
myDtoList.forEach(e -> e.setPredicted(false));

方案二：使用map()操作修改属性 #

虽然map()的语义是转换元素，但如果只是修改原有对象的属性并返回原对象，也可以用它替代peek()。不过要注意，这种方式更适合明确需要对元素做转换的场景，单纯修改属性的话，方案一的可读性更好：

List<MyDto> myDtoList = service.getDtos()
    .filter(...)
    .sorted(...)
    .map(e -> {
        e.setPredicted(false);
        return e;
    })
    .collect(Collectors.toList());

方案三：自定义Collector（链式调用风格） #

如果希望保持流式操作的链式完整性，可以用Collectors.collectingAndThen()，将收集后的修改逻辑嵌入到收集过程中：

List<MyDto> myDtoList = service.getDtos()
    .filter(...)
    .sorted(...)
    .collect(Collectors.collectingAndThen(
        Collectors.toList(),
        list -> {
            list.forEach(e -> e.setPredicted(false));
            return list;
        }
    ));

为什么不推荐用peek()？ #

peek()的设计初衷是调试流的执行过程，比如查看流中元素的状态变化，它的执行依赖于后续的终端操作。在某些特殊场景下（比如流存在短路操作、终端操作未触发元素处理），peek()中的逻辑可能不会执行，因此用它来修改元素状态不符合其设计意图，存在潜在的逻辑风险。

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