refactor(recruit-board): 모집글 조회시 QueryUseCase 의존하도록 리팩토링

Author: leebs0521

src/main/java/com/somemore/domains/recruitboard/service/command/DeleteRecruitBoardService.java

@@ -4,6 +4,7 @@
 import com.somemore.domains.recruitboard.repository.RecruitBoardRepository;
 import com.somemore.domains.recruitboard.service.validator.RecruitBoardValidator;
 import com.somemore.domains.recruitboard.usecase.command.DeleteRecruitBoardUseCase;
+import com.somemore.domains.recruitboard.usecase.query.RecruitBoardQueryUseCase;
 import com.somemore.global.exception.BadRequestException;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
@@ -18,15 +19,12 @@
 @Service
 public class DeleteRecruitBoardService implements DeleteRecruitBoardUseCase {
 
-    private final RecruitBoardRepository recruitBoardRepository;
+    private final RecruitBoardQueryUseCase recruitBoardQueryUseCase;
     private final RecruitBoardValidator recruitBoardValidator;
 
     @Override
-    public void deleteRecruitBoard(UUID centerId, Long recruitBoardId) {
-        RecruitBoard recruitBoard = recruitBoardRepository.findById(recruitBoardId).orElseThrow(
-                () -> new BadRequestException(NOT_EXISTS_RECRUIT_BOARD.getMessage())
-        );
-
+    public void deleteRecruitBoard(UUID centerId, Long id) {
+        RecruitBoard recruitBoard = recruitBoardQueryUseCase.getById(id);
         recruitBoardValidator.validateAuthor(recruitBoard, centerId);
 
         recruitBoard.markAsDeleted();

src/main/java/com/somemore/domains/recruitboard/service/command/UpdateRecruitBoardService.java

@@ -5,41 +5,37 @@
 import com.somemore.domains.recruitboard.domain.RecruitStatus;
 import com.somemore.domains.recruitboard.dto.request.RecruitBoardLocationUpdateRequestDto;
 import com.somemore.domains.recruitboard.dto.request.RecruitBoardUpdateRequestDto;
-import com.somemore.domains.recruitboard.repository.RecruitBoardRepository;
 import com.somemore.domains.recruitboard.service.validator.RecruitBoardValidator;
 import com.somemore.domains.recruitboard.usecase.command.UpdateRecruitBoardUseCase;
-import com.somemore.global.exception.BadRequestException;
+import com.somemore.domains.recruitboard.usecase.query.RecruitBoardQueryUseCase;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
 import java.time.LocalDateTime;
 import java.util.UUID;
 
-import static com.somemore.global.exception.ExceptionMessage.NOT_EXISTS_RECRUIT_BOARD;
-
 @RequiredArgsConstructor
 @Transactional
 @Service
 public class UpdateRecruitBoardService implements UpdateRecruitBoardUseCase {
 
-    private final RecruitBoardRepository recruitBoardRepository;
-    private final RecruitBoardValidator recruitBoardValidator;
+    private final RecruitBoardQueryUseCase recruitBoardQueryUseCase;
     private final UpdateLocationUseCase updateLocationUseCase;
+    private final RecruitBoardValidator recruitBoardValidator;
 
     @Override
-    public void updateRecruitBoard(RecruitBoardUpdateRequestDto dto, Long recruitBoardId, UUID centerId, String imgUrl) {
-        RecruitBoard recruitBoard = getRecruitBoard(recruitBoardId);
+    public void updateRecruitBoard(RecruitBoardUpdateRequestDto dto, Long id, UUID centerId, String imgUrl) {
+        RecruitBoard recruitBoard = recruitBoardQueryUseCase.getById(id);
         recruitBoardValidator.validateAuthor(recruitBoard, centerId);
         recruitBoardValidator.validateRecruitBoardTime(dto.volunteerStartDateTime(), dto.volunteerEndDateTime());
 
         recruitBoard.updateWith(dto, imgUrl);
     }
 
     @Override
-    public void updateRecruitBoardLocation(RecruitBoardLocationUpdateRequestDto requestDto, Long recruitBoardId,
-                                           UUID centerId) {
-        RecruitBoard recruitBoard = getRecruitBoard(recruitBoardId);
+    public void updateRecruitBoardLocation(RecruitBoardLocationUpdateRequestDto requestDto, Long id, UUID centerId) {
+        RecruitBoard recruitBoard = recruitBoardQueryUseCase.getById(id);
         recruitBoardValidator.validateAuthor(recruitBoard, centerId);
 
         updateLocationUseCase.updateLocation(requestDto.toLocationUpdateRequestDto(), recruitBoard.getLocationId());
@@ -48,18 +44,11 @@ public void updateRecruitBoardLocation(RecruitBoardLocationUpdateRequestDto requ
     }
 
     @Override
-    public void updateRecruitBoardStatus(RecruitStatus status, Long recruitBoardId, UUID centerId,
-                                         LocalDateTime currentDateTime) {
-        RecruitBoard recruitBoard = getRecruitBoard(recruitBoardId);
+    public void updateRecruitBoardStatus(RecruitStatus status, Long id, UUID centerId, LocalDateTime currentDateTime) {
+        RecruitBoard recruitBoard = recruitBoardQueryUseCase.getById(id);
         recruitBoardValidator.validateAuthor(recruitBoard, centerId);
 
         recruitBoard.changeRecruitStatus(status, currentDateTime);
     }
 
-    private RecruitBoard getRecruitBoard(Long recruitBoardId) {
-        return recruitBoardRepository.findById(recruitBoardId).orElseThrow(
-                () -> new BadRequestException(NOT_EXISTS_RECRUIT_BOARD.getMessage())
-        );
-    }
-
 }

src/main/java/com/somemore/domains/recruitboard/service/validator/RecruitBoardValidator.java

@@ -26,6 +26,6 @@ public void validateAuthor(RecruitBoard recruitBoard, UUID centerId) {
             return;
         }
 
-        throw new BadRequestException(UNAUTHORIZED_RECRUIT_BOARD.getMessage());
+        throw new BadRequestException(UNAUTHORIZED_RECRUIT_BOARD);
     }
 }

src/main/java/com/somemore/domains/recruitboard/usecase/command/DeleteRecruitBoardUseCase.java

@@ -4,5 +4,5 @@
 
 public interface DeleteRecruitBoardUseCase {
 
-    void deleteRecruitBoard(UUID centerId, Long recruitBoardId);
+    void deleteRecruitBoard(UUID centerId, Long id);
 }

src/main/java/com/somemore/domains/recruitboard/usecase/command/UpdateRecruitBoardUseCase.java

@@ -3,29 +3,14 @@
 import com.somemore.domains.recruitboard.domain.RecruitStatus;
 import com.somemore.domains.recruitboard.dto.request.RecruitBoardLocationUpdateRequestDto;
 import com.somemore.domains.recruitboard.dto.request.RecruitBoardUpdateRequestDto;
+
 import java.time.LocalDateTime;
 import java.util.UUID;
 
 public interface UpdateRecruitBoardUseCase {
+    void updateRecruitBoard(RecruitBoardUpdateRequestDto requestDto, Long id, UUID centerId, String imgUrl);
 
-    void updateRecruitBoard(
-        RecruitBoardUpdateRequestDto requestDto,
-        Long recruitBoardId,
-        UUID centerId,
-        String imgUrl
-    );
-
-    void updateRecruitBoardLocation(
-        RecruitBoardLocationUpdateRequestDto requestDto,
-        Long recruitBoardId,
-        UUID centerId
-    );
-
-    void updateRecruitBoardStatus(
-        RecruitStatus status,
-        Long recruitBoardId,
-        UUID centerId,
-        LocalDateTime currentDateTime
-    );
+    void updateRecruitBoardLocation(RecruitBoardLocationUpdateRequestDto requestDto, Long id, UUID centerId);
 
+    void updateRecruitBoardStatus(RecruitStatus status, Long id, UUID centerId, LocalDateTime currentDateTime);
 }