Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[LBP] 현정빈 로또 미션 2-5 단계 제출합니다. #90

Open
wants to merge 11 commits into
base: jeongbeanhyun
Choose a base branch
from

Conversation

JeongBeanHyun
Copy link

@JeongBeanHyun JeongBeanHyun commented Mar 3, 2025

리뷰어: 김우진님
리뷰이: 현정빈


3주차 리뷰

우진 리뷰어님 안녕하세요. 로또 미션 제출합니다!
2-5 단계 미션을 수행하면서 어려움이 많았습니다...!
특히, 코드 간의 흐름을 파악하는 데 시간이 많이 걸렸습니다. 부족한 부분이 많은 것 같습니다.. 리팩토링 과정에서 보완하겠습니다..!
이번 구현에서 SRP를 잘 준수하였는지, MVC 패턴에 알맞게 작성했는지 궁금합니다.
현재 컨트롤러에서 많은 로직을 처리하고 있는데, 서비스 클래스를 만들어 책임을 분리하는 것이 적절한 방법인지 고민됩니다.


질문

  • 테스트 코드에서 적절하게 테스트를 작성한 것인지 궁금합니다. 예외처리에서 추가할 부분이 있을까요?
  • SRP 를 잘 준수하였는지 궁금합니다..!
  • 서비스 클래스를 추가할 경우, 예외처리는 서비스에서 하는 것이 적절한지 혹은 컨트롤러에서 하는 것이 적절한지 고민됩니다. 우진 리뷰어님께서는 무엇이 더 적절하다고 생각하시나요?

기능 설명

Controller

  • LottoController : 사용자 입력을 받아서 로또 티켓을 생성하고, 당첨 결과를 출력합니다.

Model

  • Lotto : 로또 한 장(6개의 숫자) 정보를 저장합니다.
  • LottoNumbers : 로또 번호에 대한 검증 로직을 포함합니다.
  • LottoResult : 로또 당첨 결과를 계산합니다.
  • LottoTickets : 사용자가 구매한 로또 티켓 여러 장을 관리합니다.
  • Money : 로또 구매에 사용되는 금액을 관리합니다.
  • Rank : 로또 당첨 등수를 나타내는 열거형(Enum) 클래스입니다.
  • WinningNumbers : 당첨 번호와 보너스 번호를 저장합니다.

View

  • ErrorView : 에러 메시지를 출력합니다.
  • InputView : 사용자로부터 입력을 받습니다.
  • ResultView : 로또 결과를 출력합니다.

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.

1 participant