From e03206c08c9604decc4723c36fc25573681f2fee Mon Sep 17 00:00:00 2001
From: liding <756868258@qq.com>
Date: 星期四, 18 六月 2026 15:02:00 +0800
Subject: [PATCH] refactor(database): 优化库存管理相关数据库查询和业务逻辑

---
 src/main/java/com/ruoyi/procurementrecord/service/impl/ProcurementRecordServiceImpl.java |  203 +++++++++++++++++---------------------------------
 1 files changed, 69 insertions(+), 134 deletions(-)

diff --git a/src/main/java/com/ruoyi/procurementrecord/service/impl/ProcurementRecordServiceImpl.java b/src/main/java/com/ruoyi/procurementrecord/service/impl/ProcurementRecordServiceImpl.java
index fecbc59..c94bbd1 100644
--- a/src/main/java/com/ruoyi/procurementrecord/service/impl/ProcurementRecordServiceImpl.java
+++ b/src/main/java/com/ruoyi/procurementrecord/service/impl/ProcurementRecordServiceImpl.java
@@ -2,8 +2,11 @@
 
 import com.baomidou.mybatisplus.core.conditions.query.LambdaQueryWrapper;
 import com.baomidou.mybatisplus.core.metadata.IPage;
+import com.baomidou.mybatisplus.core.toolkit.ObjectUtils;
 import com.baomidou.mybatisplus.extension.plugins.pagination.Page;
 import com.baomidou.mybatisplus.extension.service.impl.ServiceImpl;
+import com.ruoyi.basic.mapper.ProductModelMapper;
+import com.ruoyi.basic.pojo.ProductModel;
 import com.ruoyi.common.utils.OrderUtils;
 import com.ruoyi.common.utils.SecurityUtils;
 import com.ruoyi.common.utils.poi.ExcelUtil;
@@ -14,9 +17,10 @@
 import com.ruoyi.procurementrecord.mapper.ProcurementRecordMapper;
 import com.ruoyi.procurementrecord.mapper.ProcurementRecordOutMapper;
 import com.ruoyi.procurementrecord.pojo.CustomStorage;
-import com.ruoyi.procurementrecord.pojo.ProcurementRecordStorage;
 import com.ruoyi.procurementrecord.pojo.ProcurementRecordOut;
+import com.ruoyi.procurementrecord.pojo.ProcurementRecordStorage;
 import com.ruoyi.procurementrecord.service.ProcurementRecordService;
+import com.ruoyi.procurementrecord.utils.StockUtils;
 import com.ruoyi.project.system.domain.SysUser;
 import com.ruoyi.project.system.mapper.SysUserMapper;
 import com.ruoyi.sales.mapper.SalesLedgerProductMapper;
@@ -29,7 +33,6 @@
 
 import javax.servlet.http.HttpServletResponse;
 import java.math.BigDecimal;
-import java.time.LocalDate;
 import java.time.LocalDateTime;
 import java.time.format.DateTimeFormatter;
 import java.util.*;
@@ -49,6 +52,8 @@
     private final ProcurementRecordOutMapper procurementRecordOutMapper;
 
     private final SalesLedgerProductMapper salesLedgerProductMapper;
+    private final StockUtils stockUtils;
+    private final ProductModelMapper productModelMapper;
 
     @Override
     public List<ProcurementDto> listProcurementBySalesLedgerId(ProcurementDto procurementDto) {
@@ -70,18 +75,18 @@
             List<ProcurementRecordStorage> collect1 = procurementRecordStorages.stream()
                     .filter(procurementRecordStorage -> procurementRecordStorage.getSalesLedgerProductId().equals(dto.getId()))
                     .collect(Collectors.toList());
-            
+
             // 濡傛灉娌℃湁鐩稿叧鐨勫叆搴撹褰曪紝璺宠繃璇ユ潯鏁版嵁
             if(CollectionUtils.isEmpty(collect1)){
                 dto.setQuantity0(dto.getQuantity());
                 continue;
             }
-            
+
             // 璁$畻宸插叆搴撴暟閲忔�诲拰锛屽苟璁剧疆寰呭叆搴撴暟閲�
             BigDecimal totalInboundNum = collect1.stream()
                     .map(ProcurementRecordStorage::getInboundNum)
                     .reduce(BigDecimal.ZERO, BigDecimal::add);
-            
+
             // 寰呭叆搴撴暟閲� = 鎬绘暟閲� - 宸插叆搴撴暟閲�
             dto.setQuantity0(dto.getQuantity().subtract(totalInboundNum));
         }
