[🚀 사이클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 삭제

2차 피드백 수정 요소

dto 테스트 삭제
Hand의 isBlackjack() 메서드 수정 및 calculateInitialScore, calculateInitialBaseScore 메서드 삭제
BlackjackGameManager의 필드(CardMachine, BlackjackJudge)값 의존성 주입으로 구현 -> BlackjackFactory로 이름 수정
BlackjackController가 BlackjackGameManager를 필드로 가지지 않고 메서드들의 파라미터로 두도록 수정

3차 피드백 수정 요소

dto 테스트 코드 삭제
BlackjackGameManager의 Participants 필드값을 불변으로 지정하고 생성자에서 값이 할당되도록 수정
BlackjackJudge의 승패판단 메서드에서 분기 코드 중복 제거
컨트롤러의 start 메서드 내에서 BlackjackGameManager를 생성하도록 수정
BlackjackGameManager를 생성하는 BlackjackGameManagerFactory 클래스 생성 및 도메인 패키지에 위치
Participants, Players의 playerIsBust 메서드 isPlayerBust로 메서드명 수정

…하도록 구현 수정

…드 삭제

…ialScore() 메서드 구현

…meManager 메서드명 변경

… cardMachine() 및 blackjackJudge() 메서드 구현

…로 두도록 수정

…ackGameManager를 인수로 넘기지 않도록 수정

…ackjackGameManager를 인수로 넘기도록 수정

…per() 사용하도록 수정

… private으로 변경

Gomding

안녕하세요 제이콥!
피드백 잘 반영해주셨네요~
추가로 코멘트 남겼으니 확인부탁드려요 :)

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

src/test/java/dto/BlackjackResultDtoTest.java

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

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

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

Gomding · 2026-03-16T13:29:50Z

src/main/java/controller/BlackjackController.java

의존이 있는것은 어쩔 수 없이 BlackjackGameManager 의 변경에 영향을 받을 수 밖에 없겠죠 :)
Controller 가 BlackjackGameManager 를 의존하는것은 당연한 의존이 아닐까요? 🤔
의존하지않고 블랙잭 게임을 하는게 가능할까요? 😃

Controller 를 새로 생성하는것이 자연스럽나요? BlackjackGameManager 를 새로 생성하는것이 자연스럽나요?
역할의 관점에서 고민해볼 수 있겠네요. 제이콥은 Controller 의 역할을 무엇으로 보고있나요?

++) start 의 인자로 받아야할 이유가 있을까요?
start 내부에서 BlackjackGameManager 를 생성하면 좋을 것 같아요 🤔
저는 BlackjackGameManager 를 Main 까지 노출할 필요가 없다고 생각해서요!

Gomding · 2026-03-16T13:31:22Z

src/main/java/factory/BlackjackFactory.java

+    public BlackjackGameManager blackjackGameManager() {
+        return new BlackjackGameManager(cardMachine(), blackjackJudge());
+    }
+
+    private CardMachine cardMachine() {
+        return new CardMachine();
+    }
+
+    private BlackjackJudge blackjackJudge() {
+        return new BlackjackJudge();


BlackjackControllerFactory 라는 이름을 변경하면서까지
여기에 도메인 객체를 생성하는 책임을 위치한 이유가 궁금해요!

꼭 Factory 객체는 하나만 존재해야할까요?
도메인 객체를 생성하는것은 domain 패키지에 넣지않은 이유가 있을까요?

Gomding · 2026-03-16T13:32:42Z

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


    private final CardMachine cardMachine;
+    private final BlackjackJudge blackjackJudge;
    private Participants participants;


생성자에서 초기화하지 않는게 final 이 아닌 이유와 이어질 것 같네요 :)
불변으로 만드는것을 항상 고려해보시면 좋겠어요!

Gomding · 2026-03-16T13:34:00Z

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

+import domain.participant.Dealer;
+import domain.participant.Player;
+
+public record BlackjackJudge() {


record 인 이유가 있을까요?
제이콥이 생각하는 record 는 어떤 객체인가요? 🤔

Gomding · 2026-03-16T13:40:08Z

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

+    public boolean playerIsBust(String name) {
+        return participants.playerIsBust(name);
    }


isPlayerBust 로 이름을 변경하면 좋을것 같아요!

…pants 필드 final 지정으로 생성자에서 값 할당되도록 수정

…명 수정

…ackjackJudge() 메서드 삭제

blackjackGameManager() 호출 제거

Gomding

안녕하세요 제이콥~
이제 코멘트 남길것이 거의 없네요
추가로 코멘트 남겼으니 확인부탁드려요!
다음 요청에는 머지할 것 같은데 추가로 궁금한 부분있으면 알려주세요 :)

