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

[1단계 - 블랙잭 게임 실행] 우르(김현우) 미션 제출합니다. #391

Merged
merged 60 commits into from
Mar 7, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
60 commits
Select commit Hold shift + click to select a range
e538576
docs : 기능 명세 작성
shin-mallang Feb 28, 2023
76aaeca
feat : cardValue 추가
shin-mallang Feb 28, 2023
97742fc
feat : 카드 서브클래스 추가
shin-mallang Feb 28, 2023
ba8e698
docs : 기능 명세 추가
shin-mallang Feb 28, 2023
ad1c3c9
feat : CardArea 생성
shin-mallang Feb 28, 2023
cd2f4e9
docs : 기능 명세 추가
shin-mallang Feb 28, 2023
1311e76
feat: 카드 추가 기능 구현
shin-mallang Feb 28, 2023
34f3580
feat : 카드 영역을 딜러 카드 영역과 참가자 카드 영역으로 나눔
shin-mallang Feb 28, 2023
d4a4c31
feat : 카드 점수 계산 기능 구현
shin-mallang Mar 1, 2023
99bc87f
feat : 카드를 더 받을 수 있는 상태인지 확인하는 기능 추가
shin-mallang Mar 1, 2023
9f5b973
feat : 카드를 더 받을 것인지 결정하는 기능 추가
shin-mallang Mar 1, 2023
1c9277d
refactor : 카드 모양을 enum으로 설정
shin-mallang Mar 1, 2023
e49a507
feat : 카드 전체 생성 기능 추가
shin-mallang Mar 1, 2023
542fb14
feat : 카드 뽑기 기능 추가
shin-mallang Mar 1, 2023
0f44385
feat : 참가자 이름 구현
shin-mallang Mar 1, 2023
fb25c59
feat : 참가자 카드 영역 Hit 여부 구현
shin-mallang Mar 1, 2023
328b773
refactor : 패키지 구조 수정
shin-mallang Mar 1, 2023
68ca019
fix : 카드값 계산 오류 수정
shin-mallang Mar 1, 2023
9311987
feat : 버스트 여부 확인
shin-mallang Mar 1, 2023
f662ffe
refactor : Player를 부모로 두고 Dealer와 Participant를 가지게끔 변경
shin-mallang Mar 2, 2023
3e41832
refactor : CardArea 를 구체 클래스로 변경
shin-mallang Mar 2, 2023
5f29f3c
feat : 블랙잭 게임 완성
shin-mallang Mar 2, 2023
a9b11a2
feat : 출력 메세지 변환
shin-mallang Mar 2, 2023
0c81d51
refactor: 게임 통계 로직 변경
shin-mallang Mar 2, 2023
390d577
refactor : 첫 번째 카드를 보여주는 행위는 CardArea 가 적합하다고 생각하여 옮김
java-saeng Mar 2, 2023
48de821
refactor : Dealer 최대 점수 상수화
java-saeng Mar 2, 2023
79fd9e7
refactor : enhanced-for 문 flatMap으로 수정
java-saeng Mar 2, 2023
97492f7
refactor : state != STAY 를 풀어서 작성
java-saeng Mar 2, 2023
5aefc5b
refactor : 패키지명 participant -> player 로 변경
java-saeng Mar 2, 2023
12fe147
refactor : dealer name 은 고정적으로 딜러다
java-saeng Mar 2, 2023
8f69d8f
feat : DealerResult 추가
java-saeng Mar 2, 2023
90940f4
refactor : OutputView 변경
java-saeng Mar 2, 2023
d33ac4a
refactor : 패키지 변경
java-saeng Mar 2, 2023
cb399f4
refactor : CardDeck의 자료구조 List -> Stack 으로 변경
java-saeng Mar 2, 2023
95b300e
style : pr 제출 전 reformatting
java-saeng Mar 3, 2023
4d88bea
fix : y 또는 n 을 입력하지 않을 경우 재입력 로직 추가
java-saeng Mar 3, 2023
94d8d7d
mission : mini mission ArrayList 구현하기
java-saeng Mar 3, 2023
fca34da
docs : readme 위치 변경
java-saeng Mar 4, 2023
61e8974
refactor : 블랙잭 규칙 중 burst -> bust 변경
java-saeng Mar 4, 2023
d236d9f
refactor : Player에 이미 점수 계산하는 메서드가 있어서 변경
java-saeng Mar 4, 2023
b09e72c
style : 개행 추가
java-saeng Mar 4, 2023
a05dfe2
refactor : 딜러에서 첫 번째 카드를 보여주는 메서드 추가
java-saeng Mar 4, 2023
30782d3
refactor : State 삭제
java-saeng Mar 4, 2023
eddb77b
feat : cardArea를 생성하는 CardTable 추가
java-saeng Mar 5, 2023
fef1279
feat : 점수를 관리하는 Score 객체 추가
java-saeng Mar 6, 2023
648f753
feat : 점수 관련 로직에서 int 가 아닌 Score 로 변경
java-saeng Mar 6, 2023
4606281
feat : 블랙잭 게임 결과를 CardTable에서 알려준다
java-saeng Mar 6, 2023
76947c8
refactor : 블랙잭 게임 결과를 CardTable 로 책임을 위임하면서 코드 변경
java-saeng Mar 6, 2023
8d60d6d
fix : 점수 출력을 위한 getter 메서드 추가
java-saeng Mar 6, 2023
254687c
refactor : 매직넘버 상수로 변경
java-saeng Mar 6, 2023
4b3dbd1
refactor : 값 객체의 변수 이름을 value 로 변경
java-saeng Mar 6, 2023
d0e0387
refactor : 카드 나눠주는 행위를 Controller에서 하지 않고 CardTable로 위임
java-saeng Mar 6, 2023
be60c82
refactor : 카드를 나눠줄 수 있으면 true를 반환
java-saeng Mar 6, 2023
f53acf3
refactor : 카드 나눠주는 행위를 CardTable 에 위임
java-saeng Mar 6, 2023
60bb6ec
style : 제출 전 reformatting
java-saeng Mar 6, 2023
7f54c2d
minimission : ArrayList, LinkedList 구현하기
java-saeng Mar 7, 2023
20fed88
feat : Score에서 Ace 계산하기
java-saeng Mar 7, 2023
4c9d00b
refactor : CardArea 에서 카드 값 계산 로직 수정
java-saeng Mar 7, 2023
3ae18c8
refactor : int 로 비교하지 않고 Score 에 있는 연산 사용
java-saeng Mar 7, 2023
0953389
style : 제출 전 code reformatting
java-saeng Mar 7, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
42 changes: 42 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,3 +5,45 @@
## 우아한테크코스 코드리뷰