@@ -593,88 +598,20 @@
 
     @Override
     public IPage<ProcurementPageDtoCopy> listPageCopyByProduction(Page page, ProcurementPageDto procurementDto) {
-        IPage<ProcurementPageDtoCopy> procurementPageDtoCopyIPage = procurementRecordMapper.listPagePRS(page, procurementDto);
+        IPage<ProcurementPageDtoCopy> procurementPageDtoCopyIPage = procurementRecordMapper.listPageCopyByProduction(page, procurementDto);
         List<ProcurementPageDtoCopy> procurementPageDtoCopyList = procurementPageDtoCopyIPage.getRecords();
-        // 璁$畻寰呭叆搴撴暟閲�
-        // 鏌ヨ閲囪喘璁板綍宸插叆搴撴暟閲�
-        List<Integer> collect = procurementPageDtoCopyList.stream().map(ProcurementPageDtoCopy::getId).collect(Collectors.toList());
-        if(CollectionUtils.isEmpty( collect)){
-            return procurementPageDtoCopyIPage;
-        }
-//        LambdaQueryWrapper<ProcurementRecordOut> procurementRecordLambdaQueryWrapper = new LambdaQueryWrapper<>();
-//        procurementRecordLambdaQueryWrapper.in(ProcurementRecordOut::getProcurementRecordStorageId, collect);
-//        procurementRecordLambdaQueryWrapper.eq(ProcurementRecordOut::getType, 2);
-//        List<ProcurementRecordOut> procurementRecords = procurementRecordOutMapper.selectList(procurementRecordLambdaQueryWrapper);
-//        if(CollectionUtils.isEmpty(procurementRecords)){
-//            return procurementPageDtoCopyIPage;
-//        }
-//        for (ProcurementPageDtoCopy dto : procurementPageDtoCopyList) {
-//            // 鏍规嵁閲囪喘鍙拌处ID绛涢�夊搴旂殑鍑哄簱璁板綍
-//            List<ProcurementRecordOut> collect1 = procurementRecords.stream()
-//                    .filter(ProcurementRecordOut -> ProcurementRecordOut.getProcurementRecordStorageId().equals(dto.getId()))
-//                    .collect(Collectors.toList());
-//
-//            // 濡傛灉娌℃湁鐩稿叧鐨勫嚭搴撹褰曪紝璺宠繃璇ユ潯鏁版嵁
-//            if(CollectionUtils.isEmpty(collect1)){
-//                dto.setInboundNum0(dto.getInboundNum());
-//                dto.setTotalInboundNum(BigDecimal.ZERO);
-//                continue;
-//            }
-//
-//            // 璁$畻宸插嚭搴撴暟閲忔�诲拰锛屽苟璁剧疆寰呭嚭搴撴暟閲�
-//            BigDecimal totalInboundNum = collect1.stream()
-//                    .map(ProcurementRecordOut::getInboundNum)
-//                    .reduce(BigDecimal.ZERO, BigDecimal::add);
-//            // 鍑哄簱鏁伴噺 = 鎬绘暟閲� - 寰呭嚭搴撴暟閲�
-//            dto.setTotalInboundNum(totalInboundNum);
-//            // 寰呭嚭搴撴暟閲� = 鎬绘暟閲� - 宸插嚭搴撴暟閲�
-//            dto.setInboundNum0(dto.getInboundNum().subtract(totalInboundNum));
-//            // 搴撳瓨浠峰��
-//            if(dto.getUnitPrice() != null){
-//                dto.setTotalPrice(dto.getTotalInboundNum().multiply(dto.getUnitPrice()));
-//            }
-//        }
-        // 1. 鏌ヨ閲囪喘璁板綍宸插叆搴撶殑鍑哄簱璁板綍锛堟寜storageId鍒嗙粍锛�
-        LambdaQueryWrapper<ProcurementRecordOut> queryWrapper = new LambdaQueryWrapper<>();
-        queryWrapper.in(ProcurementRecordOut::getProcurementRecordStorageId, collect);
-        List<ProcurementRecordOut> recordOutList = procurementRecordOutMapper.selectList(queryWrapper);
-
-        // 2. 鎸塖alesLedgerProductId鍒嗙粍锛岀粺璁℃瘡涓猧d瀵瑰簲鐨勫凡鍑哄簱鏁伴噺鎬诲拰-宸插嚭搴撴暟閲�
-        Map<Integer, BigDecimal> storageIdToTotalOutNumMap = recordOutList.stream()
-                .collect(Collectors.groupingBy(
-                        ProcurementRecordOut::getSalesLedgerProductId,
-                        Collectors.reducing(
-                                BigDecimal.ZERO,
-                                ProcurementRecordOut::getInboundNum,
-                                (a, b) -> a.add(b == null ? BigDecimal.ZERO : b)
-                        )
-                ));
-        // 2. procurementPageDtoCopyList鎸塖alesLedgerProductId鍒嗙粍锛岀粺璁℃瘡涓猧d瀵瑰簲鐨勫凡鍑哄簱鏁伴噺鎬诲拰-鍏ュ簱搴撴暟閲�
-        Map<Long, BigDecimal> storageIdToTotalintNumMap = procurementPageDtoCopyList.stream()
-                .collect(Collectors.groupingBy(
-                        ProcurementPageDtoCopy::getSalesLedgerProductId,
-                        Collectors.reducing(
-                                BigDecimal.ZERO,
-                                ProcurementPageDtoCopy::getInboundNum,
-                                (a, b) -> a.add(b == null ? BigDecimal.ZERO : b)
-                        )
-                ));
-        // 3. 寰幆缁檇to璧嬪��
+        // SQL宸查�氳繃瀛愭煡璇㈣绠椾簡鍑哄簱鏁伴噺(totalInboundNum)鍜屽緟鍑哄簱鏁伴噺(inboundNum0)锛屾澶勮ˉ鍏呭簱瀛樹环鍊�
         for (ProcurementPageDtoCopy dto : procurementPageDtoCopyList) {
-            Integer storageId = dto.getId();
-            Integer salesLedgerProductId = Integer.valueOf(Math.toIntExact(dto.getSalesLedgerProductId()));
-            // 鑾峰彇褰撳墠salesLedgerProductId瀵瑰簲鐨勫凡鍑哄簱鎬绘暟锛堥粯璁�0锛�
-            BigDecimal totalInboundNum = storageIdToTotalOutNumMap.getOrDefault(salesLedgerProductId, BigDecimal.ZERO);
-
-            // 寰呭嚭搴撴暟閲� = 鎬绘暟閲� - 宸插嚭搴撴暟閲忥紙鎬绘暟閲忕┖鍊煎垯榛樿0锛�
-//            BigDecimal totalNum = dto.getInboundNum() == null ? BigDecimal.ZERO : dto.getInboundNum();
-            BigDecimal totalNum = storageIdToTotalintNumMap.getOrDefault(salesLedgerProductId, BigDecimal.ZERO);
-            dto.setInboundNum0(totalNum.subtract(totalInboundNum));
-            // 宸插嚭搴撴暟閲�
-            dto.setTotalInboundNum(totalInboundNum);
-            // 搴撳瓨浠峰�� = 宸插嚭搴撴暟閲� * 鍗曚环锛堝崟浠风┖鍊煎垯榛樿0锛�
+            if (dto.getInboundNum0() == null) {
+                dto.setInboundNum0(dto.getInboundNum());
+            }
+            if (dto.getTotalInboundNum() == null) {
+                dto.setTotalInboundNum(BigDecimal.ZERO);
+            }
             BigDecimal unitPrice = dto.getUnitPrice() == null ? BigDecimal.ZERO : dto.getUnitPrice();
-            dto.setTotalPrice(totalInboundNum.multiply(unitPrice));
+            if (dto.getTotalInboundNum() != null) {
+                dto.setTotalPrice(dto.getTotalInboundNum().multiply(unitPrice));
+            }
         }
         return procurementPageDtoCopyIPage;
     }
