feat/ #18 claude 코드 리뷰 반영

Author: warcat12

src/main/java/com/closetnangam/be/domain/catalog/repository/StyleRepository.java

@@ -10,6 +10,8 @@ public interface StyleRepository extends JpaRepository<Style, Long> {
 
     Optional<Style> findByCode(String code);
 
+    List<Style> findByCodeIn(List<String> codes);
+
     List<Style> findAllByOrderByCodeAsc();
 
     boolean existsByCode(String code);

src/main/java/com/closetnangam/be/domain/catalog/service/CategoryCatalogService.java

@@ -140,12 +140,8 @@ public void validateStyleCodes(List<String> styleCodes) {
         if (styleCodes == null || styleCodes.isEmpty()) {
             throw new IllegalArgumentException("스타일은 1개 이상 선택해야 합니다.");
         }
-
         for (String styleCode : styleCodes) {
             StyleCode.fromCode(styleCode);
-            if (!styleRepository.existsByCode(styleCode)) {
-                throw new IllegalArgumentException("존재하지 않는 스타일 코드입니다: " + styleCode);
-            }
         }
     }
 

src/main/java/com/closetnangam/be/domain/clothes/controller/ClothesController.java

@@ -30,6 +30,7 @@ public class ClothesController {
 
     private final ClothesService clothesService;
 
+    // TODO: JWT 인증 구현 후 @PreAuthorize 또는 SecurityContextHolder로 userId 소유권 검증 추가 필요
     @Operation(summary = "보유 옷 목록 조회", description = "사용자 옷장의 보유 옷(OWNED) 목록을 조회합니다.")
     @GetMapping("/api/users/{userId}/clothes")
     public ResponseEntity<ApiResponse<List<ClothesResponse>>> getOwnedClothes(@PathVariable Long userId) {

src/main/java/com/closetnangam/be/domain/clothes/entity/Clothes.java

@@ -22,7 +22,10 @@
 import lombok.NoArgsConstructor;
 
 import java.util.ArrayList;
+import java.util.HashSet;
 import java.util.List;
+import java.util.Set;
+import java.util.stream.Collectors;
 
 @Entity
 @Getter
@@ -136,8 +139,19 @@ public void update(
     }
 
     public void replaceStyleTags(List<ClothesStyleTag> newStyleTags) {
-        this.styleTags.clear();
-        this.styleTags.addAll(newStyleTags);
+        Set<Long> newStyleIds = newStyleTags.stream()
+                .map(tag -> tag.getStyle().getId())
+                .collect(Collectors.toSet());
+
+        this.styleTags.removeIf(existing -> !newStyleIds.contains(existing.getStyle().getId()));
+
+        Set<Long> existingStyleIds = this.styleTags.stream()
+                .map(tag -> tag.getStyle().getId())
+                .collect(Collectors.toSet());
+
+        newStyleTags.stream()
+                .filter(tag -> !existingStyleIds.contains(tag.getStyle().getId()))
+                .forEach(this.styleTags::add);
     }
 
     public void addStyleTag(ClothesStyleTag styleTag) {

src/main/java/com/closetnangam/be/domain/clothes/service/ClothesService.java

@@ -1,5 +1,6 @@
 package com.closetnangam.be.domain.clothes.service;
 
+import com.closetnangam.be.domain.catalog.entity.Style;
 import com.closetnangam.be.domain.catalog.repository.StyleRepository;
 import com.closetnangam.be.domain.catalog.service.CategoryCatalogService;
 import com.closetnangam.be.domain.clothes.dto.request.ClothesCreateRequest;
@@ -129,11 +130,11 @@ private void applyStyleTags(Clothes clothes, List<String> styleCodes) {
     }
 
     private List<ClothesStyleTag> buildStyleTags(Clothes clothes, List<String> styleCodes) {
-        return styleCodes.stream()
-                .map(styleRepository::findByCode)
-                .map(optionalStyle -> optionalStyle.orElseThrow(
-                        () -> new IllegalArgumentException("존재하지 않는 스타일 코드입니다.")
-                ))
+        List<Style> styles = styleRepository.findByCodeIn(styleCodes);
+        if (styles.size() != styleCodes.size()) {
+            throw new IllegalArgumentException("존재하지 않는 스타일 코드가 포함되어 있습니다.");
+        }
+        return styles.stream()
                 .map(style -> ClothesStyleTag.create(clothes, style))
                 .toList();
     }

src/main/java/com/closetnangam/be/global/common/exception/GlobalExceptionHandler.java

@@ -1,6 +1,7 @@
 package com.closetnangam.be.global.common.exception;
 
 import com.closetnangam.be.global.common.response.ApiResponse;
+import org.springframework.dao.DataIntegrityViolationException;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.MethodArgumentNotValidException;
@@ -20,6 +21,11 @@ public ResponseEntity<ApiResponse<Void>> handleIllegalState(IllegalStateExceptio
         return ResponseEntity.status(HttpStatus.CONFLICT).body(ApiResponse.fail(exception.getMessage()));
     }
 
+    @ExceptionHandler(DataIntegrityViolationException.class)
+    public ResponseEntity<ApiResponse<Void>> handleDataIntegrityViolation(DataIntegrityViolationException exception) {
+        return ResponseEntity.status(HttpStatus.CONFLICT).body(ApiResponse.fail("이미 존재하는 데이터입니다."));
+    }
+
     @ExceptionHandler(MethodArgumentNotValidException.class)
     public ResponseEntity<ApiResponse<Void>> handleValidation(MethodArgumentNotValidException exception) {
         String message = exception.getBindingResult().getFieldErrors().stream()