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

simhokyung · 2026-03-15T12:11:19Z

PR을 main으로 잘못 날려서, 기존 PR을 닫고 다시 작성합니다! 😭

체크 리스트

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

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

1. 기존 `BlackjackGame.start()` 구조를 유지하기 위해 생성 정보 객체를 추가한 선택이 적절한가?

기존 구조에서는 BlackjackGame.start()에 플레이어 이름 목록(List<String>)을 전달하고,
BlackjackGame 내부에서 이를 바탕으로 Player를 생성하는 방식으로 구현되어 있었습니다.

이번 사이클에서 베팅 기능이 추가되면서, 더 이상 이름만으로는 Player를 생성할 수 없게 되었고,
이름과 함께 BettingMoney도 초기화해야 하는 상황이 생겼습니다.

이때 가장 먼저 고민했던 부분은, 기존의 구조를 최대한 유지하면서 확장하자는 것이었습니다.
기존처럼 BlackjackGame.start()가 게임 시작과 플레이어 생성을 함께 담당하는 구조를 유지하도록 설계하고 싶었고,
그래서 start()의 역할은 유지하되, 단순 이름 목록 대신 이름과 배팅 금액을 함께 담는 PlayerCreationInfo 객체를 만들고, 그 리스트를 넘겨주는 방식을 선택했습니다.

이 방식은 BlackjackGame.start() 가 여전히 게임 생성 책임을 가지도록 유지할 수 있고,
GameController가 도메인 객체 생성에 깊게 관여하지 않도록 할 수 있다는 점에서 괜찮다고 판단했습니다.

다만 PlayerCreationInfo는 실제 게임 진행 중에는 사용되지 않고 생성 시점에만 사용되기 때문에,
이 객체가 도메인적으로 의미가 있는 객체인지,
아니면 기존 구조를 유지하기 위해 만든 단순 포장 객체에 가까운지에 대해 고민이 있었습니다.

현재 방식이 적절한 선택인지,
혹은 플레이어 생성 책임을 다른 객체로 분리하는 것이 더 자연스러운지
제이미의 의견을 듣고 싶습니다.

2. `Hand`의 책임 분리

Score가 Hand를 만들기

Hand가 Score를 만들기

제 3자가 Score를 만들기
이렇게 적었어야 했는데 오타가 났네요.
제 3자가 Score를 만들고 있지만 결국 생성에 대한 행위는 Hand가 하고 있는 상황인데요.
뭔가, 3번을 의도했지만 어느정도 2번에 걸쳐있는 것 같아 책임 분리가 덜 되어보이기도 하는데, 의도된 부분인지 궁금했어요.

    public Score getScore() {
        return new Score(ScoreCalculator.calculate(List.copyOf(cards)));
    }

Score를 Hand의 상태로 보는 것이 자연스럽다고 생각해서 Hand.getScore()를 두었고, 다만 점수 계산 규칙까지 Hand가 모두 알 필요는 없다고 판단해 계산 과정은 ScoreCalculator에 위임했습니다. 그래서 현재는 Hand가 계산 결과를 받아 Score를 생성해 반환하는 구조로 두었습니다.

다만 리뷰 주신 내용을 보면서, 이 구조가 완전히 “제3자가 Score를 만든다”기보다는 Hand가 생성 책임까지 일부 가지고 있어서 2번과 3번 사이에 걸쳐 있는 형태로도 보일 수 있겠다고 느꼈습니다.

여기서 궁금한 부분이 있는데요,
ScoreCalculator가 int만 계산하고 Hand가 Score를 생성하는 현재 구조와, ScoreCalculator가 Score까지 반환하는 구조의 핵심 차이를 어떻게 보시는지 궁금합니다.
Score의 생성 위치가 다르긴 하지만, 사용하는 쪽에서는 결국 Hand.getScore()로 가져다 쓰게 되어서, 제가 느끼기엔 비슷한 방식이라고 생각되는데, 정답이 있는걸까요? 제이미의 의견이 궁금합니다.