Gomding · 2026-03-17T12:24:07Z

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

-    public BlackjackGameManager(CardMachine cardMachine, BlackjackJudge blackjackJudge) {
+    public BlackjackGameManager(CardMachine cardMachine, BlackjackJudge blackjackJudge, Participants participants) {


코멘트 드린 이유는 객체는 완전한 상태로 초기화 해주는게 좋다고 생각해요
일부 필드를 나중에 초기화 하는것은 처음 객체 생성 시 객체를 사용할 수 있는 상태인지 고민해보면 좋아요

물론 나중에 초기화하는 경우가 아예 없는것은 아닙니다! 😄

Gomding · 2026-03-17T12:26:05Z

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

+        if (player.calculateScore() == dealer.calculateScore()) {
+            return Result.DRAW;


딜러가 블랙잭 이고 플레이어가 일반 21점 일 때 무승부로 판단될 수 있겠네요!

Gomding · 2026-03-17T12:27:49Z

src/main/java/domain/participant/BetAmount.java

+    public static final String INVALID_BET_AMOUNT_POSITIVE = "배팅 금액은 양수여야 합니다.";
+    private static final String NUMBER_FORMAT = "-?\\d+";
+
+    private final int betAmount;


베팅 금액에 대해서 int, long BigDecimal, BigInteger 등의 타입도 고민해보셨을까요?

Gomding · 2026-03-17T12:29:21Z

src/main/java/domain/participant/Players.java

    private final List<Player> players;

-    public Players(List<String> names) {
+    public Players(List<PlayerName> names, List<BetAmount> betAmounts) {


입력받은 names 와 betAmounts 의 size 에 대한 검증도 있어야겠군요 🤔

Gomding · 2026-03-17T12:31:46Z

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

+        dealer.addCard(cardSupplier.get());
+        dealer.addCard(cardSupplier.get());


dealer 에게 2번 요청하지않고
카드를 2장 뽑으라고 한 번만 요청할 수 있겠네요 😄

kang-hyungu

제이콥 몇 가지 추가 피드백 남겼으니 참고하세요
궁금한 점 있으면 질문주세요~

kang-hyungu · 2026-03-18T01:11:21Z

src/main/java/controller/factory/BlackjackControllerFactory.java

 import view.InputView;
 import view.OutputView;

 public class BlackjackControllerFactory {


팩토리 클래스를 따로 둔 이유가 뭐죠?

kang-hyungu · 2026-03-18T01:11:38Z

src/main/java/controller/BlackjackController.java

-    private final BlackjackGameManager blackjackGameManager;

-    public BlackjackController(InputView inputView, OutputView outputView, BlackjackGameManager blackjackGameManager) {
+    public BlackjackController(InputView inputView, OutputView outputView) {


왜 클래스명을 컨트롤러라 이름 붙였죠?

kang-hyungu · 2026-03-18T01:13:01Z

src/main/java/domain/card/CardMachine.java

    }

-    private void addRepeatSix(List<Card> decks, Rank rank, Suit suit) {
+    private void addRepeatDeckCount(List<Card> decks, Rank rank, Suit suit) {


왜 함수명에 add를 붙이셨나요?

kang-hyungu · 2026-03-18T01:13:45Z

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

    }
+
+    public boolean isBlackjack() {
+        return calculateTotalScore() == BLACKJACK_SCORE && cards.size() == 2;


2가 무엇을 의미하나요? 상수로 추출해서 의미 있는 이름을 부여하세요

kang-hyungu · 2026-03-18T01:14:06Z

src/main/java/domain/factory/BlackjackGameManagerFactory.java

+import domain.participant.PlayerName;
+import java.util.List;
+
+public class BlackjackGameManagerFactory {


이 클래스는 왜 만든건가요?

kang-hyungu · 2026-03-18T01:16:20Z

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

+
+public final class BlackjackJudge {
+
+    public Result judgePlayerResult(Dealer dealer, Player player) {


if절이 너무 많고 함수가 너무 길어 코드 파악하기가 어렵네요
다른 팀원들도 이 코드를 보고 로직 파악하기가 쉽지 않을텐데요, 함수가 10줄 넘어가지 않도록 개선해보세요

kang-hyungu · 2026-03-18T01:16:28Z

src/main/java/domain/game/Result.java

@@ -0,0 +1,19 @@
+package domain.game;
+
+public enum Result {


무슨 결과에요?

kang-hyungu · 2026-03-18T01:17:23Z

src/main/java/domain/participant/BetAmount.java

+
+    private final int betAmount;
+
+    public BetAmount(String betAmountInput) {


변수명에 input을 붙일 필요가 있나요?

kang-hyungu · 2026-03-18T01:18:44Z

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

+import java.util.List;
+import java.util.function.Supplier;
+
 public record Participants(Dealer dealer, Players players) {


클래스명은 참여자들인데 이름만 봐서는 무슨 일을 처리하는 클래스인지 파악이 어렵네요
그리고 record로 만든 이유는 뭔가요?

kang-hyungu · 2026-03-18T01:19:46Z

src/main/java/dto/PlayerStatisticDto.java

-import constant.Result;
 import domain.participant.Player;

 public record PlayerStatisticDto(


dto가 뭔가요? dto가 필요한가요?

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 12 commits March 15, 2026 23:50

remove(ParticipantDtoTest): 클래스 삭제

f39628f

remove(PlayerStatisticDtoTest): 클래스 삭제

982f824

feat(BlackjackControllerFactory): BlackjackGameManager 생성시 의존성 주입을 위한…

81439be

… cardMachine() 및 blackjackJudge() 메서드 구현

feat(BlackjackGameManager): BlackjackGameManager 생성자 의존성 주입으로 구현

3d8a086

refactor(BlackjackController): BlackjackGameManager를 필드가 아닌 메서드의 파라미터…

7e5ccba

…로 두도록 수정

refactor(BlackjackControllerFactory): BlackjackController 생성자에 Blackj…

bb5ab4e

…ackGameManager를 인수로 넘기지 않도록 수정

refactor(BlackjackApplication): BlackjackController의 start 메서드 실행시 Bl…

3f672b5

…ackjackGameManager를 인수로 넘기도록 수정

test(BlackjackGameManagerTest): FixedDeckBlackjackGameManager 생성자에 su…

f2866f9

…per() 사용하도록 수정

rename(BlackjackFactory): 클래스 이름 변경

51a5194

rename(BlackjackApplication): BlackjackFactory 클래스 이름 변경으로 인한 코드 수정

dd752c0

refactor(Hand): 불필요한 상수 제거

002b4eb

refactor(BlackjackFactory): cardMachine(), blackjackJudge() 메서드 접근제어자…

9abd3c3

… private으로 변경

Gomding requested changes Mar 16, 2026

View reviewed changes

khcho96 added 15 commits March 17, 2026 06:50

refactor(BlackjackGameManager): createParticipants() 메서드 식제 및 Partici…

964f880

…pants 필드 final 지정으로 생성자에서 값 할당되도록 수정

refactor(BlackjackGameManager): playerIsBust() -> isPlayerBust()로 메서드…

334100c

…명 수정

refactor(Participants): playerIsBust() -> isPlayerBust()로 메서드명 수정

f50fc45

refactor(Players): playerIsBust() -> isPlayerBust()로 메서드명 수정

6e5357a

test(PlayersTest): playerIsBust() -> isPlayerBust()로 호출 메서드명 수정

398d72a

refactor(BlackjackJudge): 중복 분기 코드 제거

ea33f47

refactor(BlackjackFactory): blackjackGameManager(), cardMachine(), bl…

bcbf301

…ackjackJudge() 메서드 삭제

refactor(BlackjackController): BlackjackGameManager를 내부에서 생성하도록 변경

8baa491

refactor(BlackjackApplication):

5d23921

blackjackGameManager() 호출 제거

test(PlayersTest): PlayerIsBust -> IsPlayerTest로 수정

50180e1

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

40f514d

refactor(BlackjackJudge): final 클래스로 지정

4a7e05e

refactor(BlackjackGameManager): drawCard() 메서드 private으로 수정

dd887a1

refactor(OutputView): printHand() 메서드 private으로 수정

2d2aa4f

refactor(OutputView): printHand() 메서드 DELIMITER 사용 부분 수정

950c53d

Gomding requested changes Mar 17, 2026

View reviewed changes

kang-hyungu requested changes Mar 18, 2026

View reviewed changes

		public BlackjackGameManager(CardMachine cardMachine, BlackjackJudge blackjackJudge) {
		public BlackjackGameManager(CardMachine cardMachine, BlackjackJudge blackjackJudge, Participants participants) {

		if (player.calculateScore() == dealer.calculateScore()) {
		return Result.DRAW;

		dealer.addCard(cardSupplier.get());
		dealer.addCard(cardSupplier.get());


		public final class BlackjackJudge {

		public Result judgePlayerResult(Dealer dealer, Player player) {


		private final int betAmount;

		public BetAmount(String betAmountInput) {

Conversation

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

Uh oh!

체크 리스트

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

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

질문

1차 피드백 수정 요소

2차 피드백 수정 요소

3차 피드백 수정 요소

Uh oh!

Gomding left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

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!

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!

kang-hyungu 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

khcho96 commented Mar 14, 2026 •

edited

Loading