使用Stream API过滤Vehicle的Services后如何保留原始数据结构？

你的代码问题很明确：收集结果时直接把Vehicle对象替换成了过滤后的services Map，自然会丢失name和model字段。要保留原始结构，核心是保留Vehicle对象本身，仅更新其services字段为过滤后的结果。

正确实现方式 #

方式1：创建新对象（不修改原数据，推荐） #

这种方式不会改动原有的vehicles和Vehicle对象，避免意外修改原数据的副作用：

import java.util.Map;
import java.util.stream.Collectors;

// 基于你定义的Manage类实现
Map<String, Manage.Vehicle> filteredVehicles = vehicles.entrySet().stream()
    .map(entry -> {
        Manage.Vehicle originalVehicle = entry.getValue();
        // 复制原Vehicle的name和model，创建新的Vehicle实例
        Manage.Vehicle filteredVehicle = new Manage.Vehicle();
        filteredVehicle.name = originalVehicle.name;
        filteredVehicle.model = originalVehicle.model;
        // 过滤services，仅保留键为"s-1"的条目
        filteredVehicle.services = originalVehicle.services.entrySet().stream()
            .filter(subEntry -> subEntry.getKey().equals("s-1"))
            .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));
        return Map.entry(entry.getKey(), filteredVehicle);
    })
    // 可选：过滤掉services为空的Vehicle，不需要则删除此行
    .filter(entry -> !entry.getValue().services.isEmpty())
    .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));

方式2：修改原对象（简单但不推荐，会改变原数据） #

如果不需要保留原数据，可以直接修改现有Vehicle的services集合：

vehicles.values().forEach(vehicle -> {
    // 移除所有键不等于"s-1"的services条目
    vehicle.services.entrySet().removeIf(entry -> !entry.getKey().equals("s-1"));
});

优化：给Vehicle添加构造方法 #

给Vehicle类添加带参构造方法后，代码会更简洁易读：

public class Manage {
    Map<String, Vehicle> vehicles;

    public static class Vehicle {
        String name;
        String model;
        Map<String, Service> services;

        // 添加带参构造方法
        public Vehicle(String name, String model, Map<String, Service> services) {
            this.name = name;
            this.model = model;
            this.services = services;
        }

        public static class Service {
            String id;
            String report;
        }
    }
}

对应的过滤代码简化为：

Map<String, Manage.Vehicle> filteredVehicles = vehicles.entrySet().stream()
    .map(entry -> {
        Manage.Vehicle original = entry.getValue();
        Map<String, Manage.Service> filteredServices = original.services.entrySet().stream()
            .filter(sub -> sub.getKey().equals("s-1"))
            .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));
        return Map.entry(entry.getKey(), new Manage.Vehicle(original.name, original.model, filteredServices));
    })
    .filter(e -> !e.getValue().services.isEmpty())
    .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));

核心总结 #

你原始代码的错误在于破坏了Vehicle的结构，把整个对象替换成了内部的services Map。正确的思路是保留Vehicle的完整结构，仅对其内部的services字段做过滤更新，这样就能保留name、model等所有原有字段，同时完成services的过滤需求。

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