✨ feat(mes): 重构过程检验行服务，优化方法命名与逻辑

- 重命名 selectListByIpqcId 为 getIpqcLineListByIpqcId，提升可读性 - 添加 validateIpqcLineExists 方法，增强行存在性校验 - 移除不再使用的字段和方法，简化代码结构

✨ feat(mes): 重构过程检验行服务，优化方法命名与逻辑
- 重命名 selectListByIpqcId 为 getIpqcLineListByIpqcId，提升可读性 - 添加 validateIpqcLineExists 方法，增强行存在性校验 - 移除不再使用的字段和方法，简化代码结构
YunaiV
1 parent 702d212f
Showing 7 changed files with 50 additions and 53 deletions
yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/controller/admin/qc/indicatorresult/MesQcIndicatorResultController.java
yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/controller/admin/qc/ipqc/MesQcIpqcController.java
yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/controller/admin/qc/ipqc/MesQcIpqcLineController.java
yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/controller/admin/qc/ipqc/vo/MesQcIpqcSaveReqVO.java
yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/service/qc/ipqc/MesQcIpqcLineService.java
yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/service/qc/ipqc/MesQcIpqcLineServiceImpl.java
yudao-module-mes/src/main/java/cn/iocoder/yudao/module/mes/service/qc/ipqc/MesQcIpqcServiceImpl.java
@@ -137,7 +137,7 @@ public class MesQcIndicatorResultController {
             indicatorIds = CollUtil.isEmpty(lines) ? Collections.emptyList()
                     : new ArrayList<>(convertSet(lines, MesQcIqcLineDO::getIndicatorId));
         } else if (Objects.equals(qcType, MesQcTypeEnum.IPQC.getType())) {
-            List<MesQcIpqcLineDO> lines = ipqcLineService.selectListByIpqcId(qcId);
+            List<MesQcIpqcLineDO> lines = ipqcLineService.getIpqcLineListByIpqcId(qcId);
             indicatorIds = CollUtil.isEmpty(lines) ? Collections.emptyList()
                     : new ArrayList<>(convertSet(lines, MesQcIpqcLineDO::getIndicatorId));
         } else if (Objects.equals(qcType, MesQcTypeEnum.OQC.getType())) {
@@ -35,7 +35,6 @@ import org.springframework.validation.annotation.Validated;
 import org.springframework.web.bind.annotation.*;
 import java.io.IOException;
-import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
 import java.util.Map;
@@ -9,11 +9,9 @@ import cn.iocoder.yudao.module.mes.controller.admin.qc.ipqc.vo.line.MesQcIpqcLin
 import cn.iocoder.yudao.module.mes.dal.dataobject.md.unitmeasure.MesMdUnitMeasureDO;
 import cn.iocoder.yudao.module.mes.dal.dataobject.qc.indicator.MesQcIndicatorDO;
 import cn.iocoder.yudao.module.mes.dal.dataobject.qc.ipqc.MesQcIpqcLineDO;
-import cn.iocoder.yudao.module.mes.dal.dataobject.tm.tool.MesTmToolDO;
 import cn.iocoder.yudao.module.mes.service.md.unitmeasure.MesMdUnitMeasureService;
 import cn.iocoder.yudao.module.mes.service.qc.indicator.MesQcIndicatorService;
 import cn.iocoder.yudao.module.mes.service.qc.ipqc.MesQcIpqcLineService;
-import cn.iocoder.yudao.module.mes.service.tm.tool.MesTmToolService;
 import io.swagger.v3.oas.annotations.Operation;
 import io.swagger.v3.oas.annotations.Parameter;
 import io.swagger.v3.oas.annotations.tags.Tag;
@@ -46,8 +44,6 @@ public class MesQcIpqcLineController {
     private MesQcIndicatorService indicatorService;
     @Resource
     private MesMdUnitMeasureService unitMeasureService;
-    @Resource
-    private MesTmToolService toolService;
     @GetMapping("/get")
     @Operation(summary = "获得过程检验单行")
@@ -31,9 +31,6 @@ public class MesQcIpqcSaveReqVO {
     @NotNull(message = "检验类型不能为空")
     private Integer type;
-    @Schema(description = "检验模板 ID", example = "100")
-    private Long templateId;
-
     // ========== 来源单据 ==========
     @Schema(description = "来源单据类型", example = "FEEDBACK")
@@ -15,6 +15,14 @@ import java.util.List;
 public interface MesQcIpqcLineService {
     /**
+     * 校验过程检验行存在
+     *
+     * @param id 编号
+     * @return 过程检验行
+     */
+    MesQcIpqcLineDO validateIpqcLineExists(Long id);
+
+    /**
      * 获得过程检验行
      *
      * @param id 编号
@@ -31,14 +39,6 @@ public interface MesQcIpqcLineService {
     PageResult<MesQcIpqcLineDO> getIpqcLinePage(MesQcIpqcLinePageReqVO pageReqVO);
     /**
-     * 根据过程检验单 ID 查询所有行
-     *
-     * @param ipqcId 过程检验单 ID
-     * @return 行列表
-     */
-    List<MesQcIpqcLineDO> selectListByIpqcId(Long ipqcId);
-
-    /**
      * 从模板指标自动生成检验行
      *
      * @param ipqcId 过程检验单 ID
@@ -47,26 +47,26 @@ public interface MesQcIpqcLineService {
     void createLinesFromTemplate(Long ipqcId, Long templateId);
     /**
-     * 根据过程检验单 ID 级联删除所有行
+     * 根据缺陷记录重新计算行级缺陷统计
      *
-     * @param ipqcId 过程检验单 ID
+     * @param ipqcId  过程检验单 ID
+     * @param records 缺陷记录列表
      */
-    void deleteByIpqcId(Long ipqcId);
+    void recalculateLineDefectStats(Long ipqcId, List<MesQcDefectRecordDO> records);
     /**
-     * 校验过程检验行存在
+     * 根据过程检验单 ID 获取所有行
      *
-     * @param id 编号
-     * @return 过程检验行
+     * @param ipqcId 过程检验单 ID
+     * @return 行列表
      */
-    MesQcIpqcLineDO validateIpqcLineExists(Long id);
+    List<MesQcIpqcLineDO> getIpqcLineListByIpqcId(Long ipqcId);
     /**
-     * 根据缺陷记录重新计算行级缺陷统计
+     * 根据过程检验单 ID 级联删除所有行
      *
-     * @param ipqcId  过程检验单 ID
-     * @param records 缺陷记录列表
+     * @param ipqcId 过程检验单 ID
      */
-    void recalculateLineDefectStats(Long ipqcId, List<MesQcDefectRecordDO> records);
+    void deleteListByIpqcId(Long ipqcId);
 }
@@ -44,6 +44,15 @@ public class MesQcIpqcLineServiceImpl implements MesQcIpqcLineService {
     private MesQcIndicatorService indicatorService;
     @Override
+    public MesQcIpqcLineDO validateIpqcLineExists(Long id) {
+        MesQcIpqcLineDO line = ipqcLineMapper.selectById(id);
+        if (line == null) {
+            throw exception(QC_IPQC_LINE_NOT_EXISTS);
+        }
+        return line;
+    }
+
+    @Override
     public MesQcIpqcLineDO getIpqcLine(Long id) {
         return ipqcLineMapper.selectById(id);
     }
@@ -54,11 +63,6 @@ public class MesQcIpqcLineServiceImpl implements MesQcIpqcLineService {
     }
     @Override
-    public List<MesQcIpqcLineDO> selectListByIpqcId(Long ipqcId) {
-        return ipqcLineMapper.selectListByIpqcId(ipqcId);
-    }
-
-    @Override
     public void createLinesFromTemplate(Long ipqcId, Long templateId) {
         List<MesQcTemplateIndicatorDO> templateIndicators = templateIndicatorMapper.selectListByTemplateId(templateId);
         if (CollUtil.isEmpty(templateIndicators)) {
@@ -80,20 +84,6 @@ public class MesQcIpqcLineServiceImpl implements MesQcIpqcLineService {
     }
     @Override
-    public void deleteByIpqcId(Long ipqcId) {
-        ipqcLineMapper.deleteByIpqcId(ipqcId);
-    }
-
-    @Override
-    public MesQcIpqcLineDO validateIpqcLineExists(Long id) {
-        MesQcIpqcLineDO line = ipqcLineMapper.selectById(id);
-        if (line == null) {
-            throw exception(QC_IPQC_LINE_NOT_EXISTS);
-        }
-        return line;
-    }
-
-    @Override
     public void recalculateLineDefectStats(Long ipqcId, List<MesQcDefectRecordDO> records) {
         List<MesQcIpqcLineDO> lines = ipqcLineMapper.selectListByIpqcId(ipqcId);
         if (CollUtil.isEmpty(lines)) {
@@ -101,7 +91,9 @@ public class MesQcIpqcLineServiceImpl implements MesQcIpqcLineService {
         }
         List<MesQcIpqcLineDO> updateLines = new ArrayList<>(lines.size());
         for (MesQcIpqcLineDO line : lines) {
-            int critical = 0, major = 0, minor = 0;
+            int critical = 0;
+            int major = 0;
+            int minor = 0;
             for (MesQcDefectRecordDO record : records) {
                 if (ObjUtil.notEqual(record.getLineId(), line.getId())) {
                     continue;
@@ -125,4 +117,14 @@ public class MesQcIpqcLineServiceImpl implements MesQcIpqcLineService {
         }
     }
+    @Override
+    public List<MesQcIpqcLineDO> getIpqcLineListByIpqcId(Long ipqcId) {
+        return ipqcLineMapper.selectListByIpqcId(ipqcId);
+    }
+
+    @Override
+    public void deleteListByIpqcId(Long ipqcId) {
+        ipqcLineMapper.deleteByIpqcId(ipqcId);
+    }
+
 }
@@ -104,8 +104,12 @@ public class MesQcIpqcServiceImpl implements MesQcIpqcService {
         adminUserApi.validateUser(updateReqVO.getInspectorUserId());
         itemService.validateItemExists(updateReqVO.getItemId());
-        // 2. 更新
+        // 2. 更新主表
         MesQcIpqcDO updateObj = BeanUtils.toBean(updateReqVO, MesQcIpqcDO.class);
+        updateObj.setSourceDocType(null).setSourceDocId(null).setSourceLineId(null); // 不允许修改来源单据
+        updateObj.setTemplateId(null); // 不允许修改模板
+        updateObj.setItemId(null); // 不允许修改物料
+        updateObj.setWorkOrderId(null); // 不允许修改工单
         ipqcMapper.updateById(updateObj);
     }
@@ -119,9 +123,8 @@ public class MesQcIpqcServiceImpl implements MesQcIpqcService {
         }
         // 2. 更新状态为已完成
-        MesQcIpqcDO updateObj = new MesQcIpqcDO();
-        updateObj.setId(id);
-        updateObj.setStatus(MesQcStatusEnum.FINISHED.getStatus());
+        MesQcIpqcDO updateObj = new MesQcIpqcDO()
+                .setId(id).setStatus(MesQcStatusEnum.FINISHED.getStatus());
         ipqcMapper.updateById(updateObj);
         // TODO @AI（from codex）：如果检验来源是报工，需要对待检产出按检验结果拆分合格/不合格行、生成产出明细、完成产品产出入库、
@@ -137,7 +140,7 @@ public class MesQcIpqcServiceImpl implements MesQcIpqcService {
         // 2.1 删除主表
         ipqcMapper.deleteById(id);
         // 2.2 级联删除行
-        ipqcLineService.deleteByIpqcId(id);
+        ipqcLineService.deleteListByIpqcId(id);
         // 2.3 级联删除缺陷记录
         defectRecordService.deleteListByQcTypeAndQcId(MesQcTypeEnum.IPQC.getType(), id);
     }