8000 블랙잭 미션 Step1 : 리뷰 요청 드립니다! by seongbeenkim · Pull Request #6 · kiworkshop/java-blackjack · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

블랙잭 미션 Step1 : 리뷰 요청 드립니다! #6

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
Open
wants to merge 80 commits into
base: seongbeenkim
Choose a base branch
from
Open

블랙잭 미션 Step1 : 리뷰 요청 드립니다! #6

wants to merge 80 commits into from

Conversation

seongbeenkim
Copy link

안녕하세요 광일 선생님
리뷰 요청 늦게 드려서 죄송합니다..
eef0805 - 초기화 커밋 이후부터 봐주시면 감사하겠습니다.

학습 로그

상태 패턴, 추상 골격 구현 클래스

  • 블랙잭 미션에 상태 패턴을 적용하면서, 플레이어와 딜러가 가지게 되는 중복된 메서드, 인스턴스 변수를 제거하기 위해서 추상 골격 구현 클래스라는 개념을 공부하여 중복을 최소한으로 줄이려고 노력했습니다
  • 참조 : 넥스트 스텝 볼링 미션 피드백 - State 패턴, 아이템 [20] - 추상 클래스 보다는 인터페이스를 우선하라

Deque vs Stack

  • Stack으로 구현했을 때 Sonarlint에서 Stack는 사용하지 않는 것이 좋다라는 경고문을 읽게 되었습니다. 왜 사용하지 않아야 되며 어떤 자료구조로 대체할 수 있는 지 찾아보고 Deque를 적용하였습니다.
  • 참조 : Java 의 Stack 대신 Deque

