private void checkGoodsStock(Goods goods, Integer buyNum) { //商品为现货商品,判断商品现货可用库存 if (GoodsSaleType.SPOT == GoodsSaleType.findSaleType(goods.getSaleType())) { //单个商品、意向商品 if (GoodsShelveType.SINGLE.getShelveType() == goods.getShelveType() || GoodsShelveType.INTENTION.getShelveType() == goods.getShelveType()) { BaseGoods baseGoods = iBaseGoodsService.getOne(Wrappers.<BaseGoods>lambdaQuery().in(BaseGoods::getModel, goods.getModel()).last("for update")); if (buyNum > baseGoods.getSpotOccupyStock()) { throw new ServiceException(ErrorMessage.STOCK_ERROR); } } //组合商品 if (GoodsShelveType.COMBINATION.getShelveType() == goods.getShelveType()) { //查询组合商品子商品信息 List<GoodsChilds> goodsChildsList = goodsChildsService.lambdaQuery().eq(GoodsChilds::getParentId, goods.getId()).eq(GoodsChilds::getIsGift, 0).list(); if (CollectionUtils.isEmpty(goodsChildsList)) { throw new ServiceException("组合商品子商品信息为空"); } //子商品所有商品型号 List<String> goodsChildsModelList = goodsChildsList.stream().map(GoodsChilds::getModel).collect(Collectors.toList()); List<BaseGoods> baseGoodsList = iBaseGoodsService.list(Wrappers.<BaseGoods>lambdaQuery().in(BaseGoods::getModel, goodsChildsModelList).last("for update")); Map<String, BaseGoods> baseGoodsMap = AppUtils.listToMap(baseGoodsList, BaseGoods::getModel); for (GoodsChilds goodsChilds : goodsChildsList) { BaseGoods baseGoods = baseGoodsMap.get(goodsChilds.getModel()); if (baseGoods == null) { throw new ServiceException("商品型号不存在"); } //子商品总购买数 Integer goodsChildsBuyNum = goodsChilds.getNumberCoefficient().multiply(new BigDecimal(buyNum)).intValue(); if (goodsChildsBuyNum > baseGoods.getSpotOccupyStock()) { throw new ServiceException(ErrorMessage.STOCK_ERROR); } } } } //非现货商品,判断上架库存 if (GoodsSaleType.SPOT != GoodsSaleType.findSaleType(goods.getSaleType()) && buyNum > goods.getStock()) { throw new ServiceException(ErrorMessage.STOCK_ERROR); } }优化该段代码

时间: 2024-03-07 18:49:57 浏览: 21
这段代码可以进行以下优化: 1. 将数据查询和判断库存的逻辑分离,可以提高代码的可读性和可维护性。 2. 对于组合商品的处理,可以使用批量查询替代循环查询。同时,可以将子商品总购买数的计算提取出来,避免重复计算。 3. 可以通过使用缓存来提高查询性能,避免每次都查询数据库。 综上所述,可以进行如下优化: ``` private void checkGoodsStock(Goods goods, Integer buyNum) { //商品为现货商品,判断商品现货可用库存 if (GoodsSaleType.SPOT == GoodsSaleType.findSaleType(goods.getSaleType())) { //单个商品、意向商品 if (GoodsShelveType.SINGLE.getShelveType() == goods.getShelveType() || GoodsShelveType.INTENTION.getShelveType() == goods.getShelveType()) { BaseGoods baseGoods = iBaseGoodsService.getOne(Wrappers.<BaseGoods>lambdaQuery().eq(BaseGoods::getModel, goods.getModel()).last("for update")); int availableStock = baseGoods.getSpotOccupyStock(); if (buyNum > availableStock) { throw new ServiceException(ErrorMessage.STOCK_ERROR); } } //组合商品 else if (GoodsShelveType.COMBINATION.getShelveType() == goods.getShelveType()) { //查询组合商品子商品信息 List<GoodsChilds> goodsChildsList = goodsChildsService.lambdaQuery().eq(GoodsChilds::getParentId, goods.getId()).eq(GoodsChilds::getIsGift, 0).list(); if (CollectionUtils.isEmpty(goodsChildsList)) { throw new ServiceException("组合商品子商品信息为空"); } //子商品所有商品型号 List<String> goodsChildsModelList = goodsChildsList.stream().map(GoodsChilds::getModel).collect(Collectors.toList()); //批量查询子商品库存 List<BaseGoods> baseGoodsList = iBaseGoodsService.list(Wrappers.<BaseGoods>lambdaQuery().in(BaseGoods::getModel, goodsChildsModelList).last("for update")); //使用Map存储子商品库存信息,避免重复查询 Map<String, BaseGoods> baseGoodsMap = baseGoodsList.stream().collect(Collectors.toMap(BaseGoods::getModel, Function.identity())); for (GoodsChilds goodsChilds : goodsChildsList) { BaseGoods baseGoods = baseGoodsMap.get(goodsChilds.getModel()); if (baseGoods == null) { throw new ServiceException("商品型号不存在"); } //计算子商品总购买数 int goodsChildsBuyNum = goodsChilds.getNumberCoefficient().multiply(new BigDecimal(buyNum)).intValue(); int availableStock = baseGoods.getSpotOccupyStock(); if (goodsChildsBuyNum > availableStock) { throw new ServiceException(ErrorMessage.STOCK_ERROR); } } } } //非现货商品,判断上架库存 else if (buyNum > goods.getStock()) { throw new ServiceException(ErrorMessage.STOCK_ERROR); } } ``` 以上代码中,我将查询库存信息和计算可用库存的逻辑分离,使用批量查询和Map来提高查询性能,同时提取子商品总购买数的计算,避免重复计算。

相关推荐

最新推荐

recommend-type

jSP在线教学质量评价系统的设计与实现(源代码)

在线教学质量评价系统可以方便和全面地收集教师教学工作的数据,提供师生网上评教的评分结果,快速集中收集各方面的评教信息,使教务管理部门能够及时了解教学动态和师资情况,为教务老师提供相关决策支持,为职称评聘提供教学工作质量的科学依据,同时减轻了教务老师的工作量。
recommend-type

python-3.10.7-amd64.zip

python-3.10.7-amd64.zip
recommend-type

zigbee-cluster-library-specification

最新的zigbee-cluster-library-specification说明文档。
recommend-type

管理建模和仿真的文件

管理Boualem Benatallah引用此版本:布阿利姆·贝纳塔拉。管理建模和仿真。约瑟夫-傅立叶大学-格勒诺布尔第一大学,1996年。法语。NNT:电话:00345357HAL ID:电话:00345357https://theses.hal.science/tel-003453572008年12月9日提交HAL是一个多学科的开放存取档案馆,用于存放和传播科学研究论文,无论它们是否被公开。论文可以来自法国或国外的教学和研究机构,也可以来自公共或私人研究中心。L’archive ouverte pluridisciplinaire
recommend-type

实现实时数据湖架构:Kafka与Hive集成

![实现实时数据湖架构:Kafka与Hive集成](https://img-blog.csdnimg.cn/img_convert/10eb2e6972b3b6086286fc64c0b3ee41.jpeg) # 1. 实时数据湖架构概述** 实时数据湖是一种现代数据管理架构,它允许企业以低延迟的方式收集、存储和处理大量数据。与传统数据仓库不同,实时数据湖不依赖于预先定义的模式,而是采用灵活的架构,可以处理各种数据类型和格式。这种架构为企业提供了以下优势: - **实时洞察:**实时数据湖允许企业访问最新的数据,从而做出更明智的决策。 - **数据民主化:**实时数据湖使各种利益相关者都可
recommend-type

SQL怎么实现 数据透视表

SQL可以通过使用聚合函数和GROUP BY子句来实现数据透视表。 例如,假设有一个销售记录表,其中包含产品名称、销售日期、销售数量和销售额等信息。要创建一个按照产品名称、销售日期和销售额进行汇总的数据透视表,可以使用以下SQL语句: ``` SELECT ProductName, SaleDate, SUM(SaleQuantity) AS TotalQuantity, SUM(SaleAmount) AS TotalAmount FROM Sales GROUP BY ProductName, SaleDate; ``` 该语句将Sales表按照ProductName和SaleDat
recommend-type

JSBSim Reference Manual

JSBSim参考手册,其中包含JSBSim简介,JSBSim配置文件xml的编写语法,编程手册以及一些应用实例等。其中有部分内容还没有写完,估计有生之年很难看到完整版了,但是内容还是很有参考价值的。
recommend-type

"互动学习:行动中的多样性与论文攻读经历"

多样性她- 事实上SCI NCES你的时间表ECOLEDO C Tora SC和NCESPOUR l’Ingén学习互动,互动学习以行动为中心的强化学习学会互动,互动学习,以行动为中心的强化学习计算机科学博士论文于2021年9月28日在Villeneuve d'Asq公开支持马修·瑟林评审团主席法布里斯·勒菲弗尔阿维尼翁大学教授论文指导奥利维尔·皮耶昆谷歌研究教授:智囊团论文联合主任菲利普·普雷教授,大学。里尔/CRISTAL/因里亚报告员奥利维耶·西格德索邦大学报告员卢多维奇·德诺耶教授,Facebook /索邦大学审查员越南圣迈IMT Atlantic高级讲师邀请弗洛里安·斯特鲁布博士,Deepmind对于那些及时看到自己错误的人...3谢谢你首先,我要感谢我的两位博士生导师Olivier和Philippe。奥利维尔,"站在巨人的肩膀上"这句话对你来说完全有意义了。从科学上讲,你知道在这篇论文的(许多)错误中,你是我可以依
recommend-type

实现实时监控告警系统:Kafka与Grafana整合

![实现实时监控告警系统:Kafka与Grafana整合](https://imgconvert.csdnimg.cn/aHR0cHM6Ly9tbWJpei5xcGljLmNuL21tYml6X2pwZy9BVldpY3ladXVDbEZpY1pLWmw2bUVaWXFUcEdLT1VDdkxRSmQxZXB5R1lxaWNlUjA2c0hFek5Qc3FyRktudFF1VDMxQVl3QTRXV2lhSWFRMEFRc0I1cW1ZOGcvNjQw?x-oss-process=image/format,png) # 1.1 Kafka集群架构 Kafka集群由多个称为代理的服务器组成,这
recommend-type

使用vue3+elementsplus封装一个提示确认框的组件,要求将请求地址和确认框展示信息作为参数暴露出去

好的,下面是使用 Vue3 和 ElementPlus 封装一个提示确认框的组件,同时将请求地址和确认框展示信息作为参数暴露出去。 ```vue <template> <el-dialog title="确认提示" :visible.sync="dialogVisible" width="30%" :before-close="handleClose" > <p>{{ message }}</p> <span slot="footer" class="dialog-footer"> <el-button @click="di