refactor(recruit-board): RecruitBoard 수정 검증 변경에 따른 리팩토링

Author: leebs0521

src/main/java/com/somemore/domains/recruitboard/controller/RecruitBoardCommandApiController.java

@@ -63,7 +63,8 @@ public ApiResponse<String> updateRecruitBoard(
             @RequestPart(value = "img_file", required = false) MultipartFile image
     ) {
         String imgUrl = imageUploadUseCase.uploadImage(new ImageUploadRequestDto(image));
-        updateRecruitBoardUseCase.updateRecruitBoard(requestDto, id, userId, imgUrl);
+        LocalDateTime now = LocalDateTime.now();
+        updateRecruitBoardUseCase.updateRecruitBoard(requestDto, id, userId, imgUrl, now);
 
         return ApiResponse.ok("봉사 활동 모집글 수정 성공");
     }
@@ -76,8 +77,8 @@ public ApiResponse<String> updateRecruitBoardLocation(
             @PathVariable Long id,
             @Valid @RequestBody RecruitBoardLocationUpdateRequestDto requestDto
     ) {
-
-        updateRecruitBoardUseCase.updateRecruitBoardLocation(requestDto, id, userId);
+        LocalDateTime now = LocalDateTime.now();
+        updateRecruitBoardUseCase.updateRecruitBoardLocation(requestDto, id, userId, now);
         return ApiResponse.ok("봉사 활동 모집글 위치 수정 성공");
     }
 

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

@@ -8,13 +8,12 @@
 import com.somemore.domains.recruitboard.service.validator.RecruitBoardValidator;
 import com.somemore.domains.recruitboard.usecase.RecruitBoardQueryUseCase;
 import com.somemore.domains.recruitboard.usecase.UpdateRecruitBoardUseCase;
+import java.time.LocalDateTime;
+import java.util.UUID;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
-import java.time.LocalDateTime;
-import java.util.UUID;
-
 @RequiredArgsConstructor
 @Transactional
 @Service
@@ -25,31 +24,46 @@ public class UpdateRecruitBoardService implements UpdateRecruitBoardUseCase {
     private final RecruitBoardValidator recruitBoardValidator;
 
     @Override
-    public void updateRecruitBoard(RecruitBoardUpdateRequestDto dto, Long id, UUID centerId, String imgUrl) {
-        RecruitBoard recruitBoard = recruitBoardQueryUseCase.getById(id);
-        recruitBoardValidator.validateWriter(recruitBoard, centerId);
+    public void updateRecruitBoard(RecruitBoardUpdateRequestDto dto, Long id, UUID centerId,
+            String imgUrl, LocalDateTime current) {
+        RecruitBoard recruitBoard = getRecruitBoard(id);
+        validateUpdatableAndWriter(recruitBoard, centerId, current);
+
         recruitBoardValidator.validateUpdateRecruitBoardTime(recruitBoard.getCreatedAt(),
                 dto.volunteerStartDateTime(), dto.volunteerEndDateTime());
 
         recruitBoard.updateWith(dto, imgUrl);
     }
 
     @Override
-    public void updateRecruitBoardLocation(RecruitBoardLocationUpdateRequestDto requestDto, Long id, UUID centerId) {
-        RecruitBoard recruitBoard = recruitBoardQueryUseCase.getById(id);
-        recruitBoardValidator.validateWriter(recruitBoard, centerId);
-
-        updateLocationUseCase.updateLocation(requestDto.toLocationUpdateRequestDto(), recruitBoard.getLocationId());
+    public void updateRecruitBoardLocation(RecruitBoardLocationUpdateRequestDto requestDto, Long id,
+            UUID centerId, LocalDateTime current) {
+        RecruitBoard recruitBoard = getRecruitBoard(id);
+        validateUpdatableAndWriter(recruitBoard, centerId, current);
 
+        updateLocationUseCase.updateLocation(requestDto.toLocationUpdateRequestDto(),
+                recruitBoard.getLocationId());
         recruitBoard.updateWith(requestDto.region());
     }
 
     @Override
-    public void updateRecruitBoardStatus(RecruitStatus status, Long id, UUID centerId, LocalDateTime currentDateTime) {
-        RecruitBoard recruitBoard = recruitBoardQueryUseCase.getById(id);
-        recruitBoardValidator.validateWriter(recruitBoard, centerId);
+    public void updateRecruitBoardStatus(RecruitStatus status, Long id, UUID centerId,
+            LocalDateTime current) {
+        RecruitBoard recruitBoard = getRecruitBoard(id);
+        validateUpdatableAndWriter(recruitBoard, centerId, current);
+        recruitBoardValidator.validateRecruitStatus(status);
 
         recruitBoard.updateRecruitStatus(status);
     }
 
+    private void validateUpdatableAndWriter(RecruitBoard recruitBoard, UUID centerId,
+            LocalDateTime current) {
+        recruitBoardValidator.validateUpdatable(recruitBoard, current);
+        recruitBoardValidator.validateWriter(recruitBoard, centerId);
+    }
+
+    private RecruitBoard getRecruitBoard(Long id) {
+        return recruitBoardQueryUseCase.getById(id);
+    }
 }
+

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

@@ -3,14 +3,18 @@
 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 updateRecruitBoardLocation(RecruitBoardLocationUpdateRequestDto requestDto, Long id, UUID centerId);
+    void updateRecruitBoard(RecruitBoardUpdateRequestDto dto, Long id, UUID centerId, String imgUrl,
+            LocalDateTime current);
+
+
+    void updateRecruitBoardLocation(RecruitBoardLocationUpdateRequestDto requestDto, Long id,
+            UUID centerId, LocalDateTime current);
 
-    void updateRecruitBoardStatus(RecruitStatus status, Long id, UUID centerId, LocalDateTime currentDateTime);
+    void updateRecruitBoardStatus(RecruitStatus status, Long id, UUID centerId,
+            LocalDateTime currentDateTime);
 }