[refactor] 챌린지 참여 기록을 위한 메서드 리팩토링 #294

[solgito]

개요

• 챌린지 별 참여 기록 데이터를 생성하는 ChallengeRecordsService 클래스를 중심으로 서비스 메서드는 비즈니스 로직에 집중하도록 리팩토링했습니다.

---

코드 주요 내용

• 챌린지 별 참여 기록 데이터에는 성공 날짜 목록, 실패 날짜 목록, 다가오는 날짜 목록의 세 요소가 존재합니다.
• 이 세 요소를 관리하기 위한 ChallengeRecords DTO가 추가됐습니다.

// ChallengeRecords.java

public class ChallengeRecords {
    private List<LocalDate> successDayList;
    private List<LocalDate> failureDayList;
    private List<LocalDate> upcomingDayList;

    public ChallengeRecords() {
        successDayList = new ArrayList<>();
        failureDayList = new ArrayList<>();
        upcomingDayList = new ArrayList<>();
    }

    public void addSuccessDay(LocalDate date) {
        successDayList.add(date);
    }

    public void addFailureDay(LocalDate date) {
        failureDayList.add(date);
    }

    public void addUpcomingDay(LocalDate date) {
        upcomingDayList.add(date);
    }
}

---

• 위의 DTO를 활용해 비즈니스 로직을 리팩토링했습니다.

// ChallengeRecordsService.java

public SuccessResponse<ChallengeRecordsResponse> getChallengeRecords(Long challengeId, Member member) {
    ...

    ChallengeRecords challengeRecords = new ChallengeRecords();
    List<ChallengeParticipationRecord> recordList = challengeParticipationRecordSearchService.findAllByChallengeEnrollment(enrollment);
    LocalDate today = TimeZoneConverter.convertEtcToLocalTimeZone(ZonedDateTime.now()).toLocalDate();

    recordList.forEach(record -> categorizeRecord(record, today, challengeRecords));

    return SuccessResponse.of(
            SuccessCode.NO_MESSAGE,
            ChallengeRecordsResponse.builder()
                    .successDayList(challengeRecords.getSuccessDayList())
                    .failureDayList(challengeRecords.getFailureDayList())
                    .upcomingDayList(challengeRecords.getUpcomingDayList())
                    .build()
    );
}

• recordList를 순회하며 categorizeRecord() 메서드를 이용해 분류합니다.
• categorizeRecord() 메서드를 통해 분류된 날짜 데이터는 ChallengeRecords 내 적절한 목록에 담기게 됩니다.

---

• categorizeRecord() 메서드입니다.

private void categorizeRecord(ChallengeParticipationRecord record, LocalDate today, ChallengeRecords challengeRecords) {
    LocalDate targetDate = TimeZoneConverter.convertEtcToLocalTimeZone(record.getTargetDate()).toLocalDate();

    if (targetDate.isBefore(today)) {
        if (record.existsChallengePost()) {
            challengeRecords.addSuccessDay(targetDate);
        } else {
            challengeRecords.addFailureDay(targetDate);
        }
    } else if (targetDate.isEqual(today)) {
        if (record.existsChallengePost()) {
            challengeRecords.addSuccessDay(targetDate);
        } else {
            challengeRecords.addUpcomingDay(targetDate);
        }
    } else {
        challengeRecords.addUpcomingDay(targetDate);
    }
}

• record의 성공/실패 여부, API 요청일과 날짜 비교 등을 통해, 성공 날짜 목록, 실패 날짜 목록, 다가오는 날짜 목록에 날짜 데이터를 추가합니다.

---

기타

• 변수의 의미를 명확히 하기 위해, fail을 사용하는 변수명은 failure로 교체했습니다. (동사 -> 명사)

// ex

failDayList => failureDayList

---

• 변수명이 변경되어 관련 테스트 코드를 수정했습니다.