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

Chocoding1 · 2026-03-13T05:46:13Z

체크 리스트

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

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

TDD

TDD를 시도해 보려고 하니, 처음 테스트 파일의 이름을 정하는 것부터 막히는 경험을 했습니다. TDD를 위해 가장 단순한 로직을 먼저 테스트하기 위해서는 그 로직이 어떤 객체에 존재해야 하는지 먼저 정해야 하는 건가요? 만약 그렇다면 TDD로 개발을 시작하기 전에 전체 서비스의 설계를 완벽하게 해놓아야 하는지 궁금합니다.

초기 설계

초기에 설계하실 때 어떤 흐름으로 어떤 객체들이 필요할지 정하시는지 궁금합니다.

리팩토링

이번에 배팅 수익금 계산 기능을 추가하는 과정에서 많은 리팩토링이 필요했습니다. 한 번에 많은 클래스들이 꼬여있다 보니 어디서부터 풀어나가야 할지 감을 못 잡았는데요, 이렇게 복잡하게 꼬인 코드들을 풀 때는 어떤 방식으로 접근하는 것이 좋을까요?

pci2676

전체적으로 리뷰 반영이 깔끔하시네요

dm으로 이야기를 많이해서 그런지 코드에서 고민의 흔적이 잘 보이는것 같네요 👍

도전해보면 좋을 부분에 리뷰를 남겨두었으니 확인해주세요

src/main/java/blackjack/model/card/HitCommand.java

pci2676 · 2026-03-16T14:35:40Z

src/main/java/blackjack/model/user/User.java

+    public GameResult judge(User user) {
+        if (this.isBust() || user.isBust()) {
+            return judgeByBust();
+        }
+
+        if (this.isBlackjack() || user.isBlackjack()) {
+            return judgeByBlackjack(user);
+        }
+
+        return judgeByScore(user);
+    }


이 부분 로직이 아무래도 복잡하다 보니 private 메서드 호출을 많이 하고있는데요

상태패턴을 이용해서 hand를 표현해보면 어떨까요? 잘 해본다면 조금 더 간결하게 표현해볼 수 있을것 같아요

pci2676 · 2026-03-16T14:38:29Z

src/main/java/blackjack/BlackjackGame.java