@@ -699,42 +636,15 @@
         IPage<CustomStorage> pageList = customStorageMapper.listPageCopyByCustom(page, customStorage);
 
         List<CustomStorage> procurementPageDtoCopyList = pageList.getRecords();
-        // 璁$畻寰呭叆搴撴暟閲�
-        // 鏌ヨ閲囪喘璁板綍宸插叆搴撴暟閲�
-        List<Integer> collect = procurementPageDtoCopyList.stream().map(CustomStorage::getId).collect(Collectors.toList());
-        if(CollectionUtils.isEmpty( collect)){
-            return pageList;
-        }
-        LambdaQueryWrapper<ProcurementRecordOut> procurementRecordLambdaQueryWrapper = new LambdaQueryWrapper<>();
-        procurementRecordLambdaQueryWrapper.in(ProcurementRecordOut::getProcurementRecordStorageId, collect);
-        procurementRecordLambdaQueryWrapper.eq(ProcurementRecordOut::getType, 3);
-        List<ProcurementRecordOut> procurementRecords = procurementRecordOutMapper.selectList(procurementRecordLambdaQueryWrapper);
-        if(CollectionUtils.isEmpty( procurementRecords)){
-            return pageList;
-        }
+        // SQL宸查�氳繃瀛愭煡璇㈣绠椾簡鍑哄簱鏁伴噺(totalInboundNum)鍜屽緟鍑哄簱鏁伴噺(inboundNum0)锛屾澶勮ˉ鍏呭簱瀛樹环鍊�
         for (CustomStorage dto : procurementPageDtoCopyList) {
-            // 鏍规嵁閲囪喘鍙拌处ID绛涢�夊搴旂殑鍑哄簱璁板綍
-            List<ProcurementRecordOut> collect1 = procurementRecords.stream()
-                    .filter(ProcurementRecordOut -> ProcurementRecordOut.getProcurementRecordStorageId().equals(dto.getId()))
-                    .collect(Collectors.toList());
-
-            // 濡傛灉娌℃湁鐩稿叧鐨勫嚭搴撹褰曪紝璺宠繃璇ユ潯鏁版嵁
-            if(CollectionUtils.isEmpty(collect1)){
+            if (dto.getInboundNum0() == null) {
                 dto.setInboundNum0(dto.getInboundNum());
-                dto.setTotalInboundNum(BigDecimal.ZERO);
-                continue;
             }
-
-            // 璁$畻宸插嚭搴撴暟閲忔�诲拰锛屽苟璁剧疆寰呭嚭搴撴暟閲�
-            BigDecimal totalInboundNum = collect1.stream()
-                    .map(ProcurementRecordOut::getInboundNum)
-                    .reduce(BigDecimal.ZERO, BigDecimal::add);
-            // 鍑哄簱鏁伴噺 = 鎬绘暟閲� - 寰呭嚭搴撴暟閲�
-            dto.setTotalInboundNum(totalInboundNum);
-            // 寰呭嚭搴撴暟閲� = 鎬绘暟閲� - 宸插嚭搴撴暟閲�
-            dto.setInboundNum0(dto.getInboundNum().subtract(totalInboundNum));
-            // 搴撳瓨浠峰��
-            if(dto.getTaxInclusiveUnitPrice() != null){
+            if (dto.getTotalInboundNum() == null) {
+                dto.setTotalInboundNum(BigDecimal.ZERO);
+            }
+            if (dto.getTaxInclusiveUnitPrice() != null && dto.getInboundNum0() != null) {
                 dto.setTaxInclusiveTotalPrice(dto.getInboundNum0().multiply(dto.getTaxInclusiveUnitPrice()));
             }
         }
@@ -791,7 +701,10 @@
             procurementRecordLambdaQueryWrapper.eq(ProcurementRecordStorage::getSalesLedgerProductId, detail.getId())
                     .eq(ProcurementRecordStorage::getType, procurementDto.getType());
             Long aLong = procurementRecordMapper.selectCount(procurementRecordLambdaQueryWrapper);
-
+            SalesLedgerProduct salesLedgerProduct = salesLedgerProductMapper.selectById(detail.getId());
+            if (ObjectUtils.isNull(detail.getProductModelId())){
+                detail.setProductModelId(salesLedgerProduct.getProductModelId());
+            }
             ProcurementRecordStorage.ProcurementRecordStorageBuilder procurementRecordBuilder = ProcurementRecordStorage.builder()
                     .salesLedgerProductId(detail.getId())
                     .inboundBatches(aLong.equals(0L) ? "绗�1鎵规("+ procurementDto.getTypeName() +")" : "绗�"+ (aLong + 1) + "鎵规(" + procurementDto.getTypeName() + ")" )
@@ -804,7 +717,9 @@
                     .createUser(loginUser.getUserId())
                     .updateTime(LocalDateTime.now())
                     .updateUser(loginUser.getUserId())
-                    .createBy(procurementDto.getNickName());
+                    .createBy(procurementDto.getNickName())
+                    .productModelId(detail.getProductModelId())
+                    .qualityInspectId(ObjectUtils.isNotNull(procurementDto.getQualityInspectId())?procurementDto.getQualityInspectId():0L);
             this.save(procurementRecordBuilder.build());
             // 鍏ュ簱鎴愬姛鍑忔帀閲囪喘鏁伴噺
 //            LambdaQueryWrapper<SalesLedgerProduct> salesLedgerProductLambdaQueryWrapper = new LambdaQueryWrapper<>();
@@ -863,29 +778,54 @@
     public IPage<ProcurementPageDtoCopy> listPageCopy(Page page, ProcurementPageDto procurementDto) {
         IPage<ProcurementPageDtoCopy> procurementPageDtoCopyIPage = procurementRecordMapper.listPageCopy(page, procurementDto);
         List<ProcurementPageDtoCopy> procurementPageDtoCopyList = procurementPageDtoCopyIPage.getRecords();
+        // SQL宸查�氳繃瀛愭煡璇㈣绠椾簡鍑哄簱鏁伴噺(totalInboundNum)鍜屽緟鍑哄簱鏁伴噺(inboundNum0)锛屾澶勮ˉ鍏呭簱瀛樹环鍊�
+        for (ProcurementPageDtoCopy dto : procurementPageDtoCopyList) {
+            if (dto.getInboundNum0() == null) {
+                dto.setInboundNum0(dto.getInboundNum());
+            }
+            if (dto.getTotalInboundNum() == null) {
+                dto.setTotalInboundNum(BigDecimal.ZERO);
+            }
+            if (dto.getUnitPrice() != null && dto.getInboundNum0() != null) {
+                dto.setTotalPrice(dto.getInboundNum0().multiply(dto.getUnitPrice()));
+            }
+        }
+        return procurementPageDtoCopyIPage;
+    }
+
+    @Override
+    public IPage<ProductModel> listPageProductionStock(Page page, ProcurementPageDto dto) {
+        return productModelMapper.listPageProductionStock(page, dto);
+    }
+
+    @Override
+    public IPage<ProcurementPageDto> listPageByProductProduction(Page page, ProcurementPageDto procurementDto) {
+        IPage<ProcurementPageDto> procurementPageDtoIPage = procurementRecordMapper.listPageByProductProduction(page, procurementDto);
+
+
+        List<ProcurementPageDto> procurementPageDtos = procurementPageDtoIPage.getRecords();
         // 璁$畻寰呭叆搴撴暟閲�
         // 鏌ヨ閲囪喘璁板綍宸插叆搴撴暟閲�
-        List<Integer> collect = procurementPageDtoCopyList.stream().map(ProcurementPageDtoCopy::getId).collect(Collectors.toList());
+        List<Integer> collect = procurementPageDtos.stream().map(ProcurementPageDto::getId).collect(Collectors.toList());
         if(CollectionUtils.isEmpty( collect)){
-            return procurementPageDtoCopyIPage;
+            return procurementPageDtoIPage;
         }
         LambdaQueryWrapper<ProcurementRecordOut> procurementRecordLambdaQueryWrapper = new LambdaQueryWrapper<>();
-        procurementRecordLambdaQueryWrapper.in(ProcurementRecordOut::getProcurementRecordStorageId, collect);
-        procurementRecordLambdaQueryWrapper.eq(ProcurementRecordOut::getType,1);
+        procurementRecordLambdaQueryWrapper.in(ProcurementRecordOut::getProcurementRecordStorageId, collect)
+                .eq(ProcurementRecordOut::getType, 4);
         List<ProcurementRecordOut> procurementRecords = procurementRecordOutMapper.selectList(procurementRecordLambdaQueryWrapper);
         if(CollectionUtils.isEmpty( procurementRecords)){
-            return procurementPageDtoCopyIPage;
+            return procurementPageDtoIPage;
         }
-        for (ProcurementPageDtoCopy dto : procurementPageDtoCopyList) {
+        for (ProcurementPageDto dto : procurementPageDtos) {
             // 鏍规嵁閲囪喘鍙拌处ID绛涢�夊搴旂殑鍑哄簱璁板綍
             List<ProcurementRecordOut> collect1 = procurementRecords.stream()
-                    .filter(ProcurementRecordOut -> ProcurementRecordOut.getProcurementRecordStorageId().equals(dto.getId()) && ProcurementRecordOut.getType().equals(1))
+                    .filter(ProcurementRecordOut -> ProcurementRecordOut.getProcurementRecordStorageId().equals(dto.getId()))
                     .collect(Collectors.toList());
 
             // 濡傛灉娌℃湁鐩稿叧鐨勫嚭搴撹褰曪紝璺宠繃璇ユ潯鏁版嵁
             if(CollectionUtils.isEmpty(collect1)){
                 dto.setInboundNum0(dto.getInboundNum());
-                dto.setTotalInboundNum(BigDecimal.ZERO);
                 continue;
             }
 
@@ -893,16 +833,11 @@
             BigDecimal totalInboundNum = collect1.stream()
                     .map(ProcurementRecordOut::getInboundNum)
                     .reduce(BigDecimal.ZERO, BigDecimal::add);
-            // 鍑哄簱鏁伴噺 = 鎬绘暟閲� - 寰呭嚭搴撴暟閲�
-            dto.setTotalInboundNum(totalInboundNum);
+
             // 寰呭嚭搴撴暟閲� = 鎬绘暟閲� - 宸插嚭搴撴暟閲�
             dto.setInboundNum0(dto.getInboundNum().subtract(totalInboundNum));
-            // 搴撳瓨浠峰��
-            if(dto.getUnitPrice() != null){
-                dto.setTotalPrice(dto.getInboundNum0().multiply(dto.getUnitPrice()));
-            }
         }
-        return procurementPageDtoCopyIPage;
+        return procurementPageDtoIPage;
     }
 
 }

--
Gitblit v1.9.3