-
Notifications
You must be signed in to change notification settings - Fork 389
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
[2단계 - 블랙잭 베팅] 우르(김현우) 미션 제출합니다. #512
Merged
Merged
Changes from all commits
Commits
Show all changes
45 commits
Select commit
Hold shift + click to select a range
88c659b
feat : 점수가 블랙잭인지 확인
java-saeng daa17c8
feat : Cards의 합이 블랙잭인지 확인
java-saeng dd595d2
feat : Money VO 생성
java-saeng 7f20782
feat : 게임 결과에 따라 베팅액 결정 전략 추가
java-saeng 831a4de
feat : 참여자는 자신의 베팅 결과 상태를 결정할 수 있다
java-saeng 75a93e8
fix : BigDecimal equals 수정
java-saeng ef49791
feat : 참여자는 자신의 베팅 결과에 따라 베팅액 결과 금액을 반환한다.
java-saeng e6a71d9
feat : CardTable은 승부에 따라 베팅액 결정 상태 요청을 Participant에게 보냄
java-saeng 12f993e
feat : CardTable 은 경기 결과에 따라 베팅 금액을 구해준다
java-saeng 13233d1
feat : 배팅 금액 받기
java-saeng f96991a
feat : 배팅금액상태 결정 되지 않음
java-saeng 2bcdd2a
feat : Participant 수익 출력
java-saeng ad73162
feat : 돈 합하기
java-saeng 23137ec
feat : 딜러 수익 구하기
java-saeng faadcd7
feat : 참여자와 딜러의 전체 최종 수익 구하기
java-saeng 10d28bb
refactor : interface method 이름 변경
java-saeng f592f12
refactor : 참여자와 딜러 점수 비교하는 것을 Participant 에 위임
java-saeng 3231837
feat : 처음 2장에서 승부가 나지 않을 상태 추가
java-saeng a0493de
refactor : 딜러와 참여자 점수를 비교를 Participant 에 위임
java-saeng 3d20ee5
refactor : null이 아닌 승부가 나지 않은 상태인 TieState 로 초기화
java-saeng 25d2653
refactor : first, finalMatchWith 메서드 간단하게 작성
java-saeng 010aa44
refactor : 조건문 삭제
java-saeng 85cd267
refactor : setter 삭제 후, 테스트 추가
java-saeng f41b4c9
refactor : 아직 승부가 결정이 안된 상태 null object 넣기
java-saeng d592ead
refactor : dealCardTo boolean 사용하지 않기 때문에 void 로 변경
java-saeng 7380a1a
refactor : dealer 가 블랙잭이면 게임을 바로 끝내야해서 상태가 정해진 participant 는 hit 할 수 없음
java-saeng 1f8dc61
refactor : CardArea -> Hand 로 변경
java-saeng 5804c51
refactor : 도메인 용어로 메서드 네임, 상수 이름 변경
java-saeng b6c1d40
feat : 처음 두 장을 받을 때, 보여주는 방식을 추상 메서드로 변경
java-saeng 31525fc
refactor : 카드를 보여주는 추상 메서드 존재하기 때문에 기존 코드 삭제
java-saeng ee39bd0
test : final match 를 테스트 할 때, first match 도 실행
java-saeng f3633fc
test : dealer 얻은 결과 테스트 작성
java-saeng 839e394
style : pr 제출 전 reformatting
java-saeng 988c4bc
refactor : 객체 메시지 목록은 관용적인 getter보다 위에 위치
java-saeng d2d5853
refactor : 블랙잭 조건은 카드가 2장의 합이 21이어야함
java-saeng b077eb1
refactor : 점수 계산할 때 softHand일 경우 바로 score 리턴
java-saeng a6d953a
refactor : 패키지 이동
java-saeng 05d0da9
feat : 배팅 상태 판단하는 클래스 추가
java-saeng a87489c
refactor : BetResultFinder 가 베팅 결과 상태를 찾아서 Participant 가 돈을 계산
java-saeng 9b6c0ed
test : 테스트 메서드 명 수정
java-saeng 6cce44c
fix : DealerBust 에서 참여자가 블랙잭이 아니면서 3장 이상의 카드 합이 21일 때 진 걸로 결정
java-saeng 69398fe
refactor : @FunctionalInterface 추가 및 NULL object 삭제
java-saeng fb490eb
refactor : DealerBust 조건 변경 및 사용하지 않는 메서드 삭제
java-saeng 5f87fc6
refactor : 딜러와 참여자가 블랙잭이 아니더라도 나중에 점수가 같으면 무승부이다.
java-saeng 23568b0
style : 리뷰 요청 전 code formatting
java-saeng File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
보통 Score라는 객체를 만들 때 모든 점수값을 담을 수 있는 범용적인 Score 객체로 만들지는 않습니다.
이 객체 같은 경우 좀 더 분명하게 이름을 짓는다면 BlackjackScore 정도 되겠죠.
다만, 지금은 블랙잭을 다루는 어플리케이션이기 때문에 굳이 이렇게 지을 필요는 없는 것 같아요.
대부분의 리팩토링은 정말 필요할 때 해도 됩니다. 나중에 다른 게임이 추가되고 이 객체의 이름이 헷갈린다는 생각이 들면 그 때 해도 될거에요.
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.
그렇군요,, 저는 Score나 Money 와 같은 객체들은 범용적으로 사용한다는 생각을 당연하게 하고 있었습니다
그래서 위와 같은 고민이 생겼구요,,
범블비 말씀대로 범용적은 Score를 만드는게 일반적이지 않다면 필요할 때 리팩토링하는 식으로 진행하는게 합리적으로 보입니다~~