[BE] SISC1-231 [FIX] 날짜 비교 형식 수정 및 dto 반경값 수정

[ochanhyeok]

Summary by CodeRabbit

릴리스 노트

• New Features

  • 위치 정보 응답에 반경 정보가 추가되었습니다.

• Refactor

  • 내부 시간 처리 로직이 최적화되었습니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

워크스루

두 개의 출석 관련 서비스에서 코드 정리 및 응답 확장이 이루어졌습니다. AttendanceService는 별도의 LocalDate와 LocalTime 호출을 단일 LocalDateTime.now()로 통합하고, AttendanceSessionService는 위치 응답에 radiusMeters 필드를 추가했습니다.

변경 사항

동향 / 파일(들)	요약
출석 서비스 리팩토링 backend/src/main/java/org/sejongisc/backend/attendance/service/AttendanceService.java	LocalDateTime.now()으로 통합하여 checkDate와 checkTime을 파생시킴. 날짜 및 시간 검증 로그와 반환 경로를 현재 시간 객체로 업데이트.
출석 세션 응답 확장 backend/src/main/java/org/sejongisc/backend/attendance/service/AttendanceSessionService.java	AttendanceSessionResponse 변환 시 LocationInfo에 radiusMeters 필드 추가. Javadoc 주석 업데이트.

예상 코드 리뷰 노력

🎯 1 (Trivial) | ⏱️ ~4분

• AttendanceService의 LocalDateTime 리팩토링은 기계적이고 일관된 변경으로 로직 복잡도가 낮음
• AttendanceSessionService의 필드 추가는 직관적인 응답 확장

관련 가능성이 있는 PR

• [BE] SISC1-228 [FIX] session user 관련 로직 개선 #152: AttendanceService의 출석 시간/날짜 처리 및 검증/로깅 변경과 직접 관련
• [BE] SISC1-223 [FIX] 출석 세션 시간 변수 수정 #140: AttendanceService의 체크인 로직 (시간 검증 및 라운드 기반 확인/로깅) 수정과 관련
• [BE] SISC1-219 [FIX] : 출석세션 관련 불필요한 응답필드 수정 #126: AttendanceSessionService의 응답 구성 변경과 관련 (현재 PR은 radiusMeters 추가, 검색 PR은 여러 필드 제거)

제안된 리뷰어

• discipline24

시

🐰✨ 이제 한 번의 맥박으로,

시간은 통일되어 흐르고,

거리 정보는 더욱 풍성해져,

출석 기록은 더욱 정확하게 춤을 춥니다!

코드는 깔끔하고, 마음은 가볍고— 🌙

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 변경사항의 주요 내용을 명확하게 설명하고 있습니다. 날짜 비교 형식 수정과 DTO 반경값 수정이라는 두 가지 주요 변경사항을 정확히 요약합니다.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch SISC1-231-BE-날짜-비교-형식-수정

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

backend/src/main/java/org/sejongisc/backend/attendance/service/AttendanceService.java (1)

142-142: 검증 시점과 기록 시점의 타임스탬프 불일치를 확인하세요.

Line 54에서 캡처한 통합 타임스탬프 now를 검증에 사용하고 있지만, 여기서는 새로운 LocalDateTime.now()를 호출하여 출석 기록을 저장하고 있습니다. 검증 시점과 실제 기록되는 시점 사이에 시간 차이가 발생할 수 있습니다.

일관성을 위해 검증에 사용된 동일한 now 타임스탬프를 사용하는 것을 권장합니다:

         Attendance attendance = Attendance.builder()
                 .user(user)
                 .attendanceSession(session)
                 .attendanceRound(round)
                 .attendanceStatus(status)
-                .checkedAt(java.time.LocalDateTime.now())
+                .checkedAt(now)
                 .awardedPoints(session.getRewardPoints())
                 .checkInLocation(userLocation)
                 .build();

🧹 Nitpick comments (1)

backend/src/main/java/org/sejongisc/backend/attendance/service/AttendanceSessionService.java (1)

228-244: 위치 재설정 시 반경 기본값(0 vs 100) 주석/구현 불일치 가능성

메서드 주석에는 “반경은 기존 값 유지 또는 0으로 설정”이라고 되어 있지만, 실제 구현은 기존 값이 없을 때 100을 기본값으로 사용하고 있습니다.
기획 의도가 0인지 100인지 한번 확인 후,

• 의도가 0이라면 기본값을 0으로 변경하거나
• 의도가 100이라면 주석을 100으로 수정

중 하나로 맞춰 주시면 유지보수 시 혼동을 줄일 수 있을 것 같습니다.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between be1865d and 1ae376d.

📒 Files selected for processing (2)

• backend/src/main/java/org/sejongisc/backend/attendance/service/AttendanceService.java (3 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/service/AttendanceSessionService.java (2 hunks)

🔇 Additional comments (4)

backend/src/main/java/org/sejongisc/backend/attendance/service/AttendanceSessionService.java (2)

271-274: Javadoc에 radiusMeters 추가 반영 👍

위치 정보에 radiusMeters 포함을 명시해서 실제 DTO 스키마와 주석이 일관되게 맞춰졌습니다. 추후 API 문서/프론트 확인에도 도움이 될 것 같습니다.

---

279-283: LocationInfo에 radiusMeters 매핑 추가 적절

session.getLocation().getRadiusMeters()를 그대로 DTO로 전달하는 방식이라 기존 로직에 영향을 주지 않으면서 응답 필드만 확장되고, location == null 처리도 그대로 유지되어 있어 문제 없어 보입니다.

backend/src/main/java/org/sejongisc/backend/attendance/service/AttendanceService.java (2)

54-56: 시간 일관성 개선이 우수합니다!

단일 LocalDateTime.now()를 캡처하여 checkDate와 checkTime을 도출하는 방식은 중요한 개선사항입니다. 이전에 LocalDate.now()와 LocalTime.now()를 별도로 호출했을 경우, 자정 경계에서 두 호출 사이에 날짜가 바뀌는 레이스 컨디션이 발생할 수 있었습니다. 이 변경으로 모든 검증 로직이 동일한 시점을 기준으로 수행됩니다.

---

63-64: 로그 개선이 적절합니다.

통합된 now 타임스탬프를 로그에 표시하여 검증 시점의 전체 날짜와 시간을 함께 기록하도록 개선되었습니다. 디버깅 시 더 명확한 정보를 제공합니다.

Also applies to: 76-76, 84-85