Merge pull request #167 from THEGOODs-repo/bug/166

[Bug/166] 장바구니 상품 삭제 API 로직 수정

src/main/java/com/umc/TheGoods/apiPayload/code/status/ErrorStatus.java

@@ -42,6 +42,7 @@ public enum ErrorStatus implements BaseErrorCode {
     NOT_CART_OWNER(HttpStatus.BAD_REQUEST, "CART4002", "본인의 장바구니 내역이 아닙니다. 접근할 수 없습니다."),
     CART_DETAIL_FOUND_ERROR(HttpStatus.INTERNAL_SERVER_ERROR, "CART4003", "단일 상품의 장바구니 상세 내역을 찾을 수 없습니다. 관리자에게 문의 바랍니다."),
     CART_DETAIL_NOT_FOUND(HttpStatus.NOT_FOUND, "CART4004", "해당 장바구니 상세 내역을 찾을 수 없습니다"),
+    DELETE_CART_DETAIL_FAILED(HttpStatus.BAD_REQUEST, "CART4005", "장바구니 상세 내역을 삭제할 수 없습니다."),
 
     // test
     TEMP_EXCEPTION(HttpStatus.BAD_REQUEST, "TEMP4001", "테스트"),
@@ -61,11 +62,11 @@ public enum ErrorStatus implements BaseErrorCode {
 
     //JWT
 
-    JWT_BAD_REQUEST(HttpStatus.UNAUTHORIZED, "JWT4001","잘못된 JWT 서명입니다."),
-    JWT_ACCESS_TOKEN_EXPIRED(HttpStatus.UNAUTHORIZED, "JWT4002","액세스 토큰이 만료되었습니다."),
-    JWT_REFRESH_TOKEN_EXPIRED(HttpStatus.UNAUTHORIZED, "JWT4003","리프레시 토큰이 만료되었습니다. 다시 로그인하시기 바랍니다."),
-    JWT_UNSUPPORTED_TOKEN(HttpStatus.UNAUTHORIZED, "JWT4004","지원하지 않는 JWT 토큰입니다."),
-    JWT_TOKEN_NOT_FOUND(HttpStatus.UNAUTHORIZED, "JWT4005","유효한 JWT 토큰이 없습니다."),
+    JWT_BAD_REQUEST(HttpStatus.UNAUTHORIZED, "JWT4001", "잘못된 JWT 서명입니다."),
+    JWT_ACCESS_TOKEN_EXPIRED(HttpStatus.UNAUTHORIZED, "JWT4002", "액세스 토큰이 만료되었습니다."),
+    JWT_REFRESH_TOKEN_EXPIRED(HttpStatus.UNAUTHORIZED, "JWT4003", "리프레시 토큰이 만료되었습니다. 다시 로그인하시기 바랍니다."),
+    JWT_UNSUPPORTED_TOKEN(HttpStatus.UNAUTHORIZED, "JWT4004", "지원하지 않는 JWT 토큰입니다."),
+    JWT_TOKEN_NOT_FOUND(HttpStatus.UNAUTHORIZED, "JWT4005", "유효한 JWT 토큰이 없습니다."),
 
     //POST
 

src/main/java/com/umc/TheGoods/domain/order/Cart.java

@@ -49,21 +49,4 @@ public void setItem(Item item) {
         item.getItemCartList().add(this);
     }
 
-    public void detachMember() {
-        if (this.member != null) {
-            this.member.getCartList().remove(this);
-        }
-    }
-
-    public void detachItem() {
-        if (this.item != null) {
-            this.item.getItemCartList().remove(this);
-        }
-    }
-
-    public void detachCartDetail() {
-        this.cartDetailList = new ArrayList<>();
-    }
-
-
 }

src/main/java/com/umc/TheGoods/domain/order/CartDetail.java

@@ -40,12 +40,14 @@ public void setCart(Cart cart) {
     public void detachCart() {
         if (this.cart != null) {
             this.cart.getCartDetailList().remove(this);
+            this.cart = null;
         }
     }
 
     public void detachItemOption() {
         if (this.itemOption != null) {
             this.itemOption.getCartDetailList().remove(this);
+            this.itemOption = null;
         }
     }
 

src/main/java/com/umc/TheGoods/repository/cart/CartRepository.java

@@ -4,9 +4,6 @@
 import com.umc.TheGoods.domain.member.Member;
 import com.umc.TheGoods.domain.order.Cart;
 import org.springframework.data.jpa.repository.JpaRepository;
-import org.springframework.data.jpa.repository.Modifying;
-import org.springframework.data.jpa.repository.Query;
-import org.springframework.data.repository.query.Param;
 
 import java.util.List;
 import java.util.Optional;
@@ -17,7 +14,4 @@ public interface CartRepository extends JpaRepository<Cart, Long> {
 
     List<Cart> findAllByMember(Member member);
 
-    @Modifying
-    @Query("delete from Cart c where c.id=:id")
-    void deleteCartById(@Param("id") Long cartId);
 }

src/main/java/com/umc/TheGoods/service/CartService/CartCommandServiceImpl.java

@@ -172,31 +172,45 @@ public List<CartDetail> updateCart(CartRequestDTO.cartUpdateDTOList request, Mem
     @Override
     public void deleteCartDetail(CartRequestDTO.cartDetailDeleteDTO request, Member member) {
 
-        request.getCartDetailIdList().forEach(cartDetailId -> {
+        Cart firstCart = cartDetailRepository.findById(request.getCartDetailIdList().get(0)).orElseThrow(() -> new OrderHandler(ErrorStatus.CART_DETAIL_NOT_FOUND)).getCart();
 
+        // request의 cartDetailIdList로 cartDetailList 생성
+        List<CartDetail> cartDetailList = request.getCartDetailIdList().stream().map(cartDetailId -> {
             CartDetail cartDetail = cartDetailRepository.findById(cartDetailId).orElseThrow(() -> new OrderHandler(ErrorStatus.CART_DETAIL_NOT_FOUND));
 
-            Cart cart = cartDetail.getCart();
-
             // 해당 cart 내역을 수정할 권한 있는지 검증
             if (!cartDetail.getCart().getMember().equals(member)) {
                 throw new OrderHandler(ErrorStatus.NOT_CART_OWNER);
             }
-            cartDetail.detachCart();
-            cartDetail.detachItemOption();
-
-            cartDetailRepository.deleteById(cartDetail.getId());
-            cartDetailRepository.flush();
-
-            // 장바구니 상품의 마지막 옵션 내역을 삭제한 경우: 장바구니 상품 내역도 삭제
-            if (cart.getCartDetailList().isEmpty()) {
-                log.info(" ===================== 마지막 옵션 삭제 ========================");
-                cart.detachMember();
-                cart.detachItem();
-                cartRepository.deleteCartById(cart.getId());
+
+            // cartDetailId가 모두 동일한 장바구니 내역에 대한 담은 옵션이 맞는지 검증
+            if (!firstCart.equals(cartDetail.getCart())) {
+                throw new OrderHandler(ErrorStatus.DELETE_CART_DETAIL_FAILED);
             }
-        });
+            return cartDetail;
+        }).collect(Collectors.toList());
 
+
+        // 해당 장바구니 상품의 모든 담은 옵션 list 조회
+        List<CartDetail> originCartDetailList = firstCart.getCartDetailList();
+
+        List<Long> cartDetailIdList = cartDetailList.stream().map(cartDetail -> {
+            return cartDetail.getId();
+        }).collect(Collectors.toList());
+        List<Long> originCartDetailIdList = originCartDetailList.stream().map(originCartDetail -> {
+            return originCartDetail.getId();
+        }).collect(Collectors.toList());
+
+        // 장바구니 삭제
+        if (isEqualCartDetailList(cartDetailIdList, originCartDetailIdList)) { // 해당 장바구니 상품 내역의 담은 옵션 모두를 삭제하는 경우
+            cartRepository.deleteById(firstCart.getId());
+        } else { // 해당 장바구니 생품 내역의 담은 옵션 일부만 삭제하는 경우
+            cartDetailList.forEach(cartDetail -> {
+                cartDetail.detachCart();
+                cartDetail.detachItemOption();
+                cartDetailRepository.deleteById(cartDetail.getId());
+            });
+        }
     }
 
     @Override
@@ -209,29 +223,23 @@ public void deleteCart(CartRequestDTO.cartDeleteDTO request, Member member) {
                 throw new OrderHandler(ErrorStatus.NOT_CART_OWNER);
             }
 
-            List<CartDetail> cartDetailList = cartDetailRepository.findAllByCart(cart);
-            cartDetailList.forEach(cartDetail -> {
-                log.info("===== cartDetailId: {}", cartDetail.getId());
-                cartDetail.detachCart();
-                cartDetail.detachItemOption();
+            cartRepository.deleteById(cart.getId());
+        });
 
-                cartDetailRepository.deleteById(cartDetail.getId());
-            });
+    }
 
-//            cart.getCartDetailList().forEach(cartDetail -> {
-//                log.info("===== cartDetailId: {}", cartDetail.getId());
-//                cartDetail.detachCart();
-//                cartDetail.detachItemOption();
-//
-//                cartDetailRepository.deleteById(cartDetail.getId());
-//            });
+    private boolean isEqualCartDetailList(List<Long> x, List<Long> y) {
+        if (x == null) {
+            return y == null;
+        }
 
-            cartDetailRepository.flush();
+        if (x.size() != y.size()) {
+            return false;
+        }
 
-            cart.detachMember();
-            cart.detachItem();
-            cartRepository.deleteCartById(cart.getId());
-        });
+        x = x.stream().sorted().collect(Collectors.toList());
+        y = y.stream().sorted().collect(Collectors.toList());
 
+        return x.equals(y);
     }
 }