+        cardProvider.provideInitCards(users);
+    }
+
+    public void hitPlayers(List<User> users, Function<User, String> readHitCommand,


굳이 반영은 하시지 않아도 되는데요

위와 같은 부분에서 매개변수가 많이 전달되고있는데 매개변수의 수를 줄이도록 별도의 콜백 전달 객체를 고려해보시는것도 좋을것 같아요.

각 콜백들이 어떤 유기적 상호관계를 갖게되는지 시그니처만 보면 잘 파악이 되지 않아서요

그리고 Users를 넘기고 내부에서 players를 필터링하게한다면 더 안전하지 않을까요?

kang-hyungu

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

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

src/main/java/blackjack/controller/BlackjackController.java

-import java.util.List;


 public class BlackjackController {


컨트롤러가 뭐죠? 왜 이름이 컨트롤러죠?

kang-hyungu · 2026-03-18T01:22:34Z

src/main/java/blackjack/model/bet/BetAmount.java

+
+    static final String ERROR_EMPTY_INPUT = "입력값은 공백일 수 없습니다.";
+    static final String ERROR_BET_AMOUNT_NOT_INTEGER = "배팅 금액은 숫자 형태로 입력해야 합니다 : %s";
+    static final String ERROR_BET_AMOUNT_NOT_POSITIVE = "배팅 금액은 1원 이상이어야 합니다 : %d";


게임 규칙이 바껴서 배팅 금액을 1원이 아닌 10원 이상으로 바꿔야한다면?
텍스트로만 유지하면 수정 사항이 생길 때마다 모든 문자열을 직접 찾아서 바꿔야겠군요?

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

src/main/java/blackjack/model/card/CardProvider.java

 import java.util.Queue;
+import java.util.stream.Collectors;

 public class CardProvider {


provider라고 이름 지은 이유가 뭔가요?

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

src/main/java/blackjack/model/card/CardProvider.java

 public class CardProvider {

+    private static final int INIT_CARDS_START_IDX = 0;
+    private static final int INIT_CARDS_END_IDX = 2;


idx 약어 금지

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

src/main/java/blackjack/model/card/CardProvider.java

-            }
-        }
+        List<Card> cards = Arrays.stream(Rank.values())
+                .flatMap(rank -> Arrays.stream(Suit.values())


stream 안에서 또 stream을 처리하면 코드가 복잡해집니다
메서드로 추출해서 코드 읽기 좋게 바꿔보세요

kang-hyungu · 2026-03-18T01:25:51Z

src/main/java/blackjack/model/gameresult/GameResult.java


+import blackjack.model.bet.BetAmount;
+
 public enum GameResult {


적절한 클래스명인가요?
이 클래스가 맡은 일은 무엇인가요?

kang-hyungu · 2026-03-18T01:27:42Z

src/main/java/blackjack/model/user/Player.java

+    }
+
+    @Override
+    public boolean isPlayer() {


User를 상속해서 Player 객체로 선언했는데, 그 타입을 또 체크하는 메서드를 두었다는게 어색하게 느껴지네요
타입을 활용하거나 설계를 다시 고민해봐야하지 않을까요?
지금은 딜러, 플레이어만 있지만 새로운 게임 규칙이 생겨서 새로운 user가 추가된다면 그땐 또 다른 isXXX 시리즈가 계속 생길 수 있겠네요

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

src/main/java/blackjack/model/user/User.java

    }

-    public abstract boolean isHitAvailable();
+    public GameResult judge(User user) {


judge 메서드명 적절한가요?
게임 용어로 바꿔보면 어떨까요?

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

src/main/java/blackjack/view/InputView.java

@@ -11,8 +11,13 @@ public static String readPlayerName() {
        return sc.nextLine();


약어 금지입니다

kang-hyungu · 2026-03-18T01:29:04Z

src/main/java/blackjack/view/OutputView.java


 public class OutputView {
+
+    private static final int BLACKJACK_SCORE = 21;


게임 규칙에 해당하는 상수를 view에서 처리하는 이유가 있을까요

중복 제거를 위해 Finished 상위 상태 클래스 추가 Finished 상태 추가하면서 isFinished() 메서드 추가 stay() 메서드 추가

pci2676

상태패턴 적절히 구현해주셨네요

블랙잭은 여기서 마무리해도 좋을것 같습니다

남은 미션들도 건투를 빌겠습니다.

merge하겠습니다. 🚀

pci2676 · 2026-03-18T13:56:55Z

src/main/java/blackjack/model/state/Bust.java

+        return this;
+//        throw new IllegalStateException("버스트 상태에서는 카드를 추가로 받을 수 없습니다.");


어떤게 의도하시는 방향일까요?

pci2676 · 2026-03-18T14:35:42Z

src/main/java/blackjack/model/state/HandState.java

+import blackjack.model.card.Hand;
+import blackjack.model.gameresult.GameResult;
+
+public interface HandState {


khyej and others added 30 commits March 5, 2026 11:59

docs: 기능 문서 작성

c15d501

feat: 사용자 이름 입력 기능 구현

5b1fa0f

feat: 카드 최초 배분 기능 구현

6914d5e

feat: 최초 카드 현황 출력 기능 구현

54d9be6

feat: 카드 추가 지급 기능 구현

219ee29

refactor: CardProvider 역할 분리

8e6ad0a

refactor: 버스트 여부 변수 추가

89911aa

refactor: 변수명 수정

3a0577d

refactor: 블랙잭 기록 및 조회 메서드 추가

fd6108d

test: BlackjackController 단위 테스트 작성

7dd3ffe

docs: 기능 목록 수정

0ce22d2

feat: 사용자별 카드 현황 및 합계 출력

ccc2f7d

feat: 최종 승패 계산 및 출력 기능 구현

9e9bff0

feat: 게임 승패 계산 로직 추가

414c5b3

refactor: 출력문 형식 수정

18f2d9c

feat: 예외 처리 추가

f58565d

refactor: getter 제거

a2b6b06

refactor: 매직변수 상수화

d855fc4

docs: 구현 기능 목록 수정

46807d6

test: DisplayName 수정

e97ce9b

refactor: Card 객체 설계 수정

00f55da

refactor: 이름 값객체 생성

0075c48

refactor: name 값객체 생성에 따른 수정

0fa789c

refactor: 카드 추가 여부 입력값 객체 생성

05dc9f3

refactor: User 추상 클래스로 변경 & 추상 메서드 추가

7abdbed

refactor: Users 객체 생성

b9ac11c

refactor: 클래스명 변경 & CardCalculator 책임 위임

64a8d08

chore: 상수명 수정

9719248

refactor: 클래스명 수정 & 메서드 추가

417f72f

chore: getter 추가

19a74ec

Chocoding1 added 14 commits March 15, 2026 13:08

refactor: 에러 메시지 내용 수정

bb11f00

refactor: 방어적 복사 적용

ee03f6c

refactor: 미사용 필드, 메서드 제거

26b13e7

refactor: BlackjackGame에서 View 참조 제거

ad9f3b4

test: 미사용 변수 및 중복 테스트 클래스 제거

fdf622c

refactor: 미사용 파라미터 제거

6fe7b0b

refactor: 승패 판정 로직 Player로 이동

e36f7b4

test: 반복 테스트 로직 수정

18aa52e

refactor: 미사용 클래스 제거

437c65a

refactor: User 추상화를 깨지 않도록 Player, Dealer 참조 제거

98af4e9

refactor: 추상화 관련 주석 제거

04cde52

refactor: 추상 메서드 선언 순서 수정

5ccc95f

test: test 클래스에 에러 메시지 적용하기 위해 에러 메시지 package-private 설정

7a2e591

test: DisplayName 수정

b71faed

pci2676 requested changes Mar 16, 2026

View reviewed changes

refactor: 에러 메시지에 원인 요소 추가

624da9e

kang-hyungu requested changes Mar 18, 2026

View reviewed changes

Chocoding1 added 11 commits March 18, 2026 14:47

test: 에러 메시지 테스트 로직 수정

56d49a8

fix: hitPlayer 버스트 상태여도 카드 추가로 받는 오류 해결

27e753f

refactor: 상태 패턴 도입하여 User의 승패 판단 로직 위임

9293e35

chore: 메서드명 도메인에 맞게 수정

7dbe783

refactor: 상태 패턴 수정

b68752e

중복 제거를 위해 Finished 상위 상태 클래스 추가 Finished 상태 추가하면서 isFinished() 메서드 추가 stay() 메서드 추가

refactor: state 폴더 생성

ef99444

refactor: State 외부 변환 금지를 위해 setState() 메서드 제거

a3dbc43

refactor: Controller에서는 Users만 넘기도록 수정

34e18b9

refactor: command 폴더 생성

2bd6557

refactor: 미사용 메서드 삭제

7e6dd4e

refactor: HitCommandTest 경로 수정

4f9db93

pci2676 approved these changes Mar 18, 2026

View reviewed changes

pci2676 merged commit cf120f8 into woowacourse:chocoding1 Mar 18, 2026


		import blackjack.model.bet.BetAmount;

		public enum GameResult {

		@@ -11,8 +11,13 @@ public static String readPlayerName() {
		return sc.nextLine();


		public class OutputView {

		private static final int BLACKJACK_SCORE = 21;

		return this;
		// throw new IllegalStateException("버스트 상태에서는 카드를 추가로 받을 수 없습니다.");

Conversation

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

Uh oh!

체크 리스트

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

TDD

초기 설계

리팩토링

Uh oh!

pci2676 left a comment

Choose a reason for hiding this comment

Uh oh!

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!

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

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Chocoding1 commented Mar 13, 2026 •

edited

Loading