[🚀 사이클2 - 미션 (블랙잭 베팅)] 제이콥 미션 제출합니다. by khcho96 · Pull Request #1119 · woowacourse/java-blackjack

khcho96 · 2026-03-14T04:53:47Z

체크 리스트

미션의 필수 요구사항을 모두 구현했나요?
Gradle test를 실행했을 때, 모든 테스트가 정상적으로 통과했나요?
애플리케이션이 정상적으로 실행되나요?

어떤 부분에 집중하여 리뷰해야 할까요?

사이클1 마지막 코멘트에서 반영한 내용

말씀주신대로 Participants가 단순히 Dealer와 Players를 꺼내기만 하는 것이 아니라 Dealer와 Players에게 명령을 수행하거나 다루는 방향으로 책임을 부여해봤습니다.
Result와 HitOrStand 클래스를 domain으로 옮겼습니다.
Hand에 10이라는 매직넘버를 상수화했습니다.
Hand에 점수계산 로직을 메서드로 추출했습니다.

질문

의도된 중복에 대한 질문

중복을 제거하라 하지만 떄로는 의도된 중복을 만들기도 합니다 😄

의도된 중복을 만들기도 한다는 사실은 충격적이네요 😮
"중복"에 대해 100% 부정적으로 바라봤었는데, 그렇지 않을 수도 있다는 사실을 인지하고 유연하게 생각해봐야겠습니다!
한 가지 궁금한 점이 있는데, 찰리는 constant 패키지를 사용하시는 편인가요?
만약 사용하신다면 어떤 경우에, 어떤 클래스를 공통으로 관리하는 constant 패키지로 분리하시나요?

책임 분리에 대한 질문

BlackjackGameManager 가 승패에 대한 판단을 하는것은 책임이 과할것 같네요 🤔 승패 판단을 하는 객체를 분리해보는것도 좋겠어요~

제가 생각했을 때 현재 승무패 판단을 하는 메서드는 BlackjackGameManager 클래스의 getBlackjackStatistics 메서드 로직인데, 특히 해당 메서드 내 아래 3줄의 로직인 듯 합니다.

Result result = judgePlayerResult(participants.dealer(), player);
int playerProfit = calculatePlayerProfit(player, result);
dealerProfit += playerProfit * -1;

이 로직을 가지는 객체(A라고 지칭하겠습니다)를 따로 만든다면 A는 judgePlayerResult 메서드와 calculatePlayerProfit 메서드를 클래스 내에서 가질텐데, 이때 보시는 것처럼 메서드의 인수로 Dealer와 Player가 들어가기때문에 A가 Dealer와 Player에게 의존하게 되고, 결합도가 커지게 되지 않을까라는 부분이 염려됩니다. 결합도가 커지면 수정범위가 커지게 되는 단점이 있다고 생각합니다.

이 부분에 대한 찰리의 의견이 궁금합니다!

원시값 포장에 대한 질문

Rank 클래스에서 score의 원시값 포장에 대해 말씀해주셨는데, 단순히 상수를 저장하고 있고 블랙잭 점수 규칙에 대한 내용은 다른 객체에서 메서드로 구현했기때문에 score에서 어떤 추가적인 규칙을 검증하거나 다룰지 아이디어가 떠오르지 않네요😂
혹시 찰리 입장에서 score를 원시값 포장한다면 어떤식으로 구현하실지 궁금합니다!

1차 피드백 수정 요소

딜러가 블랙잭인 경우 고려해서 승패 로직 수정 구현
Participant에 getName 메서드 추상화
베팅금액을 다 입력받은 후 플레이어를 생성하도록 수정 -> Player의 BetAmount이 참조값을 null로 가지지 않도록 안정성을 높여줌
PlayerName도 컨트롤러에서 생성(검증 포함)해서 넘기도록 수정
BetAmount의 isPositive 메서드 isNotPositive로 메서드명 수정
플레이어가 카드를 뽑아 버스트가 되면 다음 차례로 넘어가도록 수정
Hand의 calculateAceCount 수정
OutputView 안쓰는 메서드 삭제
결과에 대한 배당을 BlackjackGameManager의 calculatePlayerProfit 메서드 삭제 및 해당 로직에 대한 책임을 Result가 가지도록 수정
승패 및 수익 계산 로직 객체 분리 및 승패판단 로직 수정 -> BlackjackJudge

 1. 플레이어가 버스트 -> 플레이어 LOSE
 2. 플레이어가 블랙잭(전제: 플레이어는 버스트 아님) && 딜러가 블랙잭이면 -> DRAW
 3. 플레이어가 블랙잭(전제: 플레이어는 버스트 아님) && 딜러가 블랙잭아니면 -> 플레이어 BLACKJACK_WIN
 4. 딜러가 버스트(전제: 플레이어는 버스트 또는 블랙잭 아님) -> 플레이어 WIN
 5. 플레이어 점수 > 딜러 점수(전제: 플레이어와 딜러는 버스트 또는 블랙잭 아님) -> 플레이어 WIN
 6. 플레이어 점수 == 딜러 점수(전제: 플레이어와 딜러는 버스트 또는 블랙잭 아님) -> DRAW
 7. 1~6 아님 -> 플레이어 LOSE

