-
Notifications
You must be signed in to change notification settings - Fork 8
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
[로또 게임] 이세원 과제 제출합니다. #7
base: main
Are you sure you want to change the base?
Conversation
readme 추가
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
추상화 관점에서 리뷰했습니다.
추가적으로 함수들에 유닛 테스트까지 작성해보세요.
private final CountMatch countMatch; | ||
private List<Lotto>lottoList; | ||
|
||
public Game(Input input, Print print, CountMatch countMatch){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
외부에서 주입받는거면 Interface 형태로 리팩토링 해보세요.
System.out.println(cycle + "개를 구매했습니다."); | ||
|
||
// 3. 수량만큼 로또 번호 생성 | ||
makeRandomLottoNumbers(cycle,lottoList); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
static 함수로 만들어서 사용하는 것과 countMatch.countMatchNumber
처럼 외부에서 주입받아서 함수를 사용하는 것이 어떤 기준으로 다른 방식으로 구현한건가요?
private List<Lotto>lottoList; | ||
|
||
public LottoGame(Input input, Print print, CountMatch countMatch){ | ||
this.input = input; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Game이 아니라 파라미터로 들어오는 Input, Print, CountMatch들을 인터페이스화 시키라는 뜻이었습니다.
클래스를 적절하게 분리하는 부분과,
번호 일치여부 판별 및 당첨금 계산 부분에서 어려움을 겪었습니다.
인터페이스 사용과 enum 사용이 어려워서 사용해보지 못했는데, 남은시간 더 노력해보겠습니다.