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.

Sign up for GitHub

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

Jump to bottom

[로또] 박수현 미션 제출합니다. #206

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

[로또] 박수현 미션 제출합니다. #206

wants to merge 42 commits into from

Conversation

Suxxxxhyun
Copy link

제가 사용한 domain은 아래와 같습니다.

  • InputView (입력화면)
  • OutView (출력화면)
  • PurchaseAmount (구매금액과 로또갯수를 저장하기 위한 객체)
  • PurchaseLottos (로또갯수에 따른 로또 목록을 저장하기 위한 객체)
  • LottoOneSet (하나의 로또를 저장하기 위한 객체)
  • Lotto (당첨번호를 저장하기 위한 객체)
  • BonusNumber (보너스 번호를 저장하기 위한 객체)
  • WinningHistory(ENUM) (RANK에 따라, 당첨번호를 맞춘 갯수와 상금을 저장하기 위한 객체)
  • ProfitCalculator(당첨내역과 수익률을 저장하기 위한 객체)

suhyun added 30 commits November 6, 2023 08:51
docs: 기능구현목록정리
9287012
feat(InputView): 로또 구입금액을 입력받습니다.
26b4116
feat(PurchaseAmount): 사용자가 입력한 구매금액이 올바른지 검증합니다. 구매금액이 빈값이거나, 0보다 작거나…
a439d0d 
…, 천원단위가 아닌경우 이에 맞는 예외메시지를 출력합니다.
test(PurchaseAmountTest): 사용자가 입력한 구매금액이 올바른지 검증합니다. 구매금액이 빈값이거나, 문자로…
3b7f475 
… 이루어졌거나, 0보다 작거나, 천원단위가 아닌경우 이에 맞는 예외메시지를 출력합니다.
feat(PurchaseAmount): 사용자가 입력한 구매금액에 따라 생성할 수 있는 로또개수를 저장합니다.
61eef3e
test(PurchaseAmountTest): 사용자가 입력한 구매금액에 따라 생성할 수 있는 로또개수를 올바르게 반환하는지…
939e2cc 
… 테스트합니다.
feat(LottoOneSet, PurchaseLottos): Lotto의 생성자의 파라미터값이 List<Integer>이기…
9a0a977 
… 때문에 이를 활용하기 위해 LottoOneSet클래스를 생성합니다. LottoOneSet클래스를 통해 하나의 로또 리스트를 생성합니다. 이때, 하나의 로또 리스트를 생성할때, 1부터 45까지 이루어져있는 숫자 및 이 숫자들이 중복되지 않게 구현합니다. 이를 구현하기 위해, 제공된 List<Integer> numbers = Randoms.pickUniqueNumbersInRange(1, 45, 6);메소드를 활용합니다.
test(PurchaseLottosTest): 로또의 리스트가 서로 다른 숫자로 이루어졌는지, 1부터 45의 숫자로 이루어졌…
d650a21 
…는지를 테스트한다.
refactor(LottoOneSet, PurchaseLottos): 로또목록에서 하나의 로또 리스트를 생성 후 반환받기 위…
3e12fcd 
…해 getLottoOneSet메소드를 추가합니다.
refactor(LottoOneSetTest, PurchaseLottosTest): DisplayName을 서술형으로 수정하…
2b7ca52 
…였습니다.
test(LottoOneSetTest): LottoOneSet와 Lotto가 같은 객체인지 검증합니다.
c7335c6
feat(OutputView): 로또 구매 개수와 생성된 로또 리스트를 출력합니다.
db2dc54
refactor(OutputView): System.out.println()으로 출력되도록 수정하였습니다.
07f5417
refactor(OutputView): \n이 한번 되도록 수정하였습니다.
c67af63
feat(InputView): 당첨번호와 보너스번호를 입력받기 위해 enterWinningNumbers()와 enterBon…
11a8858 
…usNumber()을 추가합니다.
refactor(InputView): 당첨번호와 보너스번호를 입력받기 위한 메소드를 일부 수정하였습니다.
fb3fc90
feat(Lotto): 당첨번호를 올바르게 입력했는지 검증을 위한 메소드를 추가합니다.
19bcfe0
feat(BonusNumber): 보너스번호를 올바르게 입력했는지 검증을 위한 메소드를 추가합니다.
c1c6e40
feat(BonusNumber): 보너스번호는 이전에 입력한 당첨번호와 다른 숫자를 입력하기 위한 메소드를 추가합니다.
4d3fb73
test(LottoTest): 당첨번호를 검증하는 메소드가 올바르게 동작하는지에 대한 테스트코드를 작성하였습니다.
f1db9a2
test(BonusNumberTest): 보너스번호를 검증하는 메소드가 올바르게 동작하는지에 대한 테스트코드를 작성하였습니다.
64959a3
docs: 기능구현목록수정
3ba7028
docs: 기능구현목록을 수정하였습니다.
c960138
feat(ProfitCalculator,WinningHistory): 수익률을 계산하기 위해, 1등부터 5등까지의 당첨내역을…
d3edbc5 
… 담당하는 WinningHistory의 ENUM클래스를 생성하였습니다. 또한, 모든 로또 리스트를 순회하여, 1등부터 5등까지 몇번나왔는지를 HashMap을 이용해 구현하였습니다.
feat(ProfitCalculator): 수익률을 계산하기 위한 메소드를 구현하였습니다.
58990a0
feat(ProfitCalculator, WinningHistory): 1등부터 5등까지 몇번 일치했는지를 출력하기 위한 메…
dfd063f 
…소드를 구현하였습니다. WinningHistory의 순서를 바꿔서, 굳이 내림차순정렬하지 않도록 구현하였습니다.
feat(OutputView): 수익률과 당첨내역을 출력하기 위한 메소드를 작성하였습니다.
040e4ab
feat(ProfitCalculator): 당첨내역을 출력형식에 맞게 출력하도록 메소드를 구현하였습니다.
894c1c1
feat(LottoController): 로또게임에 대한 예외처리를 catch문에서 처리하도록 구현하였습니다.
15c99e0
refactor(LottoOneSet): ApplicationTest에서 Randoms.pickUniqueNumbersInR…
ce8015c 
…ange()로 반환된 리스트가 불변객체임을 알게 되었고, 불변리스트를 정렬할 수 없기에 새로운 리스트를 토대로 정렬하도록 수정하였습니다.
suhyun added 12 commits November 7, 2023 09:42
refactor(ProfitCalculator): ApplicationTest한 결과 null이 난 원인을 분석해보니, 당첨…
d6cb977 
…번호를 맞춘 개수가 0일 수 있음을 판단하였습니다. 따라서 null이 아닐때만 당첨내역이 갱신되도록 수정하였습니다.
refactor(LottoOneSet): 불필요한 코드를 제거하였습니다.
ba06517
refactor(ProfitCalculator): 2등인경우, '보너스볼일치' 메시지가 당첨금액보다 앞에 나오는 것을 인지하…
fa9a126 
…여, 이에 맞게 코드를 수정하였습니다.
refactor(WinningHistory): 올바른 상금이 반환되지 못하는 이슈로 코드를 수정하였습니다.
cb07def
test(WinningHistoryTest): 당첨번호를 맞춘 개수와 보너스번호를 맞췄는지에 따라 올바른 등수를 반환하는지 …
91c418f 
…테스트하였습니다.
test(ProfitCalculatorTest): 수익률이 올바르게 소수점 둘째자리에서 반올림되는지를 테스트하였습니다.
e76c8a2
refactor: 사용하지 않는 Java doc을 삭제하였습니다.
2477047
refactor(LottoOneSet): 불필요한 주석을 삭제하였습니다.
e2817a7
refactor(WinningHistory): 중복된 코드를 제거하였습니다.
4d96167
refactor(OutputView): 사용자마다 공통되는 문장은 OutputView에서 상수처리하였습니다.
6cfbfe6
docs: 기능구현목록을 수정하였습니다.
b81e79b
docs: 기능구현목록을 수정하였습니다.
a770d49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Suxxxxhyun