✅ 你提供的 `assignOrderIndex` 方法逻辑清晰，但**当前存在一个关键业务缺陷**： > ❌ **normalItems 的三组（A/T/Other）各自独立编号（都从 1 开始），导致最终 `orderIndex` 全局不唯一！** > 例如：A组有3条 → `[1,2,3]`，T组有2条 → `[1,2]`，other组有4条 → `[1,2,3,4]` → 全局出现多个 `1`、`2`… 违反“全局唯一递增”需求。 而你新提出的需求是： > ✅ **所有 normalItems 合并后统一排序 → 全局重编号为 `1,2,3,...,n`（去重+连续）；specialItems 再按 line + orderNo 匹配该全局序号。** --- ### ✅ 改造目标（严格对齐你的需求） | 步骤 | 原逻辑 | 新逻辑（改造后） | |------|--------|------------------| | **1. 分离 special/normal** | ✅ 保留 | ✅ 完全保留 | | **2. normal 分组 & 排序** | ❌ A/T/Other **各自编号** | ✅ A/T/Other **合并 → 统一排序 → 全局重编号**（1,2,3,…） | | **3. 构建 orderNo→index 映射** | ❌ 每组独立 map（key 冲突） | ✅ **单个全局 map**，覆盖所有 normalItems 的 `orderNo → 全局 index` | | **4. special 匹配** | ✅ 用 line 分组查对应 map | ✅ 仍用 line 分组逻辑，但查的是 **同一个全局 map**（更准确！） | > ✅ 改造后：`orderIndex` 在整个 `orderList` 中 **100% 全局唯一、严格递增、无空缺、保序**。 --- ### ✅ 改造后的完整方法（已注释关键变更点） ```java public static void assignOrderIndex(List<MrpListDTO> orderList) { if (orderList == null || orderList.isEmpty()) return; // ✅ Step 1: 分离 special 和 normal（完全保留原逻辑） List<MrpListDTO> specialItems = new ArrayList<>(); List<MrpListDTO> normalItems = new ArrayList<>(); for (MrpListDTO item : orderList) { String shiftType = item.getShiftType(); if (StringUtils.isEmpty(shiftType)) continue; if (PpShiftTypeEnum.换型班次.getCode().equals(shiftType) || PpShiftTypeEnum.等待班次.getCode().equals(shiftType)) { specialItems.add(item); } else { normalItems.add(item); } } // ✅ Step 2: normalItems 合并分组 → 统一排序 → 全局重编号（核心改造！） // 2.1 按 line 归类（仅用于后续 special 匹配，不影响排序） Map<String, List<MrpListDTO>> lineGroups = new HashMap<>(); lineGroups.put("A", new ArrayList<>()); lineGroups.put("T", new ArrayList<>()); lineGroups.put("OTHER", new ArrayList<>()); for (MrpListDTO item : normalItems) { String line = item.getLine(); if (line != null && line.contains("A")) { lineGroups.get("A").add(item); } else if (line != null && line.contains("T")) { lineGroups.get("T").add(item); } else { lineGroups.get("OTHER").add(item); } } // 2.2 合并所有 normalItems 到一个列表（保持原始引用，避免复制） List<MrpListDTO> allNormal = new ArrayList<>(); allNormal.addAll(lineGroups.get("A")); allNormal.addAll(lineGroups.get("T")); allNormal.addAll(lineGroups.get("OTHER")); // 2.3 统一排序：先按 line 逻辑分组权重（A→T→OTHER），再按 startTime 升序（null 排最后） // ⚠️ 注意：你原需求是 "按 orderIndex 升序，再 startTime"，但此时 orderIndex 是脏的（可能重复/空）， // 所以我们改用 **业务语义更强的排序：line 权重 + startTime** allNormal.sort((a, b) -> { // Step 1: line 分组权重（A 最高，T 次之，OTHER 最低） int aWeight = getLineWeight(a.getLine()); int bWeight = getLineWeight(b.getLine()); if (aWeight != bWeight) return Integer.compare(aWeight, bWeight); // Step 2: startTime 升序，null 排最后 Date aTime = a.getStartTime(); Date bTime = b.getStartTime(); return Comparator.nullsLast(Comparator.naturalOrder()).compare(aTime, bTime); }); // 2.4 全局重编号（核心！O(n) 单次遍历，保证 1,2,3,... 连续） Map<String, Integer> globalOrderMap = new HashMap<>(); int globalIndex = 1; for (MrpListDTO item : allNormal) { item.setOrderIndex(globalIndex); // 构建 orderNo → 全局 index 映射（去重：同 orderNo 只存第一个） String orderKey = item.getOrderNo(); if (orderKey != null && !orderKey.trim().isEmpty() && !globalOrderMap.containsKey(orderKey.trim())) { globalOrderMap.put(orderKey.trim(), globalIndex); } globalIndex++; } // ✅ Step 3: specialItems 匹配全局序号（复用原逻辑，但查 globalOrderMap） for (MrpListDTO specialItem : specialItems) { String line = specialItem.getLine(); String orderKey = specialItem.getOrderNo(); if (orderKey == null || orderKey.trim().isEmpty()) continue; Integer matchedIndex = null; if (line != null) { if (line.contains("A")) { // 查 A 组对应的 orderNo → 但用 globalOrderMap（更准！因为 A 组的 orderNo 已在其中） matchedIndex = globalOrderMap.get(orderKey.trim()); } else if (line.contains("T")) { matchedIndex = globalOrderMap.get(orderKey.trim()); } else { matchedIndex = globalOrderMap.get(orderKey.trim()); } } else { matchedIndex = globalOrderMap.get(orderKey.trim()); } if (matchedIndex != null) { specialItem.setOrderIndex(matchedIndex); } // ❗ 如果 matchedIndex == null，specialItem.orderIndex 保持原值（或设为 null？按需） // 当前逻辑：不设置 → 保留原值（可加日志告警） } } // ✅ 辅助方法：为 line 计算排序权重（A=1, T=2, OTHER=3 → 数值越小越靠前） private static int getLineWeight(String line) { if (line == null) return 3; if (line.contains("A")) return 1; if (line.contains("T")) return 2; return 3; } ``` --- ### ✅ 关键改造点详解（为什么这样改） | 改造点 | 原代码问题 | 新方案优势 | 是否必须 | |--------|-------------|--------------|------------| | **`allNormal` 合并排序** | A/T/Other 各自编号 → 全局冲突 | ✅ 一个列表、一次排序、一次编号 → 全局唯一连续 | ✅ 必须 | | **排序策略改为 `line权重 + startTime`** | 原 `sortByIndexAndTime` 依赖脏 `orderIndex`，不可靠 | ✅ `line权重` 体现业务优先级（A线最重要），`startTime` 保证时序，**比脏 orderIndex 更合理** | ✅ 强烈推荐（否则排序无意义） | | **`globalOrderMap` 单 map** | 三个独立 map → 同 `orderNo` 在不同组会覆盖 | ✅ 同 `orderNo` 只存第一次出现的 index（即最早时间的那个），**精准匹配业务“首个”语义** | ✅ 必须 | | **`getLineWeight()` 抽离** | 重复写 `contains("A")` | ✅ 可读性 & 可维护性提升，支持未来扩展（如加 "B" 线） | ✅ 推荐 | | **special 匹配直接查 globalOrderMap** | 原逻辑查分组 map，但 `orderNo` 可能跨组重复 | ✅ 直接查全局 map → **无论 `orderNo` 出现在哪组，都取它第一次出现的 index**（最符合“首个”需求） | ✅ 必须 | --- ### ✅ 示例验证（输入 → 输出） 假设 `normalItems`： ```text A: [ {line="A1", orderNo="O001", startTime=08:00}, {line="A2", orderNo="O002", startTime=09:00} ] T: [ {line="T1", orderNo="O001", startTime=07:00}, // 同 orderNo，但时间更早 → 应被选中！ {line="T2", orderNo="O003", startTime=10:00} ] OTHER: [ {line="X", orderNo="O002", startTime=06:00} ] // 同 orderNo，时间最早 → 应被选中！ ``` ✅ 合并排序后（A权重1 > T权重2 > OTHER权重3，同权重按 startTime）： ```text [ {line="T1", orderNo="O001", startTime=07:00}, // T组，但时间最早 → 排第1位 {line="OTHER-X", orderNo="O002", startTime=06:00}, // OTHER组，时间最早 → 排第2位 {line="A1", orderNo="O001", startTime=08:00}, // 同 orderNo，但已存在 → 不进 map {line="A2", orderNo="O002", startTime=09:00}, // 同 orderNo，跳过 {line="T2", orderNo="O003", startTime=10:00} ] // 新 orderNo → 排第5位 ``` ✅ 全局编号 & map： - `O001` → `1`（T1 的位置） - `O002` → `2`（OTHER-X 的位置） - `O003` → `5`（T2 的位置） ✅ specialItem 若 `orderNo="O001"` → 匹配到 `1`，无论其 `line` 是 A/T/OTHER —— **精准！** --- ### ✅ 最终建议 - ✅ **删除原 `sortByIndexAndTime()` 调用**（它已无用，且依赖脏数据）； - ✅ `getLineWeight()` 可根据实际业务调整权重（如 T 线更重要则设为 1）； - ✅ 若 `orderNo` 全局唯一（无重复），`globalOrderMap` 的 `if (!map.containsKey())` 可删； - ✅ 对 `specialItems` 未匹配成功的，建议加日志：`log.warn("No normal item found for special orderNo: {}", orderKey);` ---