refactor: 일정 전체조회시 포함된 장소 개수 포함하여 응답

Author: TaetaetaE01

src/main/java/com/cliptripbe/feature/schedule/application/ScheduleResponseAssembler.java

@@ -2,7 +2,6 @@
 
 import com.cliptripbe.feature.place.dto.response.PlaceListResponse;
 import com.cliptripbe.feature.schedule.domain.entity.Schedule;
-import com.cliptripbe.feature.schedule.dto.response.ScheduleListResponse;
 import com.cliptripbe.feature.schedule.dto.response.ScheduleResponse;
 import com.cliptripbe.feature.translate.dto.response.TranslationInfoDto;
 import com.cliptripbe.feature.user.domain.entity.User;
@@ -43,19 +42,4 @@ public static ScheduleResponse createBookmarkResponseForForeign(
             .toList();
         return ScheduleResponse.of(scheduleWithPlaces, placeListResponses);
     }
-
-    public static List<ScheduleListResponse> createScheduleListResponse(List<Schedule> scheduleList) {
-        return scheduleList
-            .stream()
-            .map(ScheduleResponseAssembler::mapScheduleListResponseDto)
-            .toList();
-    }
-
-    private static ScheduleListResponse mapScheduleListResponseDto(Schedule schedule) {
-        return ScheduleListResponse.builder()
-            .scheduleId(schedule.getId())
-            .scheduleName(schedule.getName())
-            .description(schedule.getDescription())
-            .build();
-    }
 }

src/main/java/com/cliptripbe/feature/schedule/application/ScheduleService.java

@@ -1,7 +1,6 @@
 package com.cliptripbe.feature.schedule.application;
 
 import static com.cliptripbe.feature.schedule.application.ScheduleResponseAssembler.createBookmarkResponseForForeign;
-import static com.cliptripbe.feature.schedule.application.ScheduleResponseAssembler.createScheduleListResponse;
 import static com.cliptripbe.feature.schedule.application.ScheduleResponseAssembler.createScheduleResponseForKorean;
 
 import com.cliptripbe.feature.bookmark.domain.service.BookmarkFinder;
@@ -89,8 +88,10 @@ public void updateSchedule(
 
     @Transactional(readOnly = true)
     public List<ScheduleListResponse> getUserScheduleList(User user) {
-        List<Schedule> scheduleList = scheduleRepository.findAllByUser(user);
-        return createScheduleListResponse(scheduleList);
+        List<Schedule> scheduleList = scheduleRepository.findAllByUserWithSchedulePlaces(user);
+        return scheduleList.stream()
+            .map(ScheduleListResponse::from)
+            .toList();
     }
 
     public void deleteSchedule(User user, Long scheduleId) {
@@ -121,10 +122,12 @@ public ScheduleResponse getScheduleById(
                 user
             );
         }
-        Map<Long, TranslationInfoDto> translationsForPlaces = placeService.getTranslationsForPlaces(places,
+        Map<Long, TranslationInfoDto> translationsForPlaces = placeService.getTranslationsForPlaces(
+            places,
             user.getLanguage());
 
-        return createBookmarkResponseForForeign(scheduleWithPlaces, translationsForPlaces, bookmarkIdsMap, user);
+        return createBookmarkResponseForForeign(scheduleWithPlaces, translationsForPlaces,
+            bookmarkIdsMap, user);
     }
 
 
@@ -148,8 +151,10 @@ public ScheduleResponse createScheduleByVideo(User user, List<Place> placeList)
                 user
             );
         }
-        Map<Long, TranslationInfoDto> translationsForPlaces = placeService.getTranslationsForPlaces(placeList,
+        Map<Long, TranslationInfoDto> translationsForPlaces = placeService.getTranslationsForPlaces(
+            placeList,
             user.getLanguage());
-        return createBookmarkResponseForForeign(schedule, translationsForPlaces, bookmarkIdsMap, user);
+        return createBookmarkResponseForForeign(schedule, translationsForPlaces, bookmarkIdsMap,
+            user);
     }
 }

src/main/java/com/cliptripbe/feature/schedule/domain/entity/Schedule.java

@@ -79,7 +79,6 @@ public void addSchedulePlace(SchedulePlace newSchedulePlace) {
             throw new CustomException(EXISTS_PLACE);
 
         }
-
         this.schedulePlaces.add(newSchedulePlace);
     }
 

src/main/java/com/cliptripbe/feature/schedule/dto/response/ScheduleListResponse.java

@@ -1,12 +1,22 @@
 package com.cliptripbe.feature.schedule.dto.response;
 
+import com.cliptripbe.feature.schedule.domain.entity.Schedule;
 import lombok.Builder;
 
 @Builder
 public record ScheduleListResponse(
     Long scheduleId,
     String scheduleName,
-    String description
+    String description,
+    Integer schedulePlaceCount
 ) {
 
+    public static ScheduleListResponse from(Schedule schedule) {
+        return ScheduleListResponse.builder()
+            .scheduleId(schedule.getId())
+            .scheduleName(schedule.getName())
+            .description(schedule.getDescription())
+            .schedulePlaceCount(schedule.getSchedulePlaces().size())
+            .build();
+    }
 }

src/main/java/com/cliptripbe/feature/schedule/infrastructure/ScheduleRepository.java

@@ -11,6 +11,9 @@
 
 public interface ScheduleRepository extends JpaRepository<Schedule, Long> {
 
+    @Query("SELECT s FROM Schedule s LEFT JOIN FETCH s.schedulePlaces WHERE s.user = :user")
+    List<Schedule> findAllByUserWithSchedulePlaces(@Param("user") User user);
+
     List<Schedule> findAllByUser(User user);
 
     @Query("SELECT s FROM Schedule s LEFT JOIN FETCH s.schedulePlaces sp WHERE s.id = :scheduleId")