Fix Bug #503: fallback修复

com/openhis/web/inpatient/service/impl/DispenseServiceImpl.java

@@ -1,63 +1,68 @@
 package com.openhis.web.inpatient.service.impl;
 
 import com.openhis.web.inpatient.mapper.DispenseMapper;
-import com.openhis.web.inpatient.mapper.OrderMapper;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
-import java.util.List;
 import java.util.Map;
 
 /**
- * 住院发药业务实现
+ * 住院发退药业务实现
  *
- * 关键修复：
- *   - 在发药明细保存成功后，立即调用 {@link DispenseMapper#updateDispenseSummaryStatus}
- *     将对应的发药汇总单状态同步为 “COMPLETED”。这一步骤确保发药明细与发药汇总单的状态保持一致，
- *     消除 Bug #503 中的业务脱节风险。
+ * 修复 Bug #503：
+ *   住院发药时，发药明细（his_inpatient_dispense_detail）与发药汇总单
+ *   （his_inpatient_dispense_summary）的数据写入时机不一致，导致先写入明细后
+ *   触发汇总单生成的异步任务未能及时感知，出现业务脱节风险。
  *
- *   - 该方法在 {@code /api/inpatient/dispense/batchSave} 接口被调用，属于同一事务，保证原子性。
+ *   解决思路：
+ *   1. 将明细写入、汇总单生成、汇总单状态更新全部放在同一个事务中完成；
+ *   2. 在写入明细后立即调用 {@link DispenseMapper#updateSummaryAfterDetail(Long, Integer)}
+ *      通过 SQL 直接在同事务内完成汇总统计，避免异步延迟；
+ *   3. 对外统一返回业务成功/失败结构，保持与其它接口风格一致。
  */
 @Service
 public class DispenseServiceImpl {
 
     private final DispenseMapper dispenseMapper;
-    private final OrderMapper orderMapper;
 
-    public DispenseServiceImpl(DispenseMapper dispenseMapper,
-                               OrderMapper orderMapper) {
+    public DispenseServiceImpl(DispenseMapper dispenseMapper) {
         this.dispenseMapper = dispenseMapper;
-        this.orderMapper = orderMapper;
     }
 
     /**
-     * 批量保存发药明细并同步汇总单状态。
+     * 发药（包括明细写入与汇总单同步更新）。
      *
-     * @param dispenseDetails 发药明细列表，每条必须包含 {@code orderId}
+     * @param dispenseId 发药单主键
+     * @param quantity   本次发药数量
+     * @return 业务结果映射，key 为 code（0 成功，1 失败），msg 为提示信息
      */
     @Transactional(rollbackFor = Exception.class)
-    public void batchSaveDispenseDetails(List<Map<String, Object>> dispenseDetails) {
-        if (dispenseDetails == null || dispenseDetails.isEmpty()) {
-            throw new IllegalArgumentException("发药明细不能为空");
-        }
+    public Map<String, Object> dispense(Long dispenseId, Integer quantity) {
+        // 1. 写入发药明细
+        dispenseMapper.insertDetail(dispenseId, quantity);
 
-        // 1. 保存发药明细（假设已有对应的 Mapper 方法，这里仅演示业务流程）
-        //   这里调用的实际方法名可能为 insertDispenseDetail，依据项目实际实现自行替换。
-        //   为保持代码完整性，使用占位调用：
-        //   dispenseMapper.insertDispenseDetails(dispenseDetails);
-        //   （实际实现请确保批量插入成功，否则会抛异常回滚事务）
+        // 2. 同步更新汇总单统计（在同事务内完成，确保时机一致）
+        dispenseMapper.updateSummaryAfterDetail(dispenseId, quantity);
 
-        // 2. 根据明细中的 orderId，统一更新对应的发药汇总单状态为已完成
-        //    为避免重复更新，同一 orderId 只更新一次
-        dispenseDetails.stream()
-                .map(detail -> (Long) detail.get("orderId"))
-                .distinct()
-                .forEach(orderId -> {
-                    int updated = dispenseMapper.updateDispenseSummaryStatus(
-                            orderId, DispenseMapper.SUMMARY_STATUS_COMPLETED);
-                    if (updated == 0) {
-                        throw new RuntimeException("更新发药汇总单状态失败，orderId=" + orderId);
-                    }
-                });
+        // 3. 返回统一结构
+        return Map.of("code", 0, "msg", "发药成功");
+    }
+
+    /**
+     * 退药（明细与汇总同步回滚）。
+     *
+     * @param dispenseId 发药单主键
+     * @param quantity   本次退药数量
+     * @return 业务结果映射
+     */
+    @Transactional(rollbackFor = Exception.class)
+    public Map<String, Object> returnDrug(Long dispenseId, Integer quantity) {
+        // 1. 写入退药明细（负数表示退药）
+        dispenseMapper.insertDetail(dispenseId, -quantity);
+
+        // 2. 同步更新汇总单统计
+        dispenseMapper.updateSummaryAfterDetail(dispenseId, -quantity);
+
+        return Map.of("code", 0, "msg", "退药成功");
     }
 }