- [온라인 코드 리뷰 과정](https://github.com/woowacourse/woowacourse-docs/blob/master/maincourse/README.md)

## 도메인

### 카드 값

- [x] `2~10`, `A`, `K`, `Q`, `J` 가 있다.
- [x] `K`, `Q`, `J` 는 10으로 계산된다.

### 카드

- [x] `다이아`, `클로버`, `하트`, `스페이드` 카드가 있다.
- [x] `카드 값`을 가진다.

### 카드 영역
- [x] 딜러 카드 영역과 참가자 카드 영역으로 나뉜다.
- [x] 카드들을 가진다.
- [x] 카드를 더 받을 수 있는 상태인지 확인한다.
- [x] 카드를 더 받을 것인지 결정한다.
- [x] 카드 현숫자를 계산한다.
- [X] `A` 는 1 또는 11로 계산될 수 있다.
- [x] 카드를 추가할 수 있다.
- [x] 처음에 카드 2장을 받아 생성된다.

### 카드 덱

- [x] 전체 카드를 가진다.
- [x] 카드를 한장씩 준다.

## refactoring

### 카드 테이블
- [x] `카드 영역`을 생성해준다.
- [x] `딜러`와 `참여자`의 승부 결과를 알려줄 수 있다.

### 점수
- [x] 21점을 초과하는지 확인한다.
- [x] 21점 미만인지 확인한다.
- [x] 점수끼리 대소비교를 할 수 있다.
- [x] 작거나 같다.
- [x] 크다.
- [x] 작다.

7 changes: 7 additions & 0 deletions src/main/java/BlackJackApplication.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
import controller.BlackJackController;

public class BlackJackApplication {
public static void main(String[] args) {
new BlackJackController().run();
}
}
124 changes: 124 additions & 0 deletions src/main/java/controller/BlackJackController.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,124 @@
package controller;

import domain.cardtable.CardTable;
import domain.deck.CardDeck;
import domain.player.Name;
import domain.player.Player;
import domain.player.dealer.Dealer;
import domain.player.dealer.DealerResult;
import domain.player.participant.Participant;
import domain.player.participant.ParticipantResult;
import view.InputView;
import view.OutputView;

import java.util.ArrayList;
import java.util.List;
import java.util.Map;
import java.util.stream.Collectors;

import static java.util.stream.Collectors.counting;

public class BlackJackController {

public void run() {

final CardDeck cardDeck = CardDeck.shuffledFullCardDeck();
final CardTable cardTable = CardTable.readyToPlayBlackjack(cardDeck);

final List<Participant> participants = createParticipants();
final Dealer dealer = createDealer();
final List<Player> players = createPlayers(participants, dealer);

deal(cardTable, players);

printStateAfterDeal(participants, dealer);
hittingPlayer(cardTable, participants, dealer);
printStateAfterHit(participants, dealer);
Comment on lines +32 to +36
Copy link

Choose a reason for hiding this comment

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

Suggested change
deal(cardTable, players);
printStateAfterDeal(participants, dealer);
hittingPlayer(cardTable, participants, dealer);
printStateAfterHit(participants, dealer);
deal(cardTable, players);
printStateAfterDeal(participants, dealer);
hittingPlayer(cardTable, participants, dealer);
printStateAfterHit(participants, dealer);

요렇게 개행을 잡으면 의미 단위로 묶을 수 있을 것 같아요.


final Map<Participant, ParticipantResult> participantsResult = cardTable.determineWinner(participants, dealer);
final Map<DealerResult, Long> scoreBoard = countDealerResult(participantsResult);

printPlayerScoreBoard(participants, participantsResult, scoreBoard);
}

private static List<Player> createPlayers(final List<Participant> participants, final Dealer dealer) {
List<Player> players = new ArrayList<>(participants);
players.add(dealer);

return players;
}

private static void deal(final CardTable cardTable, final List<Player> players) {
dealCard(cardTable, players);
dealCard(cardTable, players);
}

private static void dealCard(final CardTable cardTable, final List<Player> players) {
players.forEach(cardTable::dealCardTo);
}

private static Dealer createDealer() {
return new Dealer();
}

private static List<Participant> createParticipants() {
return InputView.readParticipantsName()
.stream()
.map(Name::new)
.map(Participant::new)
.collect(Collectors.toList());
}

private static void printPlayerScoreBoard(final List<Participant> participants,
final Map<Participant, ParticipantResult> playersResult,
final Map<DealerResult, Long> scoreBoard) {
OutputView.showDealerScoreBoard(scoreBoard);
OutputView.showParticipantsScoreBoard(playersResult, participants);
}

private static void printStateAfterHit(final List<Participant> participants, final Dealer dealer) {
OutputView.showPlayerStateResult(dealer);
OutputView.showParticipantsStateResult(participants);
}

private static void printStateAfterDeal(final List<Participant> participants, final Dealer dealer) {
OutputView.showDealtCardTo(participants);
OutputView.showStateOf(dealer);
OutputView.showStateOf(participants);
}

private static Map<DealerResult, Long> countDealerResult(
final Map<Participant, ParticipantResult> playersResult) {
return playersResult.keySet()
.stream()
.collect(Collectors.groupingBy(participant -> playersResult.get(participant)
.convertToDealerResult(),
counting()));
}

private void hittingPlayer(final CardTable cardTable, final List<Participant> participants, final Dealer dealer) {
hitForParticipants(cardTable, participants);
hitForDealer(cardTable, dealer);
}

private void hitForDealer(final CardTable cardTable, final Dealer dealer) {
do {
OutputView.dealerOneMoreCard();
} while (cardTable.dealCardTo(dealer));
}

private void hitForParticipants(final CardTable cardTable, final List<Participant> participants) {
participants.forEach(participant -> hitForParticipant(cardTable, participant));
}

private void hitForParticipant(final CardTable cardTable, final Participant participant) {
while (inputHitOrStay(participant)) {
cardTable.dealCardTo(participant);
OutputView.showStateOf(participant);
}
}

private boolean inputHitOrStay(final Participant participant) {
return InputView.readMoreCard(participant).equals("y");
}
}
69 changes: 69 additions & 0 deletions src/main/java/domain/Score.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,69 @@
package domain;

import java.util.Objects;

public class Score {

public static final Score MIN = new Score(0);

private static final Score UPPER_LIMIT_SCORE = new Score(21);

private static final Score REMAIN_SCORE_ACE = new Score(10);

private final int value;

public Score(final int value) {
this.value = value;
}

public boolean isBust() {
return this.isGreaterThan(UPPER_LIMIT_SCORE);
}

public boolean canMoreCard() {
return this.isLessThan(UPPER_LIMIT_SCORE);
}

public boolean isLessThan(final Score other) {
return value < other.value;
}

public boolean isLessEqualThan(final Score other) {
return value <= other.value;
}

public boolean isGreaterThan(final Score other) {
return value > other.value;
}

public Score plusTenIfNotBurst() {
final Score plusScore = this.plus(REMAIN_SCORE_ACE);

if (plusScore.isLessEqualThan(UPPER_LIMIT_SCORE)) {
return this.plus(REMAIN_SCORE_ACE);
}

return this;
}
Comment on lines +39 to +47
Copy link

Choose a reason for hiding this comment

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

값 객체 Score 구성 잘 해주셨네요 👍

soft vs hard라는 블랙잭 용어를 사용해도 좋을 것 같네요.


개인적으로 값 객체 내부에서는 원시타입을 사용하는 편이 좀 더 코드를 읽기 편하다는 생각이 들긴 하네요.


public Score plus(Score other) {
return new Score(value + other.value);
}

@Override
public boolean equals(final Object o) {
if (this == o) return true;
if (o == null || getClass() != o.getClass()) return false;
final Score score = (Score) o;
return value == score.value;
}

@Override
public int hashCode() {
return Objects.hash(value);
}

public int value() {
return value;
}
}
54 changes: 54 additions & 0 deletions src/main/java/domain/area/CardArea.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,54 @@
package domain.area;

import domain.Score;
import domain.card.Card;

import java.util.ArrayList;
import java.util.List;

public class CardArea {

private final List<Card> cards = new ArrayList<>();

public List<Card> cards() {
return new ArrayList<>(cards);
}

public void addCard(final Card card) {
cards.add(card);
}

public Score calculate() {
Score score = score();

if (hasAce()) {
score = score.plusTenIfNotBurst();
}

return score;
}

private Score score() {
return cards.stream()
.map(card -> new Score(card.cardValue().value()))
.reduce(Score.MIN, (Score::plus));

}

private boolean hasAce() {
return cards.stream()
.anyMatch(card -> card.cardValue().isAce());
}

public boolean canMoreCard() {
return calculate().canMoreCard();
}

public boolean isBust() {
return calculate().isBust();
}

public Card firstCard() {
return cards.get(0);
}
}
35 changes: 35 additions & 0 deletions src/main/java/domain/card/Card.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
package domain.card;

import java.util.Objects;

public class Card {

private final CardShape cardShape;
private final CardValue cardValue;

public Card(final CardShape cardShape, final CardValue cardValue) {
this.cardShape = cardShape;
this.cardValue = cardValue;
}

public CardShape cardShape() {
return cardShape;
}

public CardValue cardValue() {
return this.cardValue;
}

@Override
public boolean equals(final Object o) {
if (this == o) return true;
if (!(o instanceof Card)) return false;
final Card card = (Card) o;
return cardShape == card.cardShape && cardValue == card.cardValue;
}

@Override
public int hashCode() {
return Objects.hash(cardShape, cardValue);
}
}
9 changes: 9 additions & 0 deletions src/main/java/domain/card/CardShape.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
package domain.card;

public enum CardShape {
DIAMOND,
CLOVER,
HEART,
SPADE,
;
}
34 changes: 34 additions & 0 deletions src/main/java/domain/card/CardValue.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
package domain.card;

public enum CardValue {

ACE(1),

TWO(2),
THREE(3),
FOUR(4),
FIVE(5),
SIX(6),
SEVEN(7),
EIGHT(8),
NINE(9),
TEN(10),

KING(10),
QUEEN(10),
JACK(10);

private final int value;

CardValue(final int value) {
this.value = value;
}

public int value() {
return value;
}

public boolean isAce() {
return this == ACE;
}
}
Loading