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

Step4 로또 수동 PR #550

Merged
merged 154 commits into from May 31, 2020
Merged

Step4 로또 수동 PR #550

merged 154 commits into from May 31, 2020

Conversation

JunilHwang
Copy link

안녕하세요! PR 리뷰 부탁드립니다~

p.s: 항상 프로젝트를 새로 구성하는 이유가 이전과의 차이점을 명확하게 구분하기 위해서입니다!

@JunilHwang
feat: [step1] 문자열 덧셈 계산기 클래스 작성 완료
8287c6b
@JunilHwang
test: [step1] 테스트 케이스 작성 완료
6eaa4a9
@JunilHwang
test: [step1] 테스트 케이스 작성 완료 (2)
66c1800
@JunilHwang
test: [step1] 테스트 케이스 작성 완료 (3)
778657b
@JunilHwang
test: [step1] 테스트 케이스 작성 완료 (4)
3136fdb
@JunilHwang
refactor: [step1] 리뷰 반영 완료
34a3700
@JunilHwang
test: [step1] 테스트 케이스 작성 완료
5c7e13d
@JunilHwang
test: [step1] 테스트 케이스 작성 완료 (2)
15be6ed
@JunilHwang
test: [step1] 테스트 케이스 작성 완료 (3)
dd79442
@JunilHwang
test: [step1] 테스트 케이스 작성 완료 (4)
3d3c7aa
@JunilHwang
refactor: [step1] 리뷰 반영 완료
a8ede45
@JunilHwang
Merge remote-tracking branch 'origin/junilhwang' into junilhwang
20c23aa
@JunilHwang
refactor: [step2] 프로젝트 초기화 중
e077e5d
@JunilHwang
feat: [step2] InputView 작성 완료
3267fac
@JunilHwang
feat: [step2] ResultView 작성 완료
ce41e4f
@JunilHwang
feat: [step2] Winning Price 작성 완료
d3f06ca
@JunilHwang
feat: [step2] Lotto Class 작성 완료
1fe12a2
@JunilHwang
refactor: [step2] Lotto 정책 변경에 따른 ResultView 수정
b11e32d
@JunilHwang
refactor: [step2] Lotto 생성자의 역할 최소화
449639e
@JunilHwang
feat: [step2] LottoGame 추가
85e4d96
@JunilHwang
feat: [step2] Lotto 번호를 비교하는 메소드 추가
196f36b
@JunilHwang
refactor: [step2] Lottos -> LottoGame 으로 클래스명 변경
271079e
@JunilHwang
fix: [step2] public으로 선언된 필드를 private 으로 변경
1c20c50
@JunilHwang
fix: [step2] public으로 선언된 필드를 private 으로 변경 (2)
c5127f8
@JunilHwang
refactor: [step2] WinningPrize를 LottoGame으로 옮김
1c84952
@JunilHwang
feat: [step2] 수익률 추가
6c94cb3
@JunilHwang
refactor: [step2] 수익률에 대한 타입 변경
27a38fc
@JunilHwang
feat: [step2] Controller 작성 완료
332a39e
@JunilHwang
fix: [step2] 싱글톤 생성 오류 수정
2aae695
@JunilHwang
fix: [step2] 로또 생성 번호 로직 수정
8f9ca77
@JunilHwang
refactor: [Step3] LottoGame 삭제
3cae112
@JunilHwang
refactor: [Step3] Prizes에 payoffRatio를 구하는 메소드 추가
7740f78
@JunilHwang
refactor: [Step3] 테스트 및 컨트롤러에 리팩토링 코드 적용
3c3d443
@JunilHwang
refactor: [Step3] Prizes -> LottoResult로 class name 변경
7cab79e
@JunilHwang
Merge branch 'junilhwang' of https://github.com/next-step/java-lotto …
1372180 
…into junilhwang
@JunilHwang
Merge branch 'junilhwang' into step3
0196322
@JunilHwang
feat: [Step4] 프로젝트 초기화 완료
dba08ae
@JunilHwang
feat: [Step4] 로또 연결 기능 추가
3b9e03e
@JunilHwang
feat: [Step4] 수동구매 로직 추가
42a300d
@JunilHwang
feat: [Step4] LottoShop에 수동/자동 로또 구매 로직 추가
7a9733d
@JunilHwang
refactor: [Step4] 로직 추가에 따른 클래스 수정
f3079f9
@JunilHwang
refactor: [Step4] Lottos 클래스 리팩토링
12705b0
@JunilHwang
refactor: [Step4] ResultView 리팩토링 코드 적용
970129d
@JunilHwang
feat: [Step4] 자동 완성 로또만 구매할 수 있는 로직 추가
dfa3e2c
@JunilHwang
test: [Step4] 기존 테스트 추가
60c46cf
@JunilHwang
refactor: [Step4] LottoShopTest 리팩토
cd0ac5e
@JunilHwang
feat: [Step4] LottosTest 추가
d599788
seok-2-o
seok-2-o requested changes May 29, 2020
View reviewed changes
Copy link

@seok-2-o seok-2-o left a comment

Choose a reason for hiding this comment

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

구조가 잘 잡혀 있어 큰 어려움 없이 마지막 미션도 진행해주셨네요 💯
일급 콜렉션 관련하여 코멘트 남겨 두었습니다. 확인 후 리펙토링 진행해주세요.

좋은 하루 되세요.

src/main/java/lotto/step4/domain/LottoNumber.java Outdated Show resolved Hide resolved
src/main/java/lotto/step4/domain/LottoResult.java Outdated Show resolved Hide resolved
src/main/java/lotto/step4/domain/Lottos.java Outdated Show resolved Hide resolved
src/main/java/lotto/step4/domain/LottoShop.java Outdated Show resolved Hide resolved
src/main/java/lotto/step4/domain/LottoResult.java Outdated Show resolved Hide resolved
src/main/java/lotto/step4/domain/LottoResult.java Outdated Show resolved Hide resolved
src/main/java/lotto/step4/domain/Lotto.java Outdated Show resolved Hide resolved
src/main/java/lotto/step4/domain/LottoNumber.java Show resolved Hide resolved
seok-2-o
seok-2-o approved these changes May 31, 2020
View reviewed changes
Copy link

@seok-2-o seok-2-o left a comment

Choose a reason for hiding this comment

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

피드백 반영하느라 고생 많으셨습니다. 💯
좋은 개발자로 성장하는데 조금이마 도움이 되었기를 바라봅니다.

저와의 미션은 끝이 났지만 궁금한점이나 이야기 나누고 싶은게 있다면 편하게 DM 주세요 :)
힘내서 다른 미션도 꼭 완주 하세요. 감사합니다.

@seok-2-o seok-2-o merged commit c257803 into next-step:junilhwang May 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@seok-2-o seok-2-o seok-2-o approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@JunilHwang @seok-2-o