Merge pull request #332 from GSM-MSG/feature/325-already-exist-attend…

…ance

출석부 중복 생성 예외 pr에 빠진 코드리뷰 반영

src/main/kotlin/com/msg/gcms/domain/attendance/repository/CustomScheduleRepository.kt

@@ -7,5 +7,5 @@ import java.time.LocalDate
 
 interface CustomScheduleRepository {
     fun queryByDate(club: Club, date: LocalDate?): Schedule?
-    fun existByDateAndPeriod(date: LocalDate, period: Period): Boolean
+    fun existByDateAndPeriods(date: LocalDate, period: List<Period>): Boolean
 }

src/main/kotlin/com/msg/gcms/domain/attendance/repository/impl/CustomScheduleRepositoryImpl.kt

@@ -21,12 +21,13 @@ class CustomScheduleRepositoryImpl(
             ).fetchOne()
     }
 
-    override fun existByDateAndPeriod(date: LocalDate, period: Period): Boolean {
+    override fun existByDateAndPeriods(date: LocalDate, period: List<Period>): Boolean {
         return jpaQueryFactory.selectOne()
             .from(attendance)
+            .leftJoin(attendance.schedule, schedule)
             .where(
                 attendance.schedule.date.eq(date),
-                attendance.period.eq(period)
+                attendance.period.`in`(period)
             )
             .fetchFirst() != null
     }

src/main/kotlin/com/msg/gcms/domain/attendance/service/impl/CreateScheduleServiceImpl.kt

@@ -31,9 +31,10 @@ class CreateScheduleServiceImpl(
         val schedule = scheduleConverter.toEntity(club, dto.schedule)
             .let { scheduleRepository.save(it) }
 
+        if(scheduleRepository.existByDateAndPeriods(dto.schedule.date, dto.period))
+            throw AlreadyScheduleExistException()
+
         val attendances = dto.period.flatMap { period ->
-            if(scheduleRepository.existByDateAndPeriod(dto.schedule.date, period))
-                throw AlreadyScheduleExistException()
 
             members.map { member ->
                 attendanceConverter.toEntity(