Fix: 코드 리뷰 사항 반영

- 서비스 레이어의 public 메서드를 상단에 배치하여 가독성 향상
- CQRS 적용을 위한 Query, Command 서비스 클래스에 @Transactinal(readOnly = true), @Transactinal 어노테이션을 각각 추가
- 레포지토리의 불필요한 @query 어노테이션 삭제

Author: 7zrv

src/main/java/com/somemore/center/repository/PreferItemRepository.java

@@ -2,7 +2,6 @@
 
 import com.somemore.center.domain.PreferItem;
 import org.springframework.data.jpa.repository.JpaRepository;
-import org.springframework.data.jpa.repository.Query;
 import org.springframework.data.repository.query.Param;
 import org.springframework.stereotype.Repository;
 
@@ -12,7 +11,6 @@
 @Repository
 public interface PreferItemRepository extends JpaRepository<PreferItem, Long> {
 
-    @Query("SELECT p FROM PreferItem p WHERE p.centerId = :centerId")
     List<PreferItem> findByCenterId(@Param("centerId") UUID centerId);
 
 }

src/main/java/com/somemore/center/service/command/CreatePreferItemService.java

@@ -6,8 +6,10 @@
 import com.somemore.center.usecase.query.CenterQueryUseCase;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
+import org.springframework.transaction.annotation.Transactional;
 
 @RequiredArgsConstructor
+@Transactional
 @Service
 public class CreatePreferItemService implements CreatePreferItemUseCase {
 
@@ -21,4 +23,5 @@ public void createPreferItem(PreferItemCreateRequestDto requestDto) {
 
         preferItemRepository.save(requestDto.createPreferItem());
     }
+
 }

src/main/java/com/somemore/center/service/query/CenterQueryService.java

@@ -32,16 +32,16 @@ public CenterProfileResponseDto getCenterProfileByCenterId(UUID centerId) {
         return CenterProfileResponseDto.of(center, preferItems);
     }
 
-    private Center getCenterById(UUID centerId) {
-        return centerRepository.findCenterById(centerId)
-                .orElseThrow(() -> new BadRequestException(NOT_EXISTS_CENTER.getMessage()));
-    }
-
     @Override
     public void validateCenterExists(UUID id) {
         if (centerRepository.doesNotExistById(id)) {
             throw new BadRequestException(NOT_EXISTS_CENTER.getMessage());
         }
     }
 
+    private Center getCenterById(UUID centerId) {
+        return centerRepository.findCenterById(centerId)
+                .orElseThrow(() -> new BadRequestException(NOT_EXISTS_CENTER.getMessage()));
+    }
+
 }

src/main/java/com/somemore/center/service/query/PreferItemQueryService.java

@@ -5,11 +5,13 @@
 import com.somemore.center.usecase.query.PreferItemQueryUseCase;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
+import org.springframework.transaction.annotation.Transactional;
 
 import java.util.List;
 import java.util.UUID;
 
 @RequiredArgsConstructor
+@Transactional(readOnly = true)
 @Service
 public class PreferItemQueryService implements PreferItemQueryUseCase {