Skip to content

Commit ceac93e

Browse files
committed
refactor: 비즈니스 로직 흐름에 맞게 순서 변경
1 parent 045bd50 commit ceac93e

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

src/main/java/com/example/solidconnection/mentor/service/MentoringCommandService.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,16 +59,17 @@ public MentoringConfirmResponse confirmMentoring(long siteUserId, long mentoring
5959
validateUnauthorizedMentoring(siteUserId, mentor);
6060
validateAlreadyConfirmed(mentoring);
6161

62-
if (mentoringConfirmRequest.status() == VerifyStatus.APPROVED) {
63-
mentor.increaseMenteeCount();
64-
}
65-
else if (mentoringConfirmRequest.status() == VerifyStatus.REJECTED
62+
if (mentoringConfirmRequest.status() == VerifyStatus.REJECTED
6663
&& (mentoringConfirmRequest.rejectedReason() == null || mentoringConfirmRequest.rejectedReason().isBlank())) {
6764
throw new CustomException(REJECTED_REASON_REQUIRED);
6865
}
6966

7067
mentoring.confirm(mentoringConfirmRequest.status(), mentoringConfirmRequest.rejectedReason());
7168

69+
if (mentoringConfirmRequest.status() == VerifyStatus.APPROVED) {
70+
mentor.increaseMenteeCount();
71+
}
72+
7273
return MentoringConfirmResponse.from(mentoring);
7374
}
7475

0 commit comments

Comments
 (0)