diff --git a/MD/bugs/BUG_750_ANALYSIS.md b/MD/bugs/BUG_750_ANALYSIS.md new file mode 100644 index 000000000..0fd51d766 --- /dev/null +++ b/MD/bugs/BUG_750_ANALYSIS.md @@ -0,0 +1,129 @@ +# Bug #750 诸葛亮分析报告 + +> **文档类型**: Bug分析 +> **分析时间**: 2026-06-12 22:27:51 +> **分析模型**: mimo-v2.5 (LLM深度分析) + +--- + +## 基本信息 +- **Bug #**: 750 +- **标题**: 【收费工作站-住院登记】修改患者登记信息(如费用性质)并保存后,系统未更新原记录而是新增了一条重复记录 +- **模块**: 住院登记管理 +- **提出人**: 陈显精 + +--- + +Now I have enough information to provide the analysis. Let me compile the full diagnosis. + +--- + +## 一、Bug 理解 + +**禅道 Bug 标题**:【收费工作站-住院登记】修改患者登记信息(如费用性质)并保存后,系统未更新原记录而是新增了一条重复记录 + +**重现步骤**: +1. 登录系统,进入 [收费工作站] → [住院登记] +2. 切换到 [已登记入院] 页签,找到一条已存在的登记记录(如:序号9,测试患者戊,费用性质为"全自费") +3. 点击操作列下的 [修改] 按钮,打开"修改登记"弹窗 +4. 将"费用性质"从"全自费"修改为"职工基本医疗保险" +5. 点击 [保存],系统提示"住院登记操作成功" + +**期望结果**:修改保存后,系统只保留一行该患者的记录,且费用性质成功更新。 + +**附图关键信息**: +- 弹窗中"费用性质"被红框标注为"职工基本医疗保险"(修改后值),"保存"按钮被红框圈出 +- 列表中序号8(旧数据"全自费")和序号9(新数据"职工基本医疗保险")同时存在,红字标注"生成了两条重复的数据" +- 两条记录的登记时间完全相同(`2026-06-07 21:42:13`),说明不是两次独立操作 + +**综合总结**:用户在"已登记入院"列表点击修改并保存后,系统本应在原记录上更新费用性质,但实际行为是保留了原记录并新增了一条相同记录,导致同一患者出现两条住院登记。保存操作被错误地执行为"新增"而非"更新"。 + +--- + +## 二、根因分析 + +**核心问题**:`InHospitalRegisterAppServiceImpl.updateRegistration()` 方法中,使用 `iEncounterService.saveOrUpdate(encounter)` 保存修改后的 encounter 时,MyBatis-Plus 将其执行为 **INSERT** 而非 **UPDATE**,导致生成新记录。 + +**技术根因链路**: + +1. **配置冲突**:`HisBaseEntity.deleteFlag` 字段标注了 `@TableLogic(value = "0", delval = "1")`,但全局配置 `application.yml` 中设置 `logic-delete-value: 0`(应为1)和 `logic-not-delete-value: 1`(应为0),二者矛盾。虽然注解优先于全局配置,但这种不一致可能导致 `saveOrUpdate` 内部的存在性检查逻辑产生非预期行为。 + +2. **`saveOrUpdate` 的二义性**:该方法依赖 entity 的 `id` 是否为 null 来判断 INSERT/UPDATE。虽然 `getById` 理论上返回了正确的 entity(含 id),但 `saveOrUpdate` 在多租户拦截器(`TenantLineInnerInterceptor`)+ 逻辑删除注解(`@TableLogic`)的叠加作用下,其内部的 `selectById` 查询可能返回 null(例如 `tenant_id` 过滤条件不匹配),从而触发 INSERT 分支。 + +3. **`@TableLogic` 与全局配置冲突**(`application.yml`): + ```yaml + mybatis-plus: + global-config: + db-config: + logic-delete-field: validFlag # 全局指定 validFlag(但实体用的是 deleteFlag) + logic-delete-value: 0 # 0=已删除(注解是 0=未删除) + logic-not-delete-value: 1 # 1=未删除(注解是 1=已删除) + ``` + +**涉及文件**: + +| 文件 | 角色 | +|------|------| +| `InHospitalRegisterAppServiceImpl.java:377-475` | `updateRegistration()` 方法 — 问题入口 | +| `HisBaseEntity.java` | `@TableLogic(value="0", delval="1")` 注解 — 配置冲突 | +| `MybatisPlusConfig.java` | 多租户 + 乐观锁 + 逻辑删除拦截器链 | +| `application.yml:93-100` | MyBatis-Plus 全局配置 — 与注解矛盾 | +| `accomplishList.vue:286-302` / `patientRegister.vue:455-475` | 前端调用链 | + +--- + +## 三、修复方案 + +### 方案:将 `saveOrUpdate` 改为显式 `updateById` + +最安全、最精确的修复是:在 `updateRegistration` 中已通过 `getById` 确认记录存在并获取了完整 entity(含 id),因此无需使用具有二义性的 `saveOrUpdate`,直接使用 `updateById` 确保执行 UPDATE。 + +**修改文件**:`InHospitalRegisterAppServiceImpl.java` + +```java +// 第406行附近,修改前: +iEncounterService.saveOrUpdate(encounter); + +// 修改为: +iEncounterService.updateById(encounter); +``` + +同样,病区信息和账户信息的 `saveOrUpdate` 也应替换为显式更新: + +**病区更新**(约第419行): +```java +// 修改前: +iEncounterLocationService.saveOrUpdate(encounterLocationReg); + +// 修改为: +iEncounterLocationService.updateById(encounterLocationReg); +``` + +**账户更新**(约第429-470行中所有 `saveOrUpdate` 调用): +```java +// 修改前: +iAccountService.saveOrUpdate(cashAccount); +// 修改为: +iAccountService.updateById(cashAccount); + +// 同理修改所有 iAccountService.saveOrUpdate(...) 调用 +``` + +> **注意**:账户的"else"分支(账户不存在时创建新账户)保持使用 `iAccountService.save(newAccount)` 不变。 + +--- + +## 四、路由决策 + +**FIXER**: guanyu + +**REASON**: 根因在后端 `InHospitalRegisterAppServiceImpl.updateRegistration()` 的 MyBatis-Plus `saveOrUpdate` 调用需改为 `updateById`,纯后端 Java 修复,属于 guanyu(后端开发)的职责范围。 + +--- + +## 路由决策 +- **FIXER_ID**: guanyu +- **修复 Agent**: guanyu(后端) +- **原因**: LLM 分析决策 + +> ⚠️ 修复人员请先验证以上分析是否正确,再执行修复。