이외에도 부족한 부분은 자유롭게 피드백 주시면 감사하겠습니다!

…sultDto 구현

- 딜러 카드 16이하 여부 확인 테스트 구현 - 카드 추가 확인 테스트 구현

- 플레이어의 카드 추가 여부 확인 테스트 - 플레이어의 버스트 여부 확인 테스트

- 모든 플레이어들의 버스트 여부 확인 테스트 작성

- 플레이어 이름 길이가 20자를 초과할 경우 예외 발생 테스트 작성 - Null 혹은 빈 값 입력 시 예외 발생 테스트

- draw하는 경우 테스트 추가

- DTO 생성 단계에서 공개할 카드 범위를 결정하고, 뷰는 전달받은 값만 출력하도록 하도록 수정

- Hand에 hasScoreLessThanOrEqual 메서드를 추가하여 Dealer의 checkThreshold 메서드의 반환값에서의 진입개수를 줄임

- 블랙잭 판단 테스트

- PlayerCreationInfo 클래스를 활용하는 구조로 변경

- Player: 생성자부분에 BettingMoney 추가 - Participant: getHand()메서드 접근제어자 변경(public -> protected로 변경)

- BlackjackGame.start()의 인자를 플레이어 이름과 배팅금액을 포함하고 있는 PlayerCreationInfo를 활용하는 구조로 변경

jamie9504

안녕하세요, 맥스. 리뷰어 제이미입니다.

코멘트 남겨두었습니다.

jamie9504 · 2026-03-15T17:27:18Z

src/main/java/controller/GameController.java

@@ -1,34 +1,69 @@
 package controller;


기존 BlackjackGame.start() 구조를 유지하기 위해 생성 정보 객체를 추가한 선택이 적절한가?

컨트롤러가 PlayerCreationInfo를 직접 만드는 것 자체도 생성에 관여하는 행위 아닐까요?
아래 ** 코멘트도 함께 확인해보세요.

Hand의 책임 분리

정답이 있다기보다는 트레이드오프의 문제예요.

사용하는 쪽에서 Hand.getScore()로 쓰는 건 동일하지만,
Score가 isBust() 같은 행위를 가진다면 누가 생성 책임을 갖는지가 의미 있어져요.
현재 구조에서 Hand가 new Score(...)를 직접 호출하고 있으니,
ScoreCalculator에 위임했다고 해도 생성 책임은 Hand에 남아있는 셈이에요.

결국 누가 생성 책임을 갖는지의 차이예요.

jamie9504 · 2026-03-15T17:28:00Z

src/main/java/controller/GameController.java

+        List<String> playerProfitResults = players.getPlayers().stream()
+                .map(player -> String.format("%s: %s%n",
+                        player.getNameValue(),
+                        ProfitCalculator.formatProfit(
+                                ProfitCalculator.calculatePlayerProfit(player, dealer)
+                        )))
+                .toList();


컨트롤러에서 String.format으로 직접 포맷팅하고 있어요.
출력 형식에 대한 책임은 OutputView가 가지는 것이 자연스럽지 않을까요?

jamie9504 · 2026-03-15T17:28:20Z

src/main/java/controller/GameController.java