@seongbeenkim
docs: 구현해야할 기능 목록 추가
0169491
@seongbeenkim
feat: 플레이어 이름 입력 및 공백 또는 null 체크
73b0cc1
@seongbeenkim
feat: 카드 한 장에 이름과 값을 체크해서 생성하는 테스트 코드 및 기능 구현
ac07009
@seongbeenkim
feat: 카드 한 장의 점수를 반환하는 테스트 및 기능 구현
e378c98
@seongbeenkim
feat: 카드 52개 생성되는 테스트 및 기능 구현
6b566fc
@seongbeenkim
feat: 초기 2장의 카드 지급, 추가 카드 지급 테스트 및 기능 구현
7052997
10000
@seongbeenkim
feat: 더 이상 카드를 반환할 수 없는 경우 예외 발생 테스트 및 기능 구현
d57570d
@seongbeenkim
feat: 플레이어(딜러 제외) 추가 카드 요청 입력 기능 추가
16becf5
@seongbeenkim
feat: 각 플레이어(딜러 포함)에게 2장의 카드 지급 알림 출력 기능 구현
2885dbb
@seongbeenkim
feat: 플레이어가 추가 카드 발급받는 테스트 및 기능 구현
245b7c5
@seongbeenkim
feat: 플레이어가 추가 카드 발급받는 테스트 및 기능 구현
5e00eda
@seongbeenkim
feat: 플레이어가 가진 카드 합을 반환하는 테스트 및 기능 구현
ed21358
@seongbeenkim
feat: 참가자 이름 리스트로 등록하면 게임에 참가자들이 생성되는 테스트 및 기능 구현
022020f
@seongbeenkim
feat: 카드가 Ace인지 확인하는 테스트 및 기능 구현
4dd158f
@seongbeenkim
feat: 카드의 합을 21을 기준으로 Ace 1 또는 11로 설정하는 테스트 및 기능 구현
926367e
@seongbeenkim
feat: 카드의 합이 21을 초과할 경우 참을 반환하는 테스트 및 기능 구현
7c1f065
@seongbeenkim
초기화
eef0805
@seongbeenkim
feat: 카드 점수 enum 속성 getter 테스트 및 기능 구현
05c1689
@seongbeenkim
feat: 카드 무늬 enum 속성 getter 테스트 및 기능 구현
32ba1ac
@seongbeenkim
feat: 카드 점수 enum 객체가 Ace인지 확인하는 테스트 및 기능 구현
099cfce
@seongbeenkim
feat: 카드 한 장 객체에 대한 테스트 및 기능 구현
78109d2
@seongbeenkim
feat: 카드 덱에서 카드 1장, 2장 반환하는 테스트 및 기능 구현
dd35ba8
@seongbeenkim
feat: 게임 참여자들이 받는 카드 목록에 대한 테스트 및 기능 구현
d1c6199
@seongbeenkim
feat: 게임 참여자들이 받는 카드 목록에 카드 한 장 추가하는 테스트 및 기능 구현
335f55c
@seongbeenkim
feat: 참여 상태(Running) 테스트 및 기능 구현
3b64f71
@seongbeenkim
feat: 종료 상태(Finished) 테스트 및 기능 구현
bb05ebb
@seongbeenkim
feat: 시작 상태(Started) 테스트 및 기능 구현
2ba425f
@seongbeenkim
feat: 모든 상태에서 카드 합 반환, 카드 목록 반환 기능 중복 코드 제거를 위한 Init 추상 클래스 추가 및 Star…
0a7d336 
…ted -> Hit으로 변경
@seongbeenkim
feat: 인자로 받은 카드의 합을 비교하는 테스트 및 기능 구현
4859fd6
@seongbeenkim
feat: 결과 점수 enum 구현
c07c6c6
@seongbeenkim
refactor: 플레이어의 카드 추가 시 카드 추가 응답을 인자도 받아오게 수정
c1c9137
@seongbeenkim
refactor: isFinished -> allPlayersFinished 이름 변경
bf12f9e
@seongbeenkim
feat: 플레이어(딜러 제외) 추가 카드 요청하는 기능 추가
16109f3
@seongbeenkim
refactor: Player 객체 -> Person 객체로 수정
a601843
@seongbeenkim
refactor: 인자로 받은 값보다 합이 작거나 같은 지 확인하는 기능 추가
d68db1d
@seongbeenkim
feat: 딜러 종료 여부 반환하는 테스트 및 기능 구현
c560333
@seongbeenkim
feat: 딜러의 추가 카드 지급 여부 결과 출력 기능 구현
adf3110
@seongbeenkim
refactor: 블랙잭 상수 Dealer -> GameSystem으로 이동
3dc4b10
@seongbeenkim
refactor: FirstTwoCardsResponse -> CardsResponse로 이름 변경
36211ae
@seongbeenkim
feat: 딜러, 플레이어들의 카드 반환하는 DealerAndPlayerCardsResponse 생성
10000
42f113a
@seongbeenkim
feat: 추가 요청 후 플레이어의 카드를 출력하는 기능 추가
2abc494
@seongbeenkim
feat: 각 플레이어(딜러 포함)의 총 카드의 합을 반환하는 기능 추가
80be38b
@seongbeenkim
refactor: result -> compare 이름 변경
47eb909
@seongbeenkim
feat: 총 카드의 합이 21인지 확인하는 테스트 및 기능 추가
23a37d0
@seongbeenkim
feat: 카드 추가 발급 시 카드의 합 21일 경우 Stay 상태 반환하는 테스트 및 기능 구현
2759710
@seongbeenkim
feat: Stay 상태에서 합 비교시 인자로 받아온 카드의 합이 21을 초과할 경우에 대한 테스트 및 조건 추가
d6cc2ce
@seongbeenkim
feat: 플레이어들의 승패 결과를 반환하는 테스트 및 기능 구현
e145190
@seongbeenkim
feat: 최종 승패 결과 출력 기능 추가
c434047
@seongbeenkim
feat: GameSystem의 플레이어 관련 기능 Players로 위임
035c28d
@seongbeenkim
refactor: Players에서 반환하는 객체 List<Card> -> GivenCards로 수정
db0aa56
@seongbeenkim
refactor: 카드 캐싱 적용
6f5254a
@seongbeenkim
refactor: Deck 캐싱 적용
bab3a7b
@seongbeenkim
feat: GameSystem 생성자 추가
ef17f16
@seongbeenkim
refactor: 외부에서 new 키워드로 GameSystem 생성하지 못하게 수정
af9c002
@seongbeenkim
refactor: 사용하지 않는 변수 제거
27b59a8
@seongbeenkim
style: dto 패키지 경로 수정
fb7caf3
@seongbeenkim
style: dto 패키지 경로 수정
e3dac27
@seongbeenkim
refactor: @CsvSource String -> Enum 값으로 전달하게 수정
fe515c5
@seongbeenkim
refactor: unmodifiableCollection 적용
ca4a947
@seongbeenkim
refactor: 불필요한 상태 제거
8a80512
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
@seongbeenkim
0