Skip to content

fix: type int to long#100

Merged
mjk25 merged 1 commit intodevfrom
fix/99-stayduration-type
Feb 27, 2026
Merged

fix: type int to long#100
mjk25 merged 1 commit intodevfrom
fix/99-stayduration-type

Conversation

@mjk25
Copy link
Copy Markdown
Collaborator

@mjk25 mjk25 commented Feb 26, 2026

작업 내용

stayDuration 타입 변경

참고

  • 리캡 자동 생성 중 DB가 stayDuration을 Long으로 반환했는데
    프로젝션 생성자 파라미터가 Int라서 난 타입 불일치

관련 이슈

@mjk25 mjk25 requested review from 1117mg and earlgrey02 February 26, 2026 16:38
@mjk25 mjk25 self-assigned this Feb 26, 2026
@mjk25 mjk25 linked an issue Feb 26, 2026 that may be closed by this pull request
@github-actions
Copy link
Copy Markdown

🤖 Gemini Code Review

✅ 변경 사항 요약

  • UserActivityProjectionstayDuration 타입을 Int에서 Long으로 확장.
  • 서비스 레이어 및 테스트 피처 내 관련 데이터 변환 로직 수정.

🚨 치명적인 이슈 (Critical Issues)

  • 🛑 RecapService / RecapFixtures
    • Problem: stayDurationLong으로 변경했음에도 불구하고, 결과 객체로 변환할 때 .toInt()를 사용하여 강제 형변환을 수행함. 만약 값이 Int.MAX_VALUE를 초과할 경우 오버플로우(Overflow)로 인해 데이터가 왜곡될 위험이 있음.
    • Fix: 데이터를 전달받는 UserActivityRequest 및 관련 DTO의 stayDuration 필드 타입도 모두 Long으로 변경하고, 불필요하고 위험한 .toInt() 호출을 제거해야 함.

✏️ 파일별 상세 리뷰

RecapService.kt

  • 📍 Issue: Long 타입 데이터를 Int로 다운캐스팅하여 데이터 정밀도 손실 및 런타임 오류 위험이 발생함. 타입 변경의 목적이 값의 범위 확장이라면 하위 레이어까지 타입을 전파해야 함.

RecapFixtures.kt

  • 📍 Issue: 서비스 로직의 잠재적 버그를 테스트 코드에서도 동일하게 답습하고 있음. Long 타입을 온전히 유지하도록 수정 필요.

Copy link
Copy Markdown
Member

@earlgrey02 earlgrey02 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@earlgrey02
Copy link
Copy Markdown
Member

AI 리뷰보고 생각난건데, 그냥 다 Long으로 바꾸면 좋을 것 같아요!

@mjk25
Copy link
Copy Markdown
Collaborator Author

mjk25 commented Feb 27, 2026

AI 리뷰보고 생각난건데, 그냥 다 Long으로 바꾸면 좋을 것 같아요!

좋은 의견 감사합니다! 다만 수정시 프론트엔드 코드도 바꿔야하고, stayDuration이 int 범위를 초과할 만큼 길지 않을 것 같아서 우선은 이대로 진행하고, 추후에 수정하겠습니다

@mjk25 mjk25 merged commit 7934416 into dev Feb 27, 2026
1 check passed
@mjk25 mjk25 deleted the fix/99-stayduration-type branch February 27, 2026 05:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

stayDuration 타입 오류

3 participants