+        List<PlayerCreationInfo> playerCreationInfos = new ArrayList<>();
+        for (String name : names) {
+            Integer money = InputView.readBettingMoney(name);
+            System.out.println();


System.out.println()이 컨트롤러에 직접 있네요.
출력은 OutputView에 위임하는 것이 어떨까요?

jamie9504 · 2026-03-15T17:28:33Z

src/main/java/view/InputView.java

+        System.out.printf("%s의 배팅 금액은?%n",name);
+        return Integer.parseInt(scanner.nextLine());


숫자가 아닌 값이 입력되면 어떻게 되나요?

jamie9504 · 2026-03-15T17:29:29Z

src/main/java/domain/ProfitCalculator.java

+public class ProfitCalculator {
+    private static final double BLACKJACK_PROFIT_RATE = 1.5;
+
+    public static double calculatePlayerProfit(Player player, Dealer dealer) {


함수(또는 메서드)의 길이가 10라인을 넘어가지 않도록 구현한다.

jamie9504 · 2026-03-15T17:48:45Z

src/test/java/domain/BettingMoneyTest.java

+public class BettingMoneyTest {
+
+    @Test
+    void 베팅금액을_생성할_수_있다(){
+        BettingMoney bettingMoney = BettingMoney.of(1000);
+        assertThat(bettingMoney.getMoney()).isEqualTo(1000);
+    }
+
+    @Test
+    void 베팅금액이_0이하면_예외가_발생한다(){
+        assertThatThrownBy(() -> BettingMoney.of(0))
+                .isInstanceOf(IllegalArgumentException.class);
+        assertThatThrownBy(() -> BettingMoney.of(-1000))
+                .isInstanceOf(IllegalArgumentException.class);
+    }
+}


만약 product code에서 해당 기준이 100 이하면 예외가 발생한다로 변경된다면,
이 테스트들로 무엇을 점검할 수 있나요?

jamie9504 · 2026-03-15T17:50:12Z

src/main/java/view/OutputView.java

+        System.out.println("\n\n## 최종 수익");
+        System.out.printf("딜러: %s%n", dealerProfit);


개행문자가 혼용되고 있네요.
각 개행문자별 특징을 적어주세요.

jamie9504 · 2026-03-15T17:52:06Z

src/test/java/domain/PlayersTest.java

    Player bustPlayer = Player.of(Name.from("pobi"), new Hand(new ArrayList<>(List.of(
-            Card.of(CardNumber.J, CardShape.CLOVER),
-            Card.of(CardNumber.K, CardShape.HEART),
-            Card.of(CardNumber.Q, CardShape.DIAMOND)))));
+                    Card.of(CardNumber.J, CardShape.CLOVER),
+                    Card.of(CardNumber.K, CardShape.HEART),
+                    Card.of(CardNumber.Q, CardShape.DIAMOND))))
+            , BettingMoney.of(1000)
+    );

    Player normalPlayer = Player.of(Name.from("jason"), new Hand(new ArrayList<>(List.of(
            Card.of(CardNumber.J, CardShape.CLOVER),
-            Card.of(CardNumber.Q, CardShape.DIAMOND)))));
+            Card.of(CardNumber.Q, CardShape.DIAMOND)))),
+            BettingMoney.of(2000));

    Players allBustPlayers = Players.from(List.of(bustPlayer, bustPlayer, bustPlayer));
    Players containBustPlayers = Players.from(List.of(bustPlayer, normalPlayer, normalPlayer));
    Players allNormalPlayers = Players.from(List.of(normalPlayer, normalPlayer, normalPlayer));


bustPlayer, normalPlayer 등이 클래스 필드로 선언되어 있네요.

@BeforeEach 등에서 선언하는 것과 어떠한 차이가 있을까요?

jamie9504 · 2026-03-15T17:53:17Z

src/test/java/domain/ProfitCalculatorTest.java

