refactor(schedule): 중복 로직을 헬퍼 메서드로 정리, 테이블에 인덱스 추가, Date 필드 삭제(불필요)

Author: dpwls8984

src/main/java/back/kalender/domain/schedule/dto/response/MonthlyScheduleItem.java

@@ -29,9 +29,6 @@ public record MonthlyScheduleItem(
         @Schema(description = "공연 ID (공연 일정인 경우 포함, 없을 경우 null)", example = "501")
         Optional<Long> performanceId,
 
-        @Schema(description = "일정 날짜 (YYYY-MM-DD)", example = "2025-11-02")
-        LocalDate date,
-
         @Schema(description = "일정 장소 (선택 사항)", example = "고척 스카이돔")
         Optional<String> location
 ) {
@@ -43,7 +40,6 @@ public MonthlyScheduleItem(
                 ScheduleCategory scheduleCategory,
                 LocalDateTime scheduleTime,
                 Long performanceId,
-                LocalDate date,
                 String location
         ) {
                 this(
@@ -54,7 +50,6 @@ public MonthlyScheduleItem(
                         scheduleCategory,
                         scheduleTime,
                         Optional.ofNullable(performanceId),
-                        date,
                         Optional.ofNullable(location)
                 );
         }

src/main/java/back/kalender/domain/schedule/entity/Schedule.java

@@ -9,11 +9,14 @@
 import java.time.LocalDateTime;
 
 @Entity
-@Table(name = "schedules")
+@Table(
+        name = "schedules",
+        indexes = @Index(name = "idx_schedule_artist_time", columnList = "artistId, scheduleTime")
+)
 @Getter
 @Builder
 @NoArgsConstructor(access = AccessLevel.PROTECTED)
-@AllArgsConstructor
+@AllArgsConstructor(access = AccessLevel.PRIVATE)
 public class Schedule extends BaseEntityTmp {
 
     @Column(nullable = false)
@@ -34,7 +37,4 @@ public class Schedule extends BaseEntityTmp {
     private LocalDateTime scheduleTime;
 
     private String location;
-
-    private LocalDate date;
-    //TODO: scheduleTime, date 차이 명확히 알아두기, 실제 일정 날짜는 3일이고 티켓팅은 1일일 때 데이터 처리 어떻게 할지 고민하기
 }

src/main/java/back/kalender/domain/schedule/repository/ScheduleRepository.java

@@ -22,7 +22,6 @@ public interface ScheduleRepository extends JpaRepository<Schedule, Long> {
             s.scheduleCategory,
             s.scheduleTime,
             s.performanceId,
-            CAST(s.scheduleTime AS LocalDate),
             s.location
         )
         FROM Schedule s

src/main/java/back/kalender/domain/schedule/service/ScheduleServiceImpl.java

@@ -41,18 +41,14 @@ public MonthlySchedulesResponse getFollowingSchedules(Long userId, int year, int
             throw new ServiceException(ErrorCode.INVALID_INPUT_VALUE);
         }
 
-        List<ArtistFollowTmp> follows = artistFollowRepository.findAllByUserId(userId);
-        log.debug("[Schedule] [GetFollowing] 팔로우 아티스트 조회 완료 - count={}", follows.size());
+        List<Long> artistIds = getFollowedArtistIds(userId);
 
-        if (follows.isEmpty()) {
+        if (artistIds.isEmpty()) {
             log.info("[Schedule] [GetFollowing] 팔로우한 아티스트 없음, 빈 리스트 반환 - userId={}", userId);
             return new MonthlySchedulesResponse(Collections.emptyList());
         }
 
-        List<Long> artistIds = follows.stream()
-                .map(follow -> follow.getArtist().getId())
-                .toList();
-        log.debug("[Schedule] [GetFollowing] 대상 아티스트 ID 추출 - artistIds={}", artistIds);
+        log.debug("[Schedule] [GetFollowing] 대상 아티스트 ID 추출 - count={}, ids={}", artistIds.size(), artistIds);
 
         try {
             YearMonth yearMonth = YearMonth.of(year, month);
@@ -160,17 +156,13 @@ public DailySchedulesResponse getDailySchedules(Long userId, String date, Option
             log.debug("[Schedule] [GetDaily] 단일 아티스트 필터링 적용 - artistId={}", id);
 
         } else {
-            List<ArtistFollowTmp> follows = artistFollowRepository.findAllByUserId(userId);
+            targetArtistIds = getFollowedArtistIds(userId);
 
-            if (follows.isEmpty()) {
+            if (targetArtistIds.isEmpty()) {
                 log.info("[Schedule] [GetDaily] 팔로우한 아티스트 없음 - 빈 리스트 반환");
                 return new DailySchedulesResponse(Collections.emptyList());
             }
 
-            targetArtistIds = follows.stream()
-                    .map(follow -> follow.getArtist().getId())
-                    .toList();
-
             log.debug("[Schedule] [GetDaily] 전체 팔로우 아티스트 적용 - count={}", targetArtistIds.size());
         }
 
@@ -208,16 +200,13 @@ public UpcomingEventsResponse getUpcomingEvents(Long userId, Optional<Long> arti
             log.debug("[Schedule] [GetUpcoming] 단일 아티스트 필터링 적용 - artistId={}", id);
 
         } else {
-            List<ArtistFollowTmp> follows = artistFollowRepository.findAllByUserId(userId);
+            targetArtistIds = getFollowedArtistIds(userId);
 
-            if (follows.isEmpty()) {
+            if (targetArtistIds.isEmpty()) {
                 log.info("[Schedule] [GetUpcoming] 팔로우한 아티스트 없음 - 빈 리스트 반환");
                 return new UpcomingEventsResponse(Collections.emptyList());
             }
 
-            targetArtistIds = follows.stream()
-                    .map(follow -> follow.getArtist().getId())
-                    .toList();
             log.debug("[Schedule] [GetUpcoming] 전체 팔로우 아티스트 적용 - count={}", targetArtistIds.size());
         }
 
@@ -251,4 +240,16 @@ public UpcomingEventsResponse getUpcomingEvents(Long userId, Optional<Long> arti
 
         return new UpcomingEventsResponse(processedItems);
     }
+
+    private List<Long> getFollowedArtistIds(Long userId) {
+        List<ArtistFollowTmp> follows = artistFollowRepository.findAllByUserId(userId);
+
+        if (follows.isEmpty()) {
+            return Collections.emptyList();
+        }
+
+        return follows.stream()
+                .map(follow -> follow.getArtist().getId())
+                .toList();
+    }
 }

src/test/java/back/kalender/domain/schedule/service/ScheduleServiceImplTest.java

@@ -92,7 +92,6 @@ private MonthlyScheduleItem createDto(Long id, Long artistId, String artistName,
                 ScheduleCategory.CONCERT,
                 LocalDateTime.now(),
                 Optional.empty(),
-                LocalDate.now(),
                 Optional.empty()
         );
     }