From 07b964209ad7ff91def2ed1962c5a02441710d86 Mon Sep 17 00:00:00 2001
From: Administrator <15274802129@163.com>
Date: Fri, 12 Jun 2026 09:48:25 +0800
Subject: [PATCH] refactor(mall): 重构DTO和控制器结构并添加验证注解
---
src/main/java/cc/mrbird/febs/mall/service/impl/ApiMallShoppingCartServiceImpl.java | 39 ++++++++-------------------------------
1 files changed, 8 insertions(+), 31 deletions(-)
diff --git a/src/main/java/cc/mrbird/febs/mall/service/impl/ApiMallShoppingCartServiceImpl.java b/src/main/java/cc/mrbird/febs/mall/service/impl/ApiMallShoppingCartServiceImpl.java
index ad51eb4..d176abc 100644
--- a/src/main/java/cc/mrbird/febs/mall/service/impl/ApiMallShoppingCartServiceImpl.java
+++ b/src/main/java/cc/mrbird/febs/mall/service/impl/ApiMallShoppingCartServiceImpl.java
@@ -50,41 +50,18 @@
public void addGoodsToCart(List<AddCartDto> addCartDtoList) {
Long memberId = LoginUserUtil.getLoginUser().getId();
for (AddCartDto addCartDto : addCartDtoList) {
- Long goodsId = null;
- Long skuId = null;
- Long styleId = null;
- MallShoppingCart cartGoods = null;
+ addCartDto.setType(1);
+ MallGoodsSku sku = mallGoodsSkuMapper.selectById(addCartDto.getSkuId());
- if (addCartDto.getType() == null) {
- addCartDto.setType(1);
+ if (sku == null) {
+ throw new FebsException("sku不存在");
}
- if (addCartDto.getType() == 1) {
- MallGoodsSku sku = mallGoodsSkuMapper.selectById(addCartDto.getSkuId());
+ Long goodsId = sku.getGoodsId();
+ Long skuId = sku.getId();
+ Long styleId = sku.getStyleId();
- if (sku == null) {
- throw new FebsException("sku不存在");
- }
-
- goodsId = sku.getGoodsId();
- skuId = sku.getId();
- styleId = sku.getStyleId();
-
- cartGoods = this.baseMapper.selectCartGoodsBySkuId(addCartDto.getSkuId(), memberId);
- } else {
- MallGoods mallGoods = mallGoodsMapper.selectById(addCartDto.getSkuId());
- if (mallGoods == null) {
- throw new FebsException("积分商品不存在");
- }
-
- goodsId = mallGoods.getId();
-
- LambdaQueryWrapper<MallShoppingCart> cartQuery = new LambdaQueryWrapper<>();
- cartQuery.eq(MallShoppingCart::getGoodsId, goodsId)
- .eq(MallShoppingCart::getMemberId, memberId)
- .last("limit 1");
- cartGoods = this.baseMapper.selectOne(cartQuery);
- }
+ MallShoppingCart cartGoods = this.baseMapper.selectCartGoodsBySkuId(addCartDto.getSkuId(), memberId);
if (cartGoods == null) {
cartGoods = new MallShoppingCart();
--
Gitblit v1.9.1