Refactor/diary 코드 확인

src/main/java/com/example/cp_main_be/domain/admin/service/AdminService.java

@@ -17,9 +17,9 @@
 import com.example.cp_main_be.domain.reports.domain.Reports;
 import com.example.cp_main_be.domain.reports.domain.repository.ReportRepository;
 import com.example.cp_main_be.domain.reports.enums.ReportStatus;
-import com.example.cp_main_be.global.exception.QuizNotFoundException;
+import com.example.cp_main_be.global.common.CustomApiException;
+import com.example.cp_main_be.global.common.ErrorCode;
 import com.example.cp_main_be.global.infra.S3Uploader;
-import jakarta.persistence.*;
 import java.util.List;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
@@ -97,7 +97,7 @@ public boolean deleteQuiz(Long quizId) {
     Quiz quiz =
         quizRepository
             .findById(quizId)
-            .orElseThrow(() -> new QuizNotFoundException("퀴즈가 존재하지 않습니다."));
+            .orElseThrow(() -> new CustomApiException(ErrorCode.QUIZ_NOT_FOUND));
     quizRepository.delete(quiz);
     return true;
   }
@@ -125,7 +125,7 @@ public DailyMissionMaster updateDailyMissionMasters(
     DailyMissionMaster dailyMissionMaster =
         dailyMissionMastersRepository
             .findById(id)
-            .orElseThrow(() -> new IllegalArgumentException("해당 ID의 미션을 찾을 수 없습니다."));
+            .orElseThrow(() -> new CustomApiException(ErrorCode.MISSION_NOT_FOUND));
 
     // null 인 컬럼들은 수정 안한다.
     dailyMissionMaster.update(requestDTO);
@@ -157,10 +157,10 @@ public QuizOptions createQuizOption(
     Quiz quiz =
         quizRepository
             .findById(quizId)
-            .orElseThrow(() -> new RuntimeException("해당 ID를 가진 퀴즈가 존재하지 않습니다."));
+            .orElseThrow(() -> new CustomApiException(ErrorCode.QUIZ_NOT_FOUND));
 
     if (quiz.getDailyMissionMaster().getMissionType() != MissionType.QUIZ) {
-      throw new IllegalArgumentException("퀴즈 타입의 미션에만 선지를 추가할 수 있습니다.");
+      throw new CustomApiException(ErrorCode.INVALID_MISSION_TYPE_FOR_QUIZ_OPTION);
     }
 
     return QuizOptions.builder() // QuizOptions 퀴즈의 선지
@@ -180,7 +180,7 @@ public Reports updateReportStatus(Long reportId, ReportStatus reportStatus) {
     Reports report =
         reportRepository
             .findById(reportId)
-            .orElseThrow(() -> new RuntimeException("신고를 찾을 수 없습니다."));
+            .orElseThrow(() -> new CustomApiException(ErrorCode.REPORT_NOT_FOUND));
     report.setStatus(reportStatus);
     return report;
   }

src/main/java/com/example/cp_main_be/domain/avatar/avatar/service/AvatarService.java

@@ -25,12 +25,11 @@
 @Transactional
 public class AvatarService {
 
+  private static final Long AI_AVATAR_MASTER_ID = 9999L;
   private final AvatarRepository avatarRepository;
   private final UserRepository userRepository;
   private final AvatarMasterRepository avatarMasterRepository; // [추가] AvatarMaster 조회 위해 주입
   private final NotificationService notificationService;
-
-  private static final Long AI_AVATAR_MASTER_ID = 9999L;
   private final WishTreeRepository wishTreeRepository;
   private final GardenRepository gardenRepository;
 
@@ -39,21 +38,27 @@ public void createAvatar(Long userId, String nickname, String imageUrl, Long mas
     User user =
         userRepository
             .findById(userId)
-            .orElseThrow(() -> new CustomApiException(ErrorCode.NOT_FOUND));
+            .orElseThrow(() -> new CustomApiException(ErrorCode.USER_NOT_FOUND));
 
     AvatarMaster master;
     if (masterId != null) {
       // 1. 기존 목록에서 선택한 경우: 전달받은 masterId로 AvatarMaster를 찾습니다.
+      // TODO: 'masterId + 2'와 같은 매직 넘버 로직은 위험합니다.
+      // 프론트엔드에서 전달하는 ID와 DB의 ID가 일치하도록 데이터 정합성을 맞추거나,
+      // 이 로직에 대한 명확한 주석과 문서화가 필요합니다.
       master =
           avatarMasterRepository
               .findById(masterId + 2)
-              .orElseThrow(() -> new CustomApiException(ErrorCode.NOT_FOUND)); // 예외 유형 구체화
+              .orElseThrow(() -> new CustomApiException(ErrorCode.AVATAR_MASTER_NOT_FOUND));
     } else {
       // 2. AI로 생성한 경우: 약속된 AI_AVATAR_MASTER_ID로 AvatarMaster를 찾습니다.
       master =
           avatarMasterRepository
               .findById(AI_AVATAR_MASTER_ID)
-              .orElseThrow(() -> new CustomApiException(ErrorCode.NOT_FOUND));
+              .orElseThrow(
+                  () ->
+                      new CustomApiException(
+                          ErrorCode.AVATAR_MASTER_NOT_FOUND, "AI 아바타 원본을 찾을 수 없습니다."));
     }
 
     Avatar newAvatar =
@@ -69,7 +74,8 @@ public void createAvatar(Long userId, String nickname, String imageUrl, Long mas
     WishTree wishTree =
         wishTreeRepository
             .findByUserId(userId)
-            .orElseThrow(() -> new CustomApiException(ErrorCode.NOT_FOUND));
+            .orElseThrow(
+                () -> new CustomApiException(ErrorCode.NOT_FOUND, "사용자의 소원나무를 찾을 수 없습니다."));
 
     WishTreeStage stage = wishTree.getStage();
     int maxGardens = stage.getMaxGardens();
@@ -80,22 +86,23 @@ public void createAvatar(Long userId, String nickname, String imageUrl, Long mas
           Garden.builder().user(user).slotNumber(userGardens.size() + 1).avatar(newAvatar).build();
       gardenRepository.save(newGarden);
     } else {
-      throw new CustomApiException(ErrorCode.MAX_GARDENS_REACHED);
+      // 이미 존재하는 'GARDEN_SLOT_MAXED_OUT' 에러 코드를 재사용하여 일관성을 유지합니다.
+      throw new CustomApiException(ErrorCode.GARDEN_SLOT_MAXED_OUT);
     }
   }
 
   @Transactional(readOnly = true)
   public Avatar findAvatarById(Long avatarId) {
     return avatarRepository
         .findById(avatarId)
-        .orElseThrow(() -> new CustomApiException(ErrorCode.NOT_FOUND));
+        .orElseThrow(() -> new CustomApiException(ErrorCode.NOT_FOUND, "해당 아바타를 찾을 수 없습니다."));
   }
 
   public void givePollen(Long senderId, Long avatarId) {
     User sender =
         userRepository
             .findById(senderId)
-            .orElseThrow(() -> new CustomApiException(ErrorCode.NOT_FOUND));
+            .orElseThrow(() -> new CustomApiException(ErrorCode.USER_NOT_FOUND));
 
     Avatar avatar = findAvatarById(avatarId);
     User receiver = avatar.getUser();

src/main/java/com/example/cp_main_be/domain/garden/garden/service/GardenService.java

@@ -18,16 +18,19 @@
 import java.util.List;
 import java.util.stream.Collectors;
 import lombok.RequiredArgsConstructor;
+import lombok.extern.slf4j.Slf4j;
 import org.springframework.context.ApplicationEventPublisher;
+import org.springframework.scheduling.annotation.Scheduled;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
 @Service
 @RequiredArgsConstructor
 @Transactional(readOnly = true)
+@Slf4j
 public class GardenService {
 
-  private static final int MAX_GARDEN_COUNT = 4;
+  private static final int MAX_GARDEN_COUNT = 3;
   private static final int WATERING_POINTS = 2;
   private static final int SUNLIGHT_POINTS = 3;
   private static final int MAX_FRIEND_WATERING_PER_DAY = 3;
@@ -50,57 +53,81 @@ public GardenResponse findGardenById(Long gardenId) {
 
   @Transactional
   public void waterGarden(Long actorId, Long gardenId) {
+    // N+1 문제를 방지하기 위해 Garden과 User를 함께 조회하는 것을 권장합니다.
+    // 예: gardenRepository.findByIdWithUser(gardenId)
     Garden garden =
         gardenRepository
             .findById(gardenId)
-            .orElseThrow(() -> new IllegalArgumentException("해당 텃밭을 찾을 수 없습니다."));
+            .orElseThrow(() -> new CustomApiException(ErrorCode.GARDEN_NOT_FOUND));
 
-    User owner = garden.getUser(); // 정원 주인
+    User owner = garden.getUser();
 
-    // Case 1: 자신의 정원에 물을 주는 경우
     if (owner.getId().equals(actorId)) {
-      // 8시간 쿨타임 체크
-      if (garden.getLastWateredByOwnerAt() != null
-          && garden.getLastWateredByOwnerAt().plusHours(8).isAfter(LocalDateTime.now())) {
-        throw new IllegalStateException("아직 물을 줄 수 없습니다. 8시간이 지나야 가능합니다.");
-      }
-
-      garden.increaseWaterCount();
-      userService.addExperience(actorId, WATERING_POINTS);
-      garden.recordOwnerWateringTime(); // 주인이 물 준 시간 기록
+      waterOwnGarden(actorId, garden);
+    } else {
+      waterFriendGarden(actorId, garden);
     }
-    // Case 2: 남의 정원에 물을 주는 경우
-    else {
-      User actor =
-          userRepository
-              .findById(actorId)
-              .orElseThrow(() -> new IllegalArgumentException("물을 주는 사용자를 찾을 수 없습니다."));
-
-      LocalDateTime startOfWateringDay = getStartOfCurrentWateringDay();
-
-      // 1. 하루에 3회 제한 체크
-      int todayWateringCount =
-          friendWateringLogRepository.countByWaterGiverAndWateredAtAfter(actor, startOfWateringDay);
-      if (todayWateringCount >= MAX_FRIEND_WATERING_PER_DAY) {
-        throw new IllegalStateException("오늘은 다른 사람의 정원에 더 이상 물을 줄 수 없습니다. (일일 3회 제한)");
-      }
-
-      // 2. 같은 정원에 하루 한 번 제한 체크
-      boolean alreadyWatered =
-          friendWateringLogRepository.existsByWaterGiverAndWateredGardenAndWateredAtAfter(
-              actor, garden, startOfWateringDay);
-      if (alreadyWatered) {
-        throw new IllegalStateException("이 정원에는 오늘 이미 물을 주었습니다.");
-      }
-
-      // 남한테 주는 경우에는 준 사람이 물 경험치를 받고 정원의 waterCount가 증가한다.
-      userService.addExperience(actorId, WATERING_POINTS);
-      garden.increaseWaterCount();
-
-      // 물주기 활동 기록
-      FriendWateringLog log =
-          FriendWateringLog.builder().waterGiver(actor).wateredGarden(garden).build();
-      friendWateringLogRepository.save(log);
+  }
+
+  /** 자신의 정원에 물을 주는 로직을 처리합니다. */
+  private void waterOwnGarden(Long ownerId, Garden garden) {
+    // 8시간 쿨타임 체크
+    if (garden.getLastWateredByOwnerAt() != null
+        && garden.getLastWateredByOwnerAt().plusHours(8).isAfter(LocalDateTime.now())) {
+      throw new CustomApiException(ErrorCode.WATERING_COOL_DOWN);
+    }
+
+    garden.increaseWaterCount();
+    userService.addExperience(ownerId, WATERING_POINTS);
+    garden.recordOwnerWateringTime(); // 주인이 물 준 시간 기록
+  }
+
+  /** 친구의 정원에 물을 주는 로직을 처리합니다. */
+  private void waterFriendGarden(Long actorId, Garden garden) {
+    User actor =
+        userRepository
+            .findById(actorId)
+            .orElseThrow(() -> new CustomApiException(ErrorCode.USER_NOT_FOUND));
+
+    // 낮 12시 이후 -> 그날의 12시 날짜 반환, 이전 -> 전날의 12시 날짜 반환
+    LocalDateTime startOfWateringDay = getStartOfCurrentWateringDay();
+
+    // 1. 하루에 3회 제한 체크
+    checkFriendWateringLimit(actor, startOfWateringDay);
+
+    // 2. 같은 정원에 하루 한 번 제한 체크
+    checkAlreadyWateredToday(actor, garden, startOfWateringDay);
+
+    // 남한테 주는 경우에는 준 사람이 물 경험치를 받고 정원의 waterCount가 증가한다.
+    userService.addExperience(actorId, WATERING_POINTS);
+    garden.increaseWaterCount();
+
+    // 물주기 활동 기록
+    FriendWateringLog log =
+        FriendWateringLog.builder()
+            .waterGiver(actor)
+            .wateredGarden(garden) // wateredAt은 @CreatedDate가 자동으로 설정합니다.
+            .build();
+    friendWateringLogRepository.save(log);
+  }
+
+  // 물주기 남은 횟수 확인
+  private void checkFriendWateringLimit(User actor, LocalDateTime startOfWateringDay) {
+    int todayWateringCount =
+        friendWateringLogRepository.countByWaterGiverAndWateredAtAfter(actor, startOfWateringDay);
+    if (todayWateringCount >= MAX_FRIEND_WATERING_PER_DAY) {
+      throw new CustomApiException(ErrorCode.FRIEND_WATERING_LIMIT_EXCEEDED);
+    }
+  }
+
+  // 당일에 해당 정원에 이미 물을 주었는지 확인
+  private void checkAlreadyWateredToday(
+      User actor, Garden garden, LocalDateTime startOfWateringDay) {
+    boolean alreadyWatered =
+        friendWateringLogRepository.existsByWaterGiverAndWateredGardenAndWateredAtAfter(
+            actor, garden, startOfWateringDay);
+    if (alreadyWatered) {
+      throw new CustomApiException(ErrorCode.ALREADY_WATERED_GARDEN);
     }
   }
 
@@ -141,7 +168,7 @@ public void unlockNewGardenSlot(Long userId) {
     if (currentGardens >= MAX_GARDEN_COUNT) {
       // 이미 최대치이므로 조용히 종료하거나 예외를 던질 수 있습니다.
       // 여기서는 추가 생성을 막고 그냥 리턴합니다.
-      return;
+      throw new CustomApiException(ErrorCode.GARDEN_SLOT_MAXED_OUT);
     }
 
     // [기존 레벨 체크 로직 삭제!]
@@ -211,4 +238,16 @@ private LocalDateTime getStartOfCurrentSunlightDay() {
       return todaySixAM;
     }
   }
+
+  /** 오래된 친구 물주기 로그를 주기적으로 삭제하는 스케줄링 작업입니다. cron = "0 0 4 * * *" : 매일 새벽 4시에 실행됩니다. */
+  @Scheduled(cron = "0 0 4 * * *")
+  @Transactional // 쓰기 작업이므로 클래스 레벨의 readOnly 설정을 오버라이드합니다.
+  public void cleanupOldWateringLogs() {
+    // 7일 이상된 기록을 삭제하도록 설정. 이 값은 application.yml에서 관리하는 것이 더 좋습니다.
+    final int RETENTION_DAYS = 7;
+    LocalDateTime cutoffDate = LocalDateTime.now().minusDays(RETENTION_DAYS);
+    log.info("Starting cleanup of friend watering logs older than {} days...", RETENTION_DAYS);
+    int deletedCount = friendWateringLogRepository.deleteByWateredAtBefore(cutoffDate);
+    log.info("Finished cleanup. Deleted {} old friend watering logs.", deletedCount);
+  }
 }

src/main/java/com/example/cp_main_be/domain/garden/wateringlog/domain/FriendWateringLog.java

@@ -4,15 +4,14 @@
 import com.example.cp_main_be.domain.member.user.domain.User;
 import jakarta.persistence.*;
 import java.time.LocalDateTime;
-import lombok.AccessLevel;
-import lombok.Builder;
-import lombok.Getter;
-import lombok.NoArgsConstructor;
+import lombok.*;
 import org.springframework.data.annotation.CreatedDate;
 import org.springframework.data.jpa.domain.support.AuditingEntityListener;
 
 @Entity
 @Getter
+@Builder
+@AllArgsConstructor(access = AccessLevel.PROTECTED)
 @NoArgsConstructor(access = AccessLevel.PROTECTED)
 @EntityListeners(AuditingEntityListener.class)
 @Table(name = "friend_watering_log")

src/main/java/com/example/cp_main_be/domain/garden/wateringlog/domain/repository/FriendWateringLogRepository.java

@@ -5,11 +5,23 @@
 import com.example.cp_main_be.domain.member.user.domain.User;
 import java.time.LocalDateTime;
 import org.springframework.data.jpa.repository.JpaRepository;
+import org.springframework.data.jpa.repository.Modifying;
+import org.springframework.data.jpa.repository.Query;
+import org.springframework.data.repository.query.Param;
 
 public interface FriendWateringLogRepository extends JpaRepository<FriendWateringLog, Long> {
 
   int countByWaterGiverAndWateredAtAfter(User waterGiver, LocalDateTime startOfDay);
 
   boolean existsByWaterGiverAndWateredGardenAndWateredAtAfter(
       User waterGiver, Garden wateredGarden, LocalDateTime startOfDay);
+
+  /**
+   * 지정된 날짜 이전의 모든 물주기 로그를 삭제합니다. (벌크 삭제)
+   *
+   * @return 삭제된 레코드의 수
+   */
+  @Modifying
+  @Query("DELETE FROM FriendWateringLog fwl WHERE fwl.wateredAt < :cutoffDate")
+  int deleteByWateredAtBefore(@Param("cutoffDate") LocalDateTime cutoffDate);
 }

src/main/java/com/example/cp_main_be/domain/mission/diary/domain/Diary.java

@@ -97,4 +97,8 @@ public void addComment(Comment comment) {
     this.comments.add(comment);
     comment.setDiary(this); // Comment 엔티티에 setDiary 메서드가 있다고 가정
   }
+
+  public void setDiaryImage(DiaryImage diaryImage) {
+    this.diaryImage = diaryImage;
+  }
 }

src/main/java/com/example/cp_main_be/domain/mission/diary/dto/request/CreateDiaryRequest.java

@@ -25,4 +25,6 @@ public class CreateDiaryRequest {
   // 공개 여부는 선택사항으로, 값을 보내지 않으면 엔티티의 기본값(true)을 따릅니다.
   @NotNull(message = "공개 여부는 필수값입니다. (true/false)")
   private Boolean isPublic;
+
+  private Long imageId; // 있으면 받고 없으면 안받음
 }

src/main/java/com/example/cp_main_be/domain/mission/diary/dto/request/UpdateDiaryRequest.java

@@ -23,4 +23,6 @@ public class UpdateDiaryRequest {
 
   @NotNull(message = "공개 여부는 필수값입니다. (true/false)")
   private Boolean isPublic;
+
+  private Long imageId;
 }

src/main/java/com/example/cp_main_be/domain/mission/diary/presentation/DiaryController.java

@@ -29,8 +29,9 @@ public class DiaryController {
   @PostMapping
   public ResponseEntity<ApiResponse<DiaryResponse>> createDiary(
       @AuthenticationPrincipal User user, @RequestBody @Valid CreateDiaryRequest request) {
-    Diary createdDiary = diaryService.createDiary(user, request);
-    return ResponseEntity.ok(ApiResponse.success(DiaryResponse.from(createdDiary)));
+    Long diaryId = diaryService.createDiary(user, request);
+    Diary diary = diaryService.findDiaryById(diaryId);
+    return ResponseEntity.ok(ApiResponse.success(DiaryResponse.from(diary)));
   }
 
   @Operation(summary = "내 일기 목록 조회", description = "내 일기 목록을 조회합니다")