Merge pull request #296 from prgrms-web-devcourse-final-project/refactor/EA3-209-recuritment-post-edit

[EA3-209] Refactor: 모집글 생성 수정

Author: hyeunS-P

src/main/java/grep/neogulcoder/domain/recruitment/RecruitmentErrorCode.java

@@ -14,7 +14,8 @@ public enum RecruitmentErrorCode implements ErrorCode {
     NOT_FOUND(HttpStatus.NOT_FOUND, HttpStatus.NOT_FOUND.name(), "모집글을 찾지 못했습니다."),
     NOT_FOUND_STUDY_MEMBER(HttpStatus.NOT_FOUND, HttpStatus.NOT_FOUND.name(), "스터디에 참여하고 있지 않은 회원 입니다."),
     NOT_FOUND_COMMENT(HttpStatus.NOT_FOUND, HttpStatus.NOT_FOUND.name(), "댓글을 찾지 못했습니다"),
-    NOT_OWNER(HttpStatus.BAD_REQUEST, HttpStatus.BAD_REQUEST.name(), "모집글을 등록한 당사자가 아닙니다.");
+    NOT_OWNER(HttpStatus.BAD_REQUEST, HttpStatus.BAD_REQUEST.name(), "모집글을 등록한 당사자가 아닙니다."),
+    END_DATE_ERROR(HttpStatus.BAD_REQUEST, HttpStatus.BAD_REQUEST.name(), "모집글은 스터디가 시작되고 종료될 때까지 작성 가능 합니다.");
 
     private static final String BASIC_MESSAGE = "RECRUITMENT";
     private final HttpStatus status;

src/main/java/grep/neogulcoder/domain/recruitment/post/controller/RecruitmentPostSaveController.java

@@ -1,8 +1,8 @@
 package grep.neogulcoder.domain.recruitment.post.controller;
 
 import grep.neogulcoder.domain.recruitment.post.controller.dto.request.save.RecruitmentPostCreateRequest;
-import grep.neogulcoder.domain.recruitment.post.controller.dto.response.save.JoinedStudyLoadInfo;
 import grep.neogulcoder.domain.recruitment.post.controller.dto.response.save.JoinedStudiesInfo;
+import grep.neogulcoder.domain.recruitment.post.controller.dto.response.save.JoinedStudyLoadInfo;
 import grep.neogulcoder.domain.recruitment.post.service.RecruitmentPostSaveService;
 import grep.neogulcoder.global.auth.Principal;
 import grep.neogulcoder.global.response.ApiResponse;
@@ -21,7 +21,7 @@ public class RecruitmentPostSaveController implements RecruitmentPostSaveSpecifi
 
     @PostMapping
     public ResponseEntity<ApiResponse<Long>> save(@Valid @RequestBody RecruitmentPostCreateRequest request,
-                                                 @AuthenticationPrincipal Principal userDetails) {
+                                                  @AuthenticationPrincipal Principal userDetails) {
         long postId = recruitmentPostService.create(request.toServiceRequest(), userDetails.getUserId());
         return ResponseEntity.ok(ApiResponse.success(postId));
     }
@@ -34,7 +34,7 @@ public ResponseEntity<ApiResponse<JoinedStudiesInfo>> getJoinedStudyInfo(@Authen
 
     @GetMapping("/studies/{study-id}")
     public ResponseEntity<ApiResponse<JoinedStudyLoadInfo>> getJoinedStudyLoadInfo(@PathVariable("study-id") long studyId,
-                                                                   @AuthenticationPrincipal Principal userDetails) {
+                                                                                   @AuthenticationPrincipal Principal userDetails) {
         JoinedStudyLoadInfo response = recruitmentPostService.getJoinedStudyLoadInfo(studyId, userDetails.getUserId());
         return ResponseEntity.ok(ApiResponse.success(response));
     }

src/main/java/grep/neogulcoder/domain/recruitment/post/controller/dto/request/save/RecruitmentPostCreateRequest.java

@@ -2,7 +2,6 @@
 
 import grep.neogulcoder.domain.recruitment.post.service.request.RecruitmentPostCreateServiceRequest;
 import io.swagger.v3.oas.annotations.media.Schema;
-import jakarta.validation.constraints.FutureOrPresent;
 import jakarta.validation.constraints.NotBlank;
 import jakarta.validation.constraints.NotNull;
 import jakarta.validation.constraints.Positive;
@@ -31,7 +30,7 @@ public class RecruitmentPostCreateRequest {
     private int recruitmentCount;
 
     @Schema(example = "2025-07-10", description = "모집 마감일")
-    @FutureOrPresent(message = "모집 마감일은 현재 날짜 이전으로 설정이 불가능 합니다.")
+    @NotNull
     private LocalDateTime expiredDate;
 
     private RecruitmentPostCreateRequest() {

src/main/java/grep/neogulcoder/domain/recruitment/post/service/RecruitmentPostSaveService.java

@@ -13,8 +13,10 @@
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
+import java.time.LocalDateTime;
 import java.util.List;
 
+import static grep.neogulcoder.domain.recruitment.RecruitmentErrorCode.*;
 import static grep.neogulcoder.domain.recruitment.RecruitmentErrorCode.NOT_FOUND_STUDY_MEMBER;
 import static grep.neogulcoder.domain.recruitment.RecruitmentErrorCode.NOT_STUDY_LEADER;
 
@@ -28,12 +30,16 @@ public class RecruitmentPostSaveService {
 
     @Transactional
     public long create(RecruitmentPostCreateServiceRequest request, long userId) {
-        StudyMember studyMember = studyMemberRepository.findByStudyIdAndUserId(request.getStudyId(), userId);
+        StudyMember studyMember = studyMemberRepository.findMemberWithStudyBy(request.getStudyId(), userId);
 
         if (studyMember.hasNotRoleLeader()) {
             throw new BusinessException(NOT_STUDY_LEADER);
         }
 
+        if(studyMember.getStudy().hasEndDateBefore(request.getExpiredDate())){
+            throw new BusinessException(END_DATE_ERROR);
+        }
+
         return recruitmentPostRepository.save(request.toEntity(userId)).getId();
     }
 

src/main/java/grep/neogulcoder/domain/study/Study.java

@@ -125,6 +125,10 @@ public boolean isReviewableAt(LocalDateTime currentDateTime) {
                 (currentDateTime.isEqual(reviewableDateTime) || currentDateTime.isBefore(reviewableDateTime));
     }
 
+    public boolean hasEndDateBefore(LocalDateTime dateTime) {
+        return this.endDate.isBefore(dateTime);
+    }
+
     public static boolean isOverJoinLimit(int joinedStudyCount) {
         return joinedStudyCount >= MAX_JOINED_STUDY_COUNT;
     }

src/main/java/grep/neogulcoder/domain/study/repository/StudyMemberQueryRepository.java

@@ -56,6 +56,18 @@ public StudyMember findByStudyIdAndUserId(long studyId, long userId) {
                 .fetchOne();
     }
 
+    public StudyMember findMemberWithStudyBy(long studyId, long userId) {
+        return queryFactory.select(studyMember)
+                .from(studyMember)
+                .join(studyMember.study, study).fetchJoin()
+                .where(
+                        studyMember.study.id.eq(studyId),
+                        studyMember.userId.eq(userId),
+                        studyMember.activated.isTrue()
+                )
+                .fetchOne();
+    }
+
     public List<ExtendParticipationResponse> findExtendParticipation(Long studyId) {
         return queryFactory
             .select(Projections.constructor(