Java中删除Plan关联的无效Package后如何返回更新后的Plans集合

你当前操作的packetFirst是独立的临时列表：你调用addAll(tPlan.get(0).getPacket())只是把原Plan的Packet元素复制到了新的列表中，后续对packetFirst的删除操作完全不会影响ntTPla中存储的原Plan对象内部的Packet集合，所以最终返回的还是未修改的旧数据。

你可以直接操作原Plan内部的Packet集合，或者在修改完临时列表后把新的列表写回原Plan。以下是两种可行实现：

方案1：直接操作原集合（推荐） #

如果Plan类的getPacket()返回的是可修改的集合，直接操作原集合即可：

[...]
        List<Plan> tPlan = new ArrayList<Plan>();
        tPlan.addAll(ntTPla.values());
        
        // 直接取原Plan内部的Packet集合，不做复制
        List<Packet> packetFirst = tPlan.get(0).getPacket();
        
        String test2 = null;
        for (int i = 0 ; i < packetFirst.size(); i++){ 
            if(packetFirst.get(i).getBest()) {
                test2 = packetFirst.get(i).getCode();
                break;
            }
        }

        // 新增判空逻辑，避免找不到best=true的Packet时出现空指针
        if (test2 != null) {
            for (int ii =  packetFirst.size() - 1 ; ii >= 0; ii--){ 
                if(!packetFirst.get(ii).getCode().equals(test2)) {
                    packetFirst.remove(ii);
                }
            }
        }
        
        return ntTPla.values();

方案2：临时列表修改后写回 #

如果getPacket()返回的是不可修改的集合，用该方案：

[...]
        List<Plan> tPlan = new ArrayList<Plan>();
        tPlan.addAll(ntTPla.values());
        
        List<Packet> packetFirst = new ArrayList<Packet>();
        packetFirst.addAll(tPlan.get(0).getPacket());
        
        String test2 = null;
        for (int i = 0 ; i < packetFirst.size(); i++){ 
            if(packetFirst.get(i).getBest()) {
                test2 = packetFirst.get(i).getCode();
                break;
            }
        }

        if (test2 != null) {
            for (int ii =  packetFirst.size() - 1 ; ii >= 0; ii--){ 
                if(!packetFirst.get(ii).getCode().equals(test2)) {
                    packetFirst.remove(ii);
                }
            }
            // 把修改后的Packet列表写回原Plan
            tPlan.get(0).setPacket(packetFirst);
        }
        
        return ntTPla.values();

注意：当前你只处理了所有Plan中第一个Plan的Packet，如果需要校验全量Plan的Packet，外层再加一层遍历ntTPla.values()的循环即可。

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