Skip to content

Conversation

@pleasebelieveme
Copy link
Owner

@pleasebelieveme pleasebelieveme commented Nov 3, 2025

  • 일치 개수와 보너스 일치 여부로 등수 판별
  • 등수별 상금 반환 (1등: 20억, 2등: 3000만, 3등: 150만, 4등: 5만, 5등: 5천)
  • 테스트 완료

Summary by CodeRabbit

릴리즈 노트

  • 새로운 기능

    • 로또 등급 시스템 도입 (1등~5등, 미당첨 구분 및 상금 정보 포함)
  • 문서

    • README 전면 개편: 아키텍처, 예외 처리 전략, 테스트 구성 상세 설명 추가
    • 기술 스택, 레이어 구조, 예외 케이스별 에러 메시지 명시
  • 테스트

    • 로또 등급 판정 로직 검증 테스트 추가

@coderabbitai
Copy link

coderabbitai bot commented Nov 3, 2025

Caution

Review failed

The pull request is closed.

개요

README.md 문서를 재구성하고 내용을 확장하며, 새로운 Rank 열거형을 도메인 계층에 추가하고 해당 테스트 스위트를 작성했습니다. 파서 레이어 도입, 재시도 로직, 예외 처리 전략 등이 문서에 반영되었습니다.

변경 사항

응집 / 파일 변경 요약
도메인 모델
src/main/java/lotto/domain/Rank.java
로또 당첨 순위를 나타내는 새로운 공개 열거형 Rank를 추가. FIRST부터 NONE까지 6개의 상수를 정의하며, 각각 일치 숫자 개수(matchCount), 보너스 볼 일치 여부(matchBonus), 상금(prize), 설명(description)을 포함. 정적 팩토리 메서드 valueOf(int, boolean)를 통해 일치 조건에 맞는 순위를 반환하고, getPrize(), getDescription(), isWinning() 접근자를 제공.
도메인 테스트
src/test/java/lotto/domain/RankTest.java
Rank 열거형에 대한 JUnit 테스트 스위트 추가. 각 순위별 정확한 상금 확인(FIRST부터 FIFTH까지 및 NONE), 매개변수화된 테스트를 통한 0~2개 일치 케이스 검증, 그리고 isWinning() 메서드의 참/거짓 여부를 확인하는 테스트 포함.
문서 업데이트
README.md
입력값 검증 섹션을 "재시도 로직" 포함으로 개명하고, 실행 방법 및 예시를 기술 스택 표기로 전환. 파서 레이어 도입, 예외 케이스 목록 대폭 확장, 도메인 구성요소(WinningLotto, LottoTickets, LottoResult) 설명 추가. 아키텍처 섹션에 Parser 레이어 명시, 테스트 구성에 파서 테스트 계층 추가, 예외 처리 전략 섹션 신규 추가. 학습 내용(Enum 활용, 일급 컬렉션, 경계값 테스트 등) 반영.

예상 코드 리뷰 노력

🎯 2 (단순) | ⏱️ ~10분

  • Rank 열거형: 구조가 명확하고 로직이 직관적이나, valueOf() 메서드의 매칭 로직과 NONE 기본값 처리 동작 확인 필요
  • RankTest 스위트: 표준적인 JUnit 테스트로 커버리지와 매개변수화 테스트의 완성도 확인
  • README.md 문서: 변경 범위가 광범위하나 대부분 정보성 업데이트이며, 파서 레이어 및 재시도 로직 설명의 정확성 검증

🐰 순위를 나눔에 새로운 Rank 태어났고,
테스트가 촘촘히 검증하네!
문서 한 장 펼쳐 보니 그림이 명확하니,
로또의 영역이 더욱 튼튼해지누나.

✨ 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 pleasebelieveme

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3c89ee4 and 9d5d7ab.

📒 Files selected for processing (3)
  • README.md (11 hunks)
  • src/main/java/lotto/domain/Rank.java (1 hunks)
  • src/test/java/lotto/domain/RankTest.java (1 hunks)

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

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

@pleasebelieveme pleasebelieveme merged commit c9a7eef into main Nov 3, 2025
1 check was pending
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.

2 participants