refactor: 중복 유효성 검사 제거 및 메소드 추출

related to: #18

src/main/java/com/_119/wepro/review/presentation/ReviewController.java

@@ -47,7 +47,7 @@ public ResponseEntity<Void> requestReview(@RequestBody @Valid ReviewAskRequest r
   @Operation(summary = "리뷰 임시저장 API")
   @PostMapping("/draft/{reviewFormId}")
   public ResponseEntity<Void> draftReview(@PathVariable(name = "reviewFormId") Long reviewFormId,
-      @RequestBody @Valid ReviewRequest.ReviewSaveRequest request) {
+      @RequestBody @Valid ReviewSaveRequest request) {
     Long memberId = securityUtil.getCurrentMemberId();
     reviewService.draft(memberId, reviewFormId, request);
     return ResponseEntity.ok().build();

src/main/java/com/_119/wepro/review/service/ReviewService.java

@@ -61,8 +61,8 @@ public void requestReview(ReviewAskRequest request, Long memberId) {
 
     List<Long> memberIdList = request.getMemberIdList();
     memberIdList.forEach(reviewerId -> {
-      findMemberById(reviewerId);
-      alarmService.createAlarm(member, reviewerId, AlarmType.REVIEW_REQUEST, request.getReviewFormId());
+      alarmService.createAlarm(member, reviewerId, AlarmType.REVIEW_REQUEST,
+          request.getReviewFormId());
     });
   }
 
@@ -96,19 +96,26 @@ public void submitReview(Long memberId, Long reviewFormId, ReviewSaveRequest req
     reviewRecordRepository.save(reviewRecord);
   }
 
-  private ReviewRecord getOrCreateReviewRecord(Member writer, ReviewForm reviewForm, ReviewSaveRequest request) {
+  private ReviewRecord getOrCreateReviewRecord(Member writer, ReviewForm reviewForm,
+      ReviewSaveRequest request) {
 
     return reviewRecordRepository.findByReviewForm(reviewForm)
-        .map(savedRecord -> {
-          if (!savedRecord.getIsDraft()) {
-            throw new RestApiException(ReviewErrorCode.ALREADY_SUBMITTED);
-          }
-          savedRecord.update(request);
-          return savedRecord;
-        })
+        .map(savedRecord -> updateIfDraft(savedRecord, request))
         .orElseGet(() -> ReviewRecord.of(writer, reviewForm, request));
   }
 
+  private ReviewRecord updateIfDraft(ReviewRecord savedRecord, ReviewSaveRequest request) {
+    checkIfDraft(savedRecord);
+    savedRecord.update(request);
+    return savedRecord;
+  }
+
+  private void checkIfDraft(ReviewRecord savedRecord) {
+    if (!savedRecord.getIsDraft()) {
+      throw new RestApiException(ReviewErrorCode.ALREADY_SUBMITTED);
+    }
+  }
+
   private Member findMemberById(Long memberId) {
     return memberRepository.findById(memberId)
         .orElseThrow(() -> new RestApiException(UserErrorCode.USER_NOT_FOUND));