From 89272286e1e6abd0d69d55a07246850cd2ea4857 Mon Sep 17 00:00:00 2001
From: xonmin <han51361@gmail.com>
Date: Thu, 15 Aug 2024 22:04:02 +0900
Subject: [PATCH] refactor : excludeQSet to latestQset in create QSet

---
 .../com/mashup/dojo/service/QuestionService.kt       | 12 +++++-------
 .../com/mashup/dojo/usecase/QuestionUseCase.kt       |  2 +-
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/service/src/main/kotlin/com/mashup/dojo/service/QuestionService.kt b/service/src/main/kotlin/com/mashup/dojo/service/QuestionService.kt
index f8689534..a69c88ea 100644
--- a/service/src/main/kotlin/com/mashup/dojo/service/QuestionService.kt
+++ b/service/src/main/kotlin/com/mashup/dojo/service/QuestionService.kt
@@ -61,7 +61,7 @@ interface QuestionService {
         emojiImageId: ImageId,
     ): QuestionId
 
-    fun createQuestionSet(excludedQuestionSet: QuestionSet?): QuestionSetId
+    fun createQuestionSet(latestQuestionSet: QuestionSet?): QuestionSetId
 
     fun createQuestionSet(
         questionIds: List<QuestionId>,
@@ -149,10 +149,10 @@ class DefaultQuestionService(
     }
 
     @Transactional
-    override fun createQuestionSet(excludedQuestionSet: QuestionSet?): QuestionSetId {
+    override fun createQuestionSet(latestQuestionSet: QuestionSet?): QuestionSetId {
         // 비율에 따라 questionType 선정
         val friendQuestionSize = floor(questionSetSize * friendQuestionRatio).toInt()
-        val excludedQuestionIds: List<String> = excludedQuestionSet?.questionIds?.map { it.questionId.value } ?: emptyList()
+        val excludedQuestionIds: List<String> = latestQuestionSet?.questionIds?.map { it.questionId.value } ?: emptyList()
 
         val friendQuestions =
             questionRepository.findRandomQuestions(com.mashup.dojo.QuestionType.FRIEND, excludedQuestionIds, Pageable.ofSize(friendQuestionSize))
@@ -167,7 +167,7 @@ class DefaultQuestionService(
             log.error {
                 "QSet 을 만들기 위한 남은 Question 들이 부족합니다. " +
                     "조회한 QuestionSize : ${questionList.size}, 친구용 질문 size : $friendQuestionSize, 전체용 질문 size : ${accompanyQuestions.size}, " +
-                    "이전 QSetId : ${excludedQuestionSet?.id}, 제외한 QuestionIds : $excludedQuestionIds"
+                    "이전 QSetId : ${latestQuestionSet?.id}, 제외한 QuestionIds : $excludedQuestionIds"
             }
 
             throw DojoException.of(DojoExceptionType.QUESTION_LACK_FOR_CREATE_QUESTION_SET)
@@ -179,10 +179,8 @@ class DefaultQuestionService(
             }
 
         // 마지막 QSet 의 발행 시각 가져오기
-        val latestQSet = getLatestPublishedQuestionSet()
-
         val publishedTime =
-            latestQSet?.endAt ?: run {
+            latestQuestionSet?.endAt ?: run {
                 val now = LocalTime.now()
                 val today = LocalDate.now()
 
diff --git a/service/src/main/kotlin/com/mashup/dojo/usecase/QuestionUseCase.kt b/service/src/main/kotlin/com/mashup/dojo/usecase/QuestionUseCase.kt
index 7572b84d..88913c4c 100644
--- a/service/src/main/kotlin/com/mashup/dojo/usecase/QuestionUseCase.kt
+++ b/service/src/main/kotlin/com/mashup/dojo/usecase/QuestionUseCase.kt
@@ -110,7 +110,7 @@ class DefaultQuestionUseCase(
         // 가장 마지막에 만들어진 QSet 정보는 제외
         val latestQSet = questionService.getLatestPublishedQuestionSet()
 
-        return questionService.createQuestionSet(excludedQuestionSet = latestQSet)
+        return questionService.createQuestionSet(latestQuestionSet = latestQSet)
     }
 
     @Transactional