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

[로또 게임] 이세원 과제 제출합니다. #7

Open
wants to merge 24 commits into
base: main
Choose a base branch
from

Conversation

leesewon00
Copy link
Member

클래스를 적절하게 분리하는 부분과,
번호 일치여부 판별 및 당첨금 계산 부분에서 어려움을 겪었습니다.
인터페이스 사용과 enum 사용이 어려워서 사용해보지 못했는데, 남은시간 더 노력해보겠습니다.

Copy link

@KeonHee KeonHee left a 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){
Copy link

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);
Copy link

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;
Copy link

Choose a reason for hiding this comment

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

Game이 아니라 파라미터로 들어오는 Input, Print, CountMatch들을 인터페이스화 시키라는 뜻이었습니다.

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