record 클래스는 equals, hashCode 메서드 자동 구현이므로 Card의 equals 삭제

…하도록 구현 수정

…드 삭제

…ialScore() 메서드 구현

…meManager 메서드명 변경

…서드 및 calculatePlayerProfit()를 BlackjackJudge로 이동

…ult(), calculatePlayerProfit()를 BlackjackJudge로 이동

…ult가 책임지도록 수정

…메서드 구현

…어가도록 수정 구현

Gomding

안녕하세요 제이콥!
전체적으로 잘 수정해주셨네요 :)
추가로 코멘트 남겼으니 확인부탁드려요!

궁금한 점 있으면 언제든 DM 이나 코멘트 남겨주세요~

Gomding · 2026-03-15T13:40:49Z

src/main/java/domain/card/Hand.java

    }
+
+    public boolean isBlackjack() {
+        return calculateInitialScore() == BLACKJACK_SCORE;


사실 Blackjack 이라는 점수는 조금 특이한 규칙을 가지고 있습니다
단순 21점과 Blackjack 이라는 점수 상태는 손패의 상태가 다릅니다

블랙잭은 패가 2장 + 21점일 때 를 블랙잭이란 상태로 보고있어요~

현재 calculateInitialScore()가 처음 받은 2장이 21일 때를 Blackjack으로 보고 있는데, 그냥 calculateTotalScore() == BLACKJACK_SCORE이면서 cards.size() == 2인 것을 보는게 정확하겠군요!

해당부분 위와 같이 수정했습니다!

Gomding · 2026-03-15T13:43:47Z

src/test/java/dto/BlackjackResultDtoTest.java

+import java.util.List;
+import org.junit.jupiter.api.Test;
+
+class BlackjackResultDtoTest {


Dto 는 테스트 대상이 맞을지 고민해보면 좋겠어요!
테스트를 작성하는 이유와 비슷할 것 같네요 :)

저는 테스트가 있으면 좋긴하지만 DTO 는 단순 값을 담는 객체이니 꼭 필요한가? 라는 입장이긴 합니다 😄
그 시간에 다른 중요한 클래스의 테스트 작성에 시간을 더 사용하는게 좋다고 생각해서요!

토론 시간에 정한 규칙이 "getter, setter, 그리고 단순 메서드 호출만 하는 메서드를 제외한 public 메서드는 모두 테스트한다"여서 Dto 또한 테스트 코드를 작성했습니다.

근데 작성하면서도 "이걸 해야하나.." 싶었는데, 명확히 짚어주셔서 더욱 테스트를 안해도 되겠다는 확신이 생기네요😂

Dto 테스트는 삭제했습니다!

Gomding · 2026-03-15T13:46:12Z

src/main/java/domain/game/BlackjackJudge.java

+        if (player.isBlackjack() && dealer.isBlackjack()) {
+            return Result.DRAW;
+        }
+        if (player.isBlackjack()) {
+            return Result.BLACKJACK_WIN;
+        }


이렇게 하는것도 가능할 것 같네요 🤔

Suggested change

if (player.isBlackjack() && dealer.isBlackjack()) {

return Result.DRAW;

}

if (player.isBlackjack()) {

return Result.BLACKJACK_WIN;

}

if (player.isBlackjack()) {

if (dealer.isBlackjack()) {

return Result.DRAW;

}

return Result.BLACKJACK_WIN;

}

맞습니다! 근데 이번 미션 프로그래밍 요구사항에서 indent를 1까지만 허용한다는 규칙을 지키기 위해 이렇게 구현했습니다.

사실 indent가 2정도여도 충분히 가독성있고 깔끔한 코드라고 저는 생각하는데.. 프리코스 미션에서도 indent는 2까지 허용이었구요!
indent 1을 유지하려다보니 오히려 더 가독성이 없어지는 코드가 나오는 느낌입니다..😂

찰리는 이 요구사항에 대해 어떻게 생각하시는지 궁금합니다!

Gomding · 2026-03-15T13:49:52Z

src/main/java/domain/game/BlackjackGameManager.java

    public BlackjackGameManager() {
        this.cardMachine = new CardMachine();
+        this.blackjackJudge = new BlackjackJudge();
    }


필요한 의존을 생성자 인자로 받는것이 더 유연한 구조라고 생각하는데
이미 구현하신 BlackjackControllerFactory 라는것도 있으니 BlackjackGameManager 도 고민해볼 수 있겠네요

Gomding · 2026-03-15T13:51:08Z

src/main/java/domain/game/BlackjackGameManager.java

+    public void createParticipants(List<PlayerName> playerNames, List<BetAmount> betAmounts) {
+        participants = Participants.of(playerNames, betAmounts);


생성자에서 플레이어들을 받지않고 별도로 생성하는것도 고민해보면 좋겠어요~

Gomding · 2026-03-15T13:51:54Z

src/main/java/controller/BlackjackController.java


    private final InputView inputView;
    private final OutputView outputView;
    private final BlackjackGameManager blackjackGameManager;


이부분도 사이클1에서 피드백을 못드렸는데
Controller 가 BlackjackGameManager 라는것을 상태로 가져도 괜찮을까요?
Controller 는 재사용 가능한게 좋다고 생각해요
제이콥은 어떻게 생각하시나요?

Gomding · 2026-03-15T13:54:37Z

src/main/java/domain/participant/Participants.java

+    public void drawInitialCards(Supplier<Card> cardSupplier) {
+        dealer.addCard(cardSupplier.get());
+        dealer.addCard(cardSupplier.get());
+        players.drawInitialCards(cardSupplier);
+    }
+
+    public void drawCardsByDealer(Supplier<Card> cardSupplier) {
+        dealer.addCard(cardSupplier.get());
+    }
+
+    public Player drawCardsByPlayer(String name, Supplier<Card> cardSupplier) {
+        players.addCard(name, cardSupplier.get());
+        return players.getPlayer(name);
+    }


테스트 가능한 구조로 만들어주신점 좋네요! 👍 👍 👍

+) 테스트의 어려움이라면 아래 글도 학습해보시면 좋겠어요~

https://tecoble.techcourse.co.kr/post/2020-05-17-appropriate_method_for_test_by_interface/

Gomding · 2026-03-15T13:56:04Z

src/main/java/domain/card/Hand.java

+    private Integer calculateBaseScore() {
+        return cards.stream()
+                .map(Card::getScore)
+                .reduce(0, Integer::sum);
+    }
+
+    private Integer calculateInitialBaseScore() {
+        return cards.subList(0, INITIAL_CARD_COUNT).stream()
+                .map(Card::getScore)
+                .reduce(0, Integer::sum);
+    }


코드에 중복되는 부분이 있어서 중복되는 부분만 private 메서드로 추출해볼 수 있겠네요

위의 피드백 적용 과정에서 calculateInitialBaseScore 메서드가 삭제되었습니다!

Gomding · 2026-03-15T13:59:14Z

README.md

기준이 명확하네요 👍 👍 👍
전체적으로 동의합니다!

khcho96 added 30 commits March 13, 2026 08:45

refactor(Hand): 메서드명 변경 및 메서드 추출

87b8cad

refactor(HitOrStand): domain.game 패키지로 이동

1b1c966

refactor(Result): domain.game 패키지로 이동

a7aa54b

refactor(Hand): REDUCED_SCORE_FROM_ACE 상수 처리

a6539dc

refactor(BlackjackGameManager): Dealer를 가져오지 않고 Participants의 메서드를 호출…

caba72c

…하도록 구현 수정

refactor(BlackjackGameManager): generatePlayerDtoList() 메서드 내용 스트림으로 구현

7035b92

refactor(BlackjackGameManager): getBlackjackStatistics() 메서드 수정

640c510

test(BlackjackGameManager): calculatePlayerResults() 메서드 삭제로 인한 테스트 코…

28f5d22

…드 삭제

docs(README): 배팅 기능 추가로 인한 기능 구현 목록 추가 작성

51c80be

feat(BlackjackController): inputBetAmounts 메서드 구현

b78f443

feat(BetAmount): 배팅 금액을 저장 및 검증하는 BetAmount 틀래스 생성

7f3f851

feat(BlackjackController): inputBetAmounts 메서드 구현

c3004ea

feat(BlackjackGameManager): getPlayerDtoList 및 setBetAmount 메서드 구현

7431a4e

feat(InputView): inputBetAmount 메서드 구현

7c75ff3

feat(ParticipantDto): from 메서드 구현

59130bb

feat(Player): setBetAmount 메서드 구현

eb50b41

feat(DealerStatisticDto): DealerStatisticDto 클래스 삭제

cd807d3

feat(BlackjackStatisticsDto): 필드 수정

841c634

feat(Hand): calculateInitialBaseScore(), isBlackjack(), calculateInit…

fbf79e0

…ialScore() 메서드 구현

feat(Hand): getBlackjackStatistics(), calculatePlayerProfit() 메서드 구현

1b369ed

feat(OutputView): 최종 수익 출력 기능 구현

6c2486f

feat(Player): isBlack(), getBetAmount() 메서드 구현

91e304c

feat(PlayerStatisticDto): 필드 수정

0f4bfaf

docs(README): 기능 구현한 항목 체크 표시

3e0ec19

refactor(CardMachine): addRepeatDeckCount 메서드명 변경

7872210

refactor(Hand): 메서드 위치 변경

e06ffda

refactor(Rank): 메서드 위치 변경

927179b

refactor(BlackjackGameManager): getPlayerDtoList() 메서드 삭제

0990745

refactor(ParticipantDto): Players를 인수로 가지는 from 메서드 1개 삭제

83da4c2

refactor(BlackjackControllerFactory): blackjackService -> blackjackGa…

fc36d0f

…meManager 메서드명 변경

khcho96 added 29 commits March 14, 2026 22:58

refactor(Hand): calculateAceCount 메서드 수정

a1c6f29

refactor(OutputView): 사용하지 않는 printResult 메서드 삭제

2757c76

refactor(BlackjackJudge): BlackjackGameManager의 judgePlayerResult() 메…

eee034c

…서드 및 calculatePlayerProfit()를 BlackjackJudge로 이동

refactor(BlackjackGameManager): BlackjackJudge 필드 추가 및 judgePlayerRes…

14cfcc1

…ult(), calculatePlayerProfit()를 BlackjackJudge로 이동

refactor(Result): BlackjackJudge의 calculatePlayerProfit() 메서드 내용을 Res…

afb1761

…ult가 책임지도록 수정

refactor(BlackjackJudge): 플레이어가 블랙잭이 아닌 점수로 이겼을 경우 WIN을 반환하도록 수정

0df7db2

refactor(BlackjackJudge): calculatePlayerProfit 메서드 삭제

529ca7c

refactor(BetAmount): 상수 public으로 변경

cd917e1

refactor(BlackjackController): 플레이어가 버스트이면 더이상 카드를 뽑지 못하도록 수정 구현

ae02ed2

refactor(BlackjackGameManager): isHit, isStand 메서드 삭제 및 playerIsBust …

1bb8255

…메서드 구현

refactor(Participants): playerIsBust 메서드 구현

4c28283

refactor(Players): playerIsBust 메서드 구현

3da7d09

refactor(Players): 플레이어가 카드를 추가로 뽑았을 때 버스트이면 카드를 뽑을지 물어보지 않고 다음 차례로 넘…

7650a5a

…어가도록 수정 구현

refactor(BlackjackController): drawCardOnPlayer 메서드 hitOrStand 파라미터 제거

38c446b

refactor(Card): equals 메서드 삭제

ecaf295

refactor(BlackjackJudge): 승패 판단 메서드 수정

8549a01

fix(Result): 실수 처리 버그 수정

22d2ee1

refactor(BetAmount): BlackjackException 사용하도록 수정

09a2d3d

test(HandTest): 테스트 코드 작성

522fad1

test(BlackjackGameManagerTest): 테스트 코드 작성

f712103

test(BlackjackJudgeTest): 테스트 코드 작성

3dc17b8

remove(HitOrStandTest): constant 패키지 및 하위 클래스 HitOrStandTest 삭제

2578a5f

test(ResultTest): 테스트 코드 작성

e244b53

refactor(PlayerName): equals 및 hashCode 메서드 삭제

f961d98

test(ParticipantsTest): 테스트 코드 작성

db6bbc3

test(PlayersTest): 테스트 코드 작성

f9c39d1

remove(PlayerTest): 테스트 코드 작성 불필요하므로 삭제

b252768

test(ParticipantDtoTest): 테스트 코드 작성

133cb90

test(PlayerStatisticDtoTest): 테스트 코드 작성

55e5d33

Gomding requested changes Mar 15, 2026

View reviewed changes

		public void createParticipants(List<PlayerName> playerNames, List<BetAmount> betAmounts) {
		participants = Participants.of(playerNames, betAmounts);

Conversation

khcho96 commented Mar 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

체크 리스트

어떤 부분에 집중하여 리뷰해야 할까요?

사이클1 마지막 코멘트에서 반영한 내용

질문

1차 피드백 수정 요소

Uh oh!

Gomding left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

khcho96 commented Mar 14, 2026 •

edited

Loading