Merge pull request #185 from Team-HMH/fix/#162-duplicate-daily-challenge

fix - 챌린지 생성 시 중복된 일별 챌린지를 생성하지 않도록 유효성 검사 로직 추가

src/main/java/sopt/org/hmh/domain/dailychallenge/domain/exception/DailyChallengeError.java

@@ -9,8 +9,8 @@ public enum DailyChallengeError implements ErrorBase {
 
     DAILY_CHALLENGE_NOT_FOUND(HttpStatus.NOT_FOUND, "일별 챌린지를 찾을 수 없습니다."),
     DAILY_CHALLENGE_PERIOD_INDEX_NOT_FOUND(HttpStatus.NOT_FOUND, "해당 인덱스의 일별 챌린지를 찾을 수 없습니다."),
-    DAILY_CHALLENGE_ALREADY_PROCESSED(HttpStatus.BAD_REQUEST, "이미 처리된 일별 챌린지입니다.")
-    ;
+    DAILY_CHALLENGE_ALREADY_PROCESSED(HttpStatus.BAD_REQUEST, "이미 처리된 일별 챌린지입니다."),
+    DAILY_CHALLENGE_ALREADY_EXISTS(HttpStatus.BAD_REQUEST, "이미 존재하는 일별 챌린지입니다.");
 
     private final HttpStatus status;
     private final String errorMessage;

src/main/java/sopt/org/hmh/domain/dailychallenge/repository/DailyChallengeRepository.java

@@ -13,4 +13,5 @@ public interface DailyChallengeRepository {
 
     List<DailyChallenge> findAllByChallengeIdOrderByChallengeDate(Long challengeId);
 
+    boolean existsByUserIdAndChallengeDateIn(Long userId, List<LocalDate> localDates);
 }

src/main/java/sopt/org/hmh/domain/dailychallenge/repository/DailyChallengeRepositoryImpl.java

@@ -1,17 +1,20 @@
 package sopt.org.hmh.domain.dailychallenge.repository;
 
+import com.querydsl.jpa.impl.JPAQueryFactory;
 import java.time.LocalDate;
 import java.util.List;
 import java.util.Optional;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Repository;
 import sopt.org.hmh.domain.dailychallenge.domain.DailyChallenge;
+import sopt.org.hmh.domain.dailychallenge.domain.QDailyChallenge;
 
 @Repository
 @RequiredArgsConstructor
 public class DailyChallengeRepositoryImpl implements DailyChallengeRepository {
 
     private final DailyChallengeJpaRepository dailyChallengeJpaRepository;
+    private final JPAQueryFactory queryFactory;
 
     @Override
     public void saveAll(List<DailyChallenge> dailyChallengeByChallengePeriod) {
@@ -27,4 +30,15 @@ public Optional<DailyChallenge> findByChallengeDateAndUserId(LocalDate challenge
     public List<DailyChallenge> findAllByChallengeIdOrderByChallengeDate(Long challengeId) {
         return dailyChallengeJpaRepository.findAllByChallengeIdOrderByChallengeDate(challengeId);
     }
+
+    @Override
+    public boolean existsByUserIdAndChallengeDateIn(Long userId, List<LocalDate> localDates) {
+        QDailyChallenge dailyChallenge = QDailyChallenge.dailyChallenge;
+        Integer fetchOne = queryFactory.selectOne()
+                .from(dailyChallenge)
+                .where(dailyChallenge.userId.eq(userId)
+                        .and(dailyChallenge.challengeDate.in(localDates)))
+                .fetchFirst();
+        return fetchOne != null;
+    }
 }

src/main/java/sopt/org/hmh/domain/dailychallenge/service/DailyChallengeService.java

@@ -59,9 +59,20 @@ private void handleAlreadyProcessedDailyChallenge(DailyChallenge dailyChallenge)
     }
 
     public void addDailyChallenge(Challenge challenge) {
+        validateDuplicateDailyChallenge(challenge);
         dailyChallengeRepository.saveAll(createDailyChallengeByChallengePeriod(challenge));
     }
 
+    private void validateDuplicateDailyChallenge(Challenge challenge) {
+        List<LocalDate> localDatesToCheck = IntStream.range(0, challenge.getPeriod())
+                .mapToObj(i -> challenge.getStartDate().plusDays(i))
+                .toList();
+
+        if (dailyChallengeRepository.existsByUserIdAndChallengeDateIn(challenge.getUserId(), localDatesToCheck)) {
+            throw new DailyChallengeException(DailyChallengeError.DAILY_CHALLENGE_ALREADY_EXISTS);
+        }
+    }
+
     private List<DailyChallenge> createDailyChallengeByChallengePeriod(Challenge challenge) {
         LocalDate startDate = challenge.getStartDate();
         Long userId = challenge.getUserId();