From 25bea4920c05daa4b9004475aee2c851630e263d Mon Sep 17 00:00:00 2001 From: YunaiV Date: Fri, 9 Jun 2023 19:17:25 +0800 Subject: [PATCH] =?UTF-8?q?mall=20+=20trade=EF=BC=9Areview=20=E7=89=A9?= =?UTF-8?q?=E6=B5=81=E8=AE=A1=E7=AE=97?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../DeliveryExpressTemplateService.java | 3 +- .../DeliveryExpressTemplateServiceImpl.java | 15 ++++++--- .../bo/SpuDeliveryExpressTemplateRespBO.java | 5 ++- .../TradeDeliveryPriceCalculator.java | 32 +++++++++++-------- 4 files changed, 35 insertions(+), 20 deletions(-) diff --git a/yudao-module-mall/yudao-module-trade-biz/src/main/java/cn/iocoder/yudao/module/trade/service/delivery/DeliveryExpressTemplateService.java b/yudao-module-mall/yudao-module-trade-biz/src/main/java/cn/iocoder/yudao/module/trade/service/delivery/DeliveryExpressTemplateService.java index 6ec95a27a..3480f4072 100644 --- a/yudao-module-mall/yudao-module-trade-biz/src/main/java/cn/iocoder/yudao/module/trade/service/delivery/DeliveryExpressTemplateService.java +++ b/yudao-module-mall/yudao-module-trade-biz/src/main/java/cn/iocoder/yudao/module/trade/service/delivery/DeliveryExpressTemplateService.java @@ -83,10 +83,11 @@ public interface DeliveryExpressTemplateService { */ DeliveryExpressTemplateDO validateDeliveryExpressTemplate(Long templateId); + // TODO @jason 方法名可以改成 getExpressTemplateMapBySpuIdsAndArea /** * 基于指定的 SPU 编号数组和收件人地址区域编号. 获取匹配运费模板 * - * @param ids SPU 编号列表 + * @param ids SPU 编号列表 // TODO @jason：模版编号？ * @param areaId 区域编号 * @return Map (spuId -> 运费模板设置) */ diff --git a/yudao-module-mall/yudao-module-trade-biz/src/main/java/cn/iocoder/yudao/module/trade/service/delivery/DeliveryExpressTemplateServiceImpl.java b/yudao-module-mall/yudao-module-trade-biz/src/main/java/cn/iocoder/yudao/module/trade/service/delivery/DeliveryExpressTemplateServiceImpl.java index 87b2d4920..07e80b5f3 100644 --- a/yudao-module-mall/yudao-module-trade-biz/src/main/java/cn/iocoder/yudao/module/trade/service/delivery/DeliveryExpressTemplateServiceImpl.java +++ b/yudao-module-mall/yudao-module-trade-biz/src/main/java/cn/iocoder/yudao/module/trade/service/delivery/DeliveryExpressTemplateServiceImpl.java @@ -5,7 +5,10 @@ import cn.hutool.core.lang.Assert; import cn.iocoder.yudao.framework.common.pojo.PageResult; import cn.iocoder.yudao.module.product.api.spu.ProductSpuApi; import cn.iocoder.yudao.module.product.api.spu.dto.ProductSpuRespDTO; -import cn.iocoder.yudao.module.trade.controller.admin.delivery.vo.expresstemplate.*; +import cn.iocoder.yudao.module.trade.controller.admin.delivery.vo.expresstemplate.DeliveryExpressTemplateCreateReqVO; +import cn.iocoder.yudao.module.trade.controller.admin.delivery.vo.expresstemplate.DeliveryExpressTemplateDetailRespVO; +import cn.iocoder.yudao.module.trade.controller.admin.delivery.vo.expresstemplate.DeliveryExpressTemplatePageReqVO; +import cn.iocoder.yudao.module.trade.controller.admin.delivery.vo.expresstemplate.DeliveryExpressTemplateUpdateReqVO; import cn.iocoder.yudao.module.trade.dal.dataobject.delivery.DeliveryExpressTemplateChargeDO; import cn.iocoder.yudao.module.trade.dal.dataobject.delivery.DeliveryExpressTemplateDO; import cn.iocoder.yudao.module.trade.dal.dataobject.delivery.DeliveryExpressTemplateFreeDO; @@ -234,14 +237,16 @@ public class DeliveryExpressTemplateServiceImpl implements DeliveryExpressTempla List templateList = expressTemplateMapper.selectBatchIds(spuMap.keySet()); Map result = new HashMap<>(templateList.size()); templateList.forEach(item -> { + // TODO @jason：if return ，更简洁哈； if (spuMap.containsKey(item.getId())) { - ProductSpuRespDTO spuDTO = spuMap.get(item.getId()); + ProductSpuRespDTO spu = spuMap.get(item.getId()); SpuDeliveryExpressTemplateRespBO bo = new SpuDeliveryExpressTemplateRespBO() - .setSpuId(spuDTO.getId()).setAreaId(areaId) + .setSpuId(spu.getId()).setAreaId(areaId) .setChargeMode(item.getChargeMode()) + // TODO @jason：是不是只要查询到一个，就不用查询下一个了；TemplateCharge 和 TemplateFree .setTemplateCharge(findMatchExpressTemplateCharge(item.getId(), areaId)) .setTemplateFree(findMatchExpressTemplateFree(item.getId(), areaId)); - result.put(spuDTO.getId(), bo); + result.put(spu.getId(), bo); } }); return result; @@ -249,6 +254,7 @@ public class DeliveryExpressTemplateServiceImpl implements DeliveryExpressTempla private DeliveryExpressTemplateChargeDO findMatchExpressTemplateCharge(Long templateId, Integer areaId) { List list = expressTemplateChargeMapper.selectListByTemplateId(templateId); + // TODO @jason：可以使用 CollectionUtils.findFirst() for (DeliveryExpressTemplateChargeDO item : list) { // 第一个匹配的返回。 areaId 不能重复 if (item.getAreaIds().contains(areaId)) { @@ -260,6 +266,7 @@ public class DeliveryExpressTemplateServiceImpl implements DeliveryExpressTempla private DeliveryExpressTemplateFreeDO findMatchExpressTemplateFree(Long templateId, Integer areaId) { List list = expressTemplateFreeMapper.selectListByTemplateId(templateId); + // TODO @jason：可以使用 CollectionUtils.findFirst() for (DeliveryExpressTemplateFreeDO item : list) { // 第一个匹配的返回。 areaId 不能重复 if (item.getAreaIds().contains(areaId)) { diff --git a/yudao-module-mall/yudao-module-trade-biz/src/main/java/cn/iocoder/yudao/module/trade/service/delivery/bo/SpuDeliveryExpressTemplateRespBO.java b/yudao-module-mall/yudao-module-trade-biz/src/main/java/cn/iocoder/yudao/module/trade/service/delivery/bo/SpuDeliveryExpressTemplateRespBO.java index 87d04001b..2d5568915 100644 --- a/yudao-module-mall/yudao-module-trade-biz/src/main/java/cn/iocoder/yudao/module/trade/service/delivery/bo/SpuDeliveryExpressTemplateRespBO.java +++ b/yudao-module-mall/yudao-module-trade-biz/src/main/java/cn/iocoder/yudao/module/trade/service/delivery/bo/SpuDeliveryExpressTemplateRespBO.java @@ -13,9 +13,11 @@ import lombok.Data; @Data public class SpuDeliveryExpressTemplateRespBO { + // TODO @jason：是不是可以简单一点；是否包邮；然后直接把 templateCharge、templateFree 需要的字段平铺开 + /** * 配送计费方式 - *

+ * * 枚举 {@link DeliveryExpressChargeModeEnum} */ private Integer chargeMode; @@ -30,6 +32,7 @@ public class SpuDeliveryExpressTemplateRespBO { */ private DeliveryExpressTemplateFreeDO templateFree; + // TODO @jason：下面两个字段不用返回也可以呀 /** * SPU 编号 *

diff --git a/yudao-module-mall/yudao-module-trade-biz/src/main/java/cn/iocoder/yudao/module/trade/service/price/calculator/TradeDeliveryPriceCalculator.java b/yudao-module-mall/yudao-module-trade-biz/src/main/java/cn/iocoder/yudao/module/trade/service/price/calculator/TradeDeliveryPriceCalculator.java index 46399b203..95e45182d 100644 --- a/yudao-module-mall/yudao-module-trade-biz/src/main/java/cn/iocoder/yudao/module/trade/service/price/calculator/TradeDeliveryPriceCalculator.java +++ b/yudao-module-mall/yudao-module-trade-biz/src/main/java/cn/iocoder/yudao/module/trade/service/price/calculator/TradeDeliveryPriceCalculator.java @@ -54,33 +54,34 @@ public class TradeDeliveryPriceCalculator implements TradePriceCalculator { AddressRespDTO address = addressApi.getAddress(param.getAddressId(), param.getUserId()); Assert.notNull(address, "收件人({})的地址，不能为空", param.getUserId()); - //1.3 过滤出已选中的商品SKU + // 2. 过滤出已选中的商品SKU List selectedItem = filterList(result.getItems(), OrderItem::getSelected); - + Set spuIds = convertSet(selectedItem, OrderItem::getSpuId); Map spuExpressTemplateMap = - deliveryExpressTemplateService.getExpressTemplateBySpuIdsAndArea( - convertSet(selectedItem, OrderItem::getSpuId), address.getAreaId()); + deliveryExpressTemplateService.getExpressTemplateBySpuIdsAndArea(spuIds, address.getAreaId()); - // 1.4 计算配送费用 + // 3. 计算配送费用 + // TODO @jason：这里应该不能判断空；如果找不到模版，就要报错；不然配送费就亏了 if (CollUtil.isNotEmpty(spuExpressTemplateMap)) { calculateDeliveryPrice(selectedItem, spuExpressTemplateMap, result); } - } private void calculateDeliveryPrice(List selectedSkus, Map spuExpressTemplateMap, TradePriceCalculateRespBO result) { + // 得到 SKU 详情 + // TODO @jason：可以去掉这里的读取；在 TradePriceCalculateRespBO 初始化的时候，把 weight、volume 拿到 Set skuIds = convertSet(selectedSkus, OrderItem::getSkuId); - // 得到SKU 详情。得到 重量体积 Map skuRespMap = convertMap(productSkuApi.getSkuList(skuIds), ProductSkuRespDTO::getId); - // 按spu 来计算商品的运费 一个 spuId 可能对应多条订单商品 SKU, + // 按 SPU 来计算商品的运费：一个 spuId 可能对应多条订单商品 SKU Map> spuIdItemMap = convertMultiMap(selectedSkus, OrderItem::getSpuId); // 依次计算每个 SPU 的快递运费 for (Map.Entry> entry : spuIdItemMap.entrySet()) { Long spuId = entry.getKey(); List orderItems = entry.getValue(); + // TODO @jason：如果找不到，则打印 error log SpuDeliveryExpressTemplateRespBO templateBO = spuExpressTemplateMap.get(spuId); if (templateBO == null) { // 记录错误日志 @@ -93,18 +94,18 @@ public class TradeDeliveryPriceCalculator implements TradePriceCalculator { double totalVolume = 0; for (OrderItem orderItem : orderItems) { totalCount += orderItem.getCount(); - totalPrice += orderItem.getPayPrice(); // 先按应付总金额来算，后面确认一下 + totalPrice += orderItem.getPayPrice(); // 先按应付总金额来算，后面确认一下 TODO @jason：是的哈 ProductSkuRespDTO skuResp = skuRespMap.get(orderItem.getSkuId()); - if (skuResp != null) { - totalWeight = totalWeight + skuResp.getWeight() * orderItem.getCount(); - totalVolume = totalVolume + skuResp.getVolume() * orderItem.getCount(); - } + // TODO @jason：是不是要保持风格统一，都用 += + totalWeight = totalWeight + skuResp.getWeight() * orderItem.getCount(); + totalVolume = totalVolume + skuResp.getVolume() * orderItem.getCount(); } // 优先判断是否包邮. 如果包邮不计算快递运费 if (checkExpressFree(templateBO.getChargeMode(), totalCount, totalWeight, totalVolume, totalPrice, templateBO.getTemplateFree())) { continue; } + // TODO @jason：这块判断，可以收到 calculateExpressFeeByChargeMode 里；另外找不到，要打 error log if (templateBO.getTemplateCharge() == null) { continue; } @@ -175,6 +176,7 @@ public class TradeDeliveryPriceCalculator implements TradePriceCalculator { * @param orderItems SKU 商品 */ private void divideDeliveryPrice(int deliveryPrice, List orderItems) { + // TODO @jason：分摊的话，是不是要按照比例呀？重量、价格、数量等等 int dividePrice = deliveryPrice / orderItems.size(); for (OrderItem item : orderItems) { // 更新快递运费 @@ -192,6 +194,7 @@ public class TradeDeliveryPriceCalculator implements TradePriceCalculator { * @param totalPrice 总金额 * @param templateFree 包邮配置 */ + // TODO @jason：isExpressFree 更合适；因为 check 是一种校验，往往抛出异常； private boolean checkExpressFree(Integer chargeMode, int totalCount, double totalWeight, double totalVolume, int totalPrice, DeliveryExpressTemplateFreeDO templateFree) { if (templateFree == null) { @@ -206,7 +209,8 @@ public class TradeDeliveryPriceCalculator implements TradePriceCalculator { } break; case WEIGHT: - // freeCount 是不是应该是 double ?? + // freeCount 是不是应该是 double ?? + // TODO @jason：要不配置的时候，把它的单位和商品对齐？到底是 kg、还是斤 if (totalWeight >= templateFree.getFreeCount() && totalPrice >= templateFree.getFreePrice()) { return true;