+
+import static org.assertj.core.api.Assertions.assertThat;
+
+public class ProfitCalculatorTest {


딜러와 플레이어가 모두 버스트인 경우는 없어도 될까요?

jamie9504 · 2026-03-15T17:54:26Z

src/main/java/domain/card/CardNumber.java

+package domain.card;

 public enum CardNumber {
    ACE("A", 1),


그러고보니 ACE는 풀네임으로, J, Q, K는 약어로 선언한 이유가 무엇인가요?
일관성 있게 통일한다면 어떤 기준으로 맞추는 게 좋을까요?

simhokyung and others added 30 commits March 5, 2026 12:52

docs: 블랙잭 미션 README 초안 작성

d32da74

docs(맥스): README 예외 처리 규칙 추가

ce21f2c

feat: InputView 사용자 이름 입력, 카드 추가 여부 구현

75962f0

test: 덱에 카드가 없는 경우 예외 처리 테스트 구현

d7a1d18

feat: Deck에서 카드를 뽑는 draw 메서드 구현

f70f371

feat: 참가자 이름을 파싱하는 NameParser 구현

88ff16f

feat: 카드숫자 1~10을 관리하는 Enum 구현

ee164cf

feat: 카드 모양을 나타내는 CardShape을 Enum으로 구현

4865776

feat: CardNumber(카드숫자)와 CardShape(카드모양)을 받아 관리하는 Card 구현

a9f6fb3

feat: 플레이어와 딜러가 손에 가지고있는 카드를 나타내는 도메인 구현

3af8be8

feat: 점수를 나타내는 Score 도메인 구현(원시값 포장)

b8311b7

feat: CardNumber 도메인에 Symbol 추가

f1fa65a

feat: 딜러와 플레이어가 받는 카드를 출력하는 OutputView 구현

6b70e46

feat: Dealer 도메인 구현

c7411c4

feat: 참가자를 나타내는 Player 도메인 구현

fc1a151

feat: Player 리스트를 모아놓은 Players 도메인 구현

9ca1e73

feat: GameController 뼈대 구성

9e59c62

test: 카드 점수계산 test 작성

442c944

feat: 카드 점수를 계산하는 ScoreCalculator 도메인 구현

df03b79

feat: View에 값 전달을 위한 DealerResultDto, PlayerDto, PlayersDto, PlayerRe…

fe60635

…sultDto 구현

feat: 파라미터 Dto로 변경, 카드 현황 출력 로직 및 점수 출력 로직 구현

c26fded

refactor: Scanner 중복 제거

896b284

test: Dealer 테스트 추가

75c7e6f

- 딜러 카드 16이하 여부 확인 테스트 구현 - 카드 추가 확인 테스트 구현

feat: Dealer의 카드 추가 로직 구현

26c5b86

test: 플레이어 테스트 작성

71ab035

- 플레이어의 카드 추가 여부 확인 테스트 - 플레이어의 버스트 여부 확인 테스트

feat: 플레이어의 카드 추가 로직 및 버스트 여부 확인 로직 추가

1cdc20b

test: Players 테스트 구현

9ab3080

- 모든 플레이어들의 버스트 여부 확인 테스트 작성

feat: 모든 플레이어들의 버스트 여부를 확인하는 로직 구현

f7b44e2

test: Name 클래스 테스트 작성

e95ef28

- 플레이어 이름 길이가 20자를 초과할 경우 예외 발생 테스트 작성 - Null 혹은 빈 값 입력 시 예외 발생 테스트

feat: 플레이어의 이름을 관리하는 Name 클래스 구현

a83d96c

simhokyung added 27 commits March 12, 2026 00:28

refactor: Deck 메서드 순서 변경

5ccee00

refactor: PlayersDto.from(players) 중복사용 제거

c7d3de8

refactor(test): Deck 테스트케이스 추가

d127436

- draw하는 경우 테스트 추가

refactor: getDealerCardStatus 로직 제거

b83635b

- DTO 생성 단계에서 공개할 카드 범위를 결정하고, 뷰는 전달받은 값만 출력하도록 하도록 수정

refactor: 전체 코드 줄맞춤(Ctrl+alt+l)

7875c22

refactor: 메서드 순서 리팩토링

a08777d

refactor: 디미터의 법칙 적용

e649ed8

- Hand에 hasScoreLessThanOrEqual 메서드를 추가하여 Dealer의 checkThreshold 메서드의 반환값에서의 진입개수를 줄임

refactor(all): 전체 코드에 디미터의 법칙 적용

5d50dc6

Merge remote-tracking branch 'woowacourse/simhokyung' into simhokyung

a0f1c13

test: BettingMoney 테스트 추가

35c13aa

feat: 배팅금액을 관리하는 BettingMoney 클래스 구현

238769e

feat: Hand클래스에 블랙잭을 판단하는 isBlackjack 메서드 추가

a88c4c6

test: Hand 테스트 추가

f743aa8

- 블랙잭 판단 테스트

refactor: 사용하지 않는 클래스 제거

ac97136

test: 플레이어 이름과 배팅금액을 가지고 있는 PlayerCreationInfo 테스트코드 작성

6d80700

test: 플레이어 이름과 배팅금액을 가지고 있는 PlayerCreationInfo 테스트코드 작성

edae525

refactor: 마지막 줄 개행문자 추가, 오타 수정

ade6ff6

test: 수익 계산을 테스트하는 코드 작성

91f0740

feat: 수익 계산을 담당하는 ProfitCalculator 클래스 작성

545e9ed

refactor: 배팅금액을 추가하여 전체적인 구조 변경

85c4481

- PlayerCreationInfo 클래스를 활용하는 구조로 변경

refactor(test): 배팅금액을 추가하여 전체적인 구조 변경

e34f9d1

- PlayerCreationInfo 클래스를 활용하는 구조로 변경

refactor: 배팅 금액 추가, 최종 수익 추가로 인한 구조 변경

34cc79d

refactor(test): 배팅 금액 추가로 인한 구조 변경

7b050bb

refactor: 배팅 금액 추가로 인한 구조 변경 및 접근제어자 변경

dca12aa

- Player: 생성자부분에 BettingMoney 추가 - Participant: getHand()메서드 접근제어자 변경(public -> protected로 변경)

refactor: 배팅금액이 추가되어 Controller 전체 구조 변경

0910024

- BlackjackGame.start()의 인자를 플레이어 이름과 배팅금액을 포함하고 있는 PlayerCreationInfo를 활용하는 구조로 변경

refactor: 안쓰는 클래스, 메서드 제거 / 패키지 분리

c7fb9c0

docs: README 기능구현 목록 추가

fa93618

jamie9504 assigned jamie9504 and simhokyung Mar 15, 2026

jamie9504 requested changes Mar 15, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[🚀 사이클2 - 미션 (블랙잭 베팅)] 맥스 미션 제출합니다. #1130

[🚀 사이클2 - 미션 (블랙잭 베팅)] 맥스 미션 제출합니다. #1130
simhokyung wants to merge 87 commits intowoowacourse:simhokyungfrom
simhokyung:step2

simhokyung commented Mar 15, 2026

Uh oh!

jamie9504 left a comment

Uh oh!

jamie9504 Mar 15, 2026

Uh oh!

jamie9504 Mar 15, 2026

Uh oh!

jamie9504 Mar 15, 2026

Uh oh!

jamie9504 Mar 15, 2026

Uh oh!

jamie9504 Mar 15, 2026

Uh oh!

jamie9504 Mar 15, 2026

Uh oh!

jamie9504 Mar 15, 2026

Uh oh!

jamie9504 Mar 15, 2026

Uh oh!

jamie9504 Mar 15, 2026

Uh oh!

jamie9504 Mar 15, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		System.out.printf("%s의 배팅 금액은?%n",name);
		return Integer.parseInt(scanner.nextLine());

		System.out.println("\n\n## 최종 수익");
		System.out.printf("딜러: %s%n", dealerProfit);


		import static org.assertj.core.api.Assertions.assertThat;

		public class ProfitCalculatorTest {

Conversation

simhokyung commented Mar 15, 2026

체크 리스트

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

1. 기존 BlackjackGame.start() 구조를 유지하기 위해 생성 정보 객체를 추가한 선택이 적절한가?

2. Hand의 책임 분리

Uh oh!

jamie9504 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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

1. 기존 `BlackjackGame.start()` 구조를 유지하기 위해 생성 정보 객체를 추가한 선택이 적절한가?

2. `Hand`의 책임 분리