[EA3-183] refactor : incrementStats 메서드 제거 (recalculateStats 방식으로 통일)

Author: endorsement0912

src/main/java/grep/neogulcoder/domain/timevote/service/vote/TimeVoteService.java

@@ -10,6 +10,8 @@
 import grep.neogulcoder.domain.timevote.repository.TimeVoteQueryRepository;
 import grep.neogulcoder.domain.timevote.service.TimeVoteMapper;
 import grep.neogulcoder.domain.timevote.service.stat.TimeVoteStatService;
+import jakarta.persistence.EntityManager;
+import java.time.LocalDateTime;
 import java.util.List;
 import lombok.RequiredArgsConstructor;
 import lombok.extern.slf4j.Slf4j;
@@ -27,6 +29,7 @@ public class TimeVoteService {
   private final TimeVoteStatService timeVoteStatService;
   private final TimeVoteMapper timeVoteMapper;
   private final TimeVoteValidator timeVoteValidator;
+  private final EntityManager em;
 
   @Transactional(readOnly = true)
   public TimeVoteResponse getMyVotes(Long studyId, Long userId) {
@@ -42,56 +45,13 @@ public TimeVoteResponse getMyVotes(Long studyId, Long userId) {
   }
 
   public TimeVoteResponse submitVotes(TimeVoteCreateRequest request, Long studyId, Long userId) {
-    long totalStart = System.currentTimeMillis();
-    log.info("[TimeVote] 투표 제출 시작 - studyId={}, userId={}, 요청 슬롯 수={}",
-        studyId, userId, request.getTimeSlots().size());
-
-    // 투표 전 필요한 모든 유효성 검증 및 도메인 정보 캡슐화
     TimeVoteContext context = timeVoteValidator.getSubmitContext(studyId, userId, request.getTimeSlots());
-
-    long voteStart = System.currentTimeMillis();
-    List<TimeVote> votes = timeVoteMapper.toEntities(request, context.period(), context.studyMember().getId());
-    timeVoteRepository.saveAll(votes);
-    log.info("[TimeVote] 투표 저장 완료 - {}건, {}ms 소요", votes.size(), System.currentTimeMillis() - voteStart);
-
-    long statStart = System.currentTimeMillis();
-    timeVoteStatService.incrementStats(context.period().getPeriodId(), request.getTimeSlots());
-    log.info("[TimeVote] 통계 반영 완료 - {}ms 소요", System.currentTimeMillis() - statStart);
-
-    List<TimeVote> saved = timeVoteRepository.findByPeriodAndStudyMemberIdAndActivatedTrue(context.period(), context.studyMember().getId());
-    log.info("[TimeVote] 저장된 투표 재조회 완료 - {}건", saved.size());
-
-    log.info("[TimeVote] 투표 제출 전체 완료 - 총 {}ms 소요", System.currentTimeMillis() - totalStart);
-
-    return TimeVoteResponse.from(context.studyMember().getId(), saved);
+    return executeVoteSubmission(context, request.getTimeSlots());
   }
 
   public TimeVoteResponse updateVotes(TimeVoteUpdateRequest request, Long studyId, Long userId) {
-    long totalStart = System.currentTimeMillis();
-    log.info("[TimeVote] 투표 수정 시작 - studyId={}, userId={}, 요청 슬롯 수={}", studyId, userId, request.getTimeSlots().size());
-
-    // 투표 전 필요한 모든 유효성 검증 및 도메인 정보 캡슐화
     TimeVoteContext context = timeVoteValidator.getUpdateContext(studyId, userId, request.getTimeSlots());
-
-    long deleteStart = System.currentTimeMillis();
-    timeVoteRepository.deactivateByPeriodAndStudyMember(context.period(), context.studyMember().getId());
-    log.info("[TimeVote] 기존 투표 soft delete 완료 - {}ms 소요", System.currentTimeMillis() - deleteStart);
-
-    long saveStart = System.currentTimeMillis();
-    List<TimeVote> newVotes = timeVoteMapper.toEntities(request, context.period(), context.studyMember().getId());
-    timeVoteRepository.saveAll(newVotes);
-    log.info("[TimeVote] 새로운 투표 저장 완료 - {}건, {}ms 소요", newVotes.size(), System.currentTimeMillis() - saveStart);
-
-    long statStart = System.currentTimeMillis();
-    timeVoteStatService.recalculateStats(context.period().getPeriodId());
-    log.info("[TimeVote] 통계 재계산 완료 - {}ms 소요", System.currentTimeMillis() - statStart);
-
-    List<TimeVote> saved = timeVoteRepository.findByPeriodAndStudyMemberIdAndActivatedTrue(context.period(), context.studyMember().getId());
-    log.info("[TimeVote] 저장된 투표 재조회 완료 - {}건", saved.size());
-
-    log.info("[TimeVote] 투표 수정 전체 완료 - 총 {}ms 소요", System.currentTimeMillis() - totalStart);
-
-    return TimeVoteResponse.from(context.studyMember().getId(), saved);
+    return executeVoteSubmission(context, request.getTimeSlots());
   }
 
   public void deleteAllVotes(Long studyId, Long userId) {
@@ -122,4 +82,27 @@ public List<TimeVoteSubmissionStatusResponse> getSubmissionStatusList(Long study
     log.info("[TimeVote] 제출 현황 조회 완료 - 총 {}명", result.size());
     return result;
   }
+
+  private TimeVoteResponse executeVoteSubmission(TimeVoteContext context, List<LocalDateTime> timeSlots) {
+    long totalStart = System.currentTimeMillis();
+
+    timeVoteRepository.deactivateByPeriodAndStudyMember(context.period(), context.studyMember().getId());
+    em.flush();
+    log.info("[TimeVote] 기존 투표 soft delete 완료");
+
+    List<TimeVote> newVotes = timeVoteMapper.toEntities(timeSlots, context.period(), context.studyMember().getId());
+    timeVoteRepository.saveAll(newVotes);
+    log.info("[TimeVote] 새 투표 저장 완료 - {}건", newVotes.size());
+
+    timeVoteStatService.recalculateStats(context.period().getPeriodId());
+    log.info("[TimeVote] 통계 재계산 완료");
+
+    List<TimeVote> saved = timeVoteRepository.findByPeriodAndStudyMemberIdAndActivatedTrue(
+        context.period(), context.studyMember().getId());
+
+    log.info("[TimeVote] 저장된 투표 재조회 완료 - {}건", saved.size());
+    log.info("[TimeVote] 전체 완료 - 총 {}ms", System.currentTimeMillis() - totalStart);
+
+    return TimeVoteResponse.from(context.studyMember().getId(), saved);
+  }
 }