[🚀 사이클1 - 미션 (블랙잭 게임 실행)] 이산 미션 제출합니다. by Rix01 · Pull Request #1001 · woowacourse/java-blackjack

Rix01 · 2026-03-07T14:39:52Z

🃏 블랙잭

체크 리스트

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

객체지향 생활체조 요구사항을 얼마나 잘 충족했다고 생각하시나요?

1~5점 중에서 선택해주세요.

선택한 점수의 이유를 적어주세요.

구현 완료 후 코드를 다시 살펴보니 객체지향 생활체조 요구사항 9원칙 중, '모든 엔티티를 작게 유지한다'는 규칙을 잘 지키지 못한 것 같다고 생각합니다.

BlackJackController에 다소 긴 코드가 있는 것 같고 해당 컨트롤러가 너무 많은 책임을 지고 있는 것 같다고 느껴졌습니다. 아직은 책임 분리를 위해 어떤 것을 해야할지에 대해 지식이 조금
부족하다고 스스로 느꼈습니다.
'게터 프로퍼티를 쓰지 않는다'는 규칙을 잘 지키지 못한 것 같아서 부족하다고 느껴졌습니다.

사실 지금까지 프로젝트를 진행하면서 getter 없이 수행해 본 적이 없었습니다. 그래서 이번 1차 페어프로그래밍에서도 본격적으로 도입할 생각을 못했던 것 같습니다.

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

책임 분리를 위해 어떤 식으로 보완하는 것이 좋을지 피드백 받고 싶습니다.

BlackJackController에 다소 많은 책임이 부여되고 있습니다. 해당 부분을 어떤 식으로 책임분리를 하면 좋을지 피드백 받고 싶습니다. 또한, 책임 분리를 하기 위해 필요한 선수 지식이 있다면
해당
부분이 어떤 부분인지 알려주시면 먼저 공부한 후 도입해보고 싶습니다.
이번 과제를 진행하면서 처음으로 제대로 TDD 형식으로 과제를 진행해보았습니다. 현재 실무에서는 TDD 방식이 어떻게 생각되고 있는지 궁금합니다.

처음에는 TDD로 과제를 진행하게 되면 너무 많은 시간이 들 것 같다고 생각을 했었는데 오히려 이렇게 차근차근 진행하다보니 실수가 많이 줄었고, 오히려 더 빠르게 개발이 진행될 수 있었다고 생각합니다. 저는
이번
과제를 통해 TDD가 정말 좋다고 생각하게 되었는데 실무에서도 TDD 방식을 많이 선호하는지 알고 싶습니다. 또한, 페어 덕분에 TDD 방식을 제대로 경험해 볼 수 있었는데 조금 더 보완해야 할 부분이
있다면 어떤 부분인지 꼭 피드백 받고 싶습니다. TDD에 대해 아직 모르는 부분이 많다보니 피드백 많이 해주시면 감사하겠습니다.
해당 블랙잭 과제에서 카드를 셔플하고 나눠주는 로직이 있는데 그런 랜덤 테스트에 대해서는 어떤 방식으로 테스트 코드를 작성하면 좋은지 피드백 받고 싶습니다.

어떤 방식으로 하면 좋을지 잘 모르겠어서 우선은 디버깅을 통해 잘 섞였는지 여부를 확인하고 넘어갔습니다.
Player와 Dealer를 Participant라는 추상 클래스로 묶었는데 잘 한 부분인지, 부족한 부분이 있다면 어떻게 보완하면 좋을지 피드백 받고 싶습니다.

verus-j

안녕하세요 이산~
테스트 코드 개선하시느라 고생하셨습니다! 이번에는 프로덕션 코드 위주의 피드백 남겨드렸습니다. 확인 후 수정 부탁드려요

src/main/java/blackjack/domain/Participant.java

src/main/java/blackjack/domain/Dealer.java

src/main/java/blackjack/controller/BlackJackController.java

src/main/java/blackjack/domain/BlackJackGame.java

src/main/java/blackjack/domain/BlackJackJudge.java

src/test/java/blackjack/domain/BlackJackGameTest.java

src/test/java/blackjack/domain/DealerTest.java

src/test/java/blackjack/domain/PlayersTest.java

verus-j

안녕하세요 이산~
피드백 반영하느라 고생하셨습니다. 이번 사이클은 여기서 이만 머지하겠습니다. 몇가지 피드백 남겨뒀는데, 다음 사이클 진행하실 때 참고 부탁드려요~

verus-j · 2026-03-11T12:53:42Z

src/test/java/blackjack/domain/BlackJackGameTest.java

+    void 게임이_시작되면_참여자_모두_카드를_2장씩_받는다() {
+        // arrange
+        List<String> names = List.of("이산", "모카", "바니", "소낙눈");
+        Players players = new Players(names);
+        Dealer dealer = new Dealer();
+        Deck deck = new Deck();
+        BlackJackGame blackJackGame = new BlackJackGame(players, dealer, deck);
+
+        // act
+        blackJackGame.initDraw();
+
+        // assert
+        List<Participant> allParticipants = Stream.concat(
+                Stream.of(dealer), players.getPlayers().stream()
+        ).toList();
+        assertThat(allParticipants).allSatisfy(participant ->
+                assertThat(participant.getCardCount()).isEqualTo(2));
+    }


BlackJackGame의 동작을 검증하기 위해 외부에서 생성되어 주입된 객체의 상태를 확인하는게 이상하지 않으신가요? BlackJackGame은 initDraw로 주입 받은 객체의 상태만 변경시키고 이후에 사용할 수 있는 기능이 없네요. 현재 구조에서 BlackJackGame이 유틸성 클래스와 어떤 점이 다르다고 볼 수 있는걸까요?

안녕하세요 베루스!
1차 테스트 코드에서부터 코드 부분까지 모두 상세하게 피드백 해주셔서 정말 감사합니다.🙇🏻‍♀️
BlackJackJudge 클래스를 없애는 과정만 생각하다보니 그 과정에서 BlackJackGame의 책임이 매우 작아진 것을 제대로 확인하지 못했네요. 수정해보겠습니다!

verus-j · 2026-03-11T12:54:28Z

src/main/java/blackjack/domain/Hand.java

+    private int getNormalPoint() {
+        int totalPoint = 0;
+        for (Card card : cards) {
+            totalPoint += card.getCardPoint();
+        }
+        return totalPoint;
+    }


여기도 Stream을 적용해보면 좋을 것 같아요~

적용하겠습니다!!

verus-j · 2026-03-11T13:14:10Z

src/test/java/blackjack/domain/ParticipantTest.java

+    @Test
+    void 참여자는_카드를_받을_수_있다() {
+        Participant participant = new Player("이산", new Hand());
+        Deck deck = new Deck();
+        participant.receiveCard(deck.draw());
+    }


여기는 검증문이 없네요 ㅠㅠ

헉..! 테스트 코드 수정 도중에 깜박하고 검증문을 작성하지 못한 것 같네요.. step2에서 꼭 수정하겠습니다!!

softmoca added 30 commits March 4, 2026 17:20

docs : 기능 구현 목록 작성

fbae0b9

docs : 요구사항분석 및 이해와 도메인 후보 및 유즈케이스 추가

65408f8

test(CardPattern) : 카드 패턴은 4종류만 가능한지 테스트 추가

2c22909

feat(CardPattern) : 카드패턴 enum 구현

0abe7d0

test(CardPoint) : 이름과 점수를 가지는 CardPoint enum 생성 테스트

eb2e0a5

feat(CardPoint) : 이름과 점수를 가지는 CardPoint 구현

c896db4

test(Card) : CardPoint와 CardPattern을 가지는 Card 생성 테스트

e697ccc

feat(Card) : CardPoint와 CardPattern을 가지는 Card 생성 구현

18b928b

test(Deck) : 52장의 카드를 가지는 Deck 생성 테스트

5144b7a

feat(Deck) : 52장의 카드를 가지는 Deck 생성 구현

041afd4

feat(Deck) : 셔플 기능 구현

a0877c4

test(Card) : 에이스인지 아닌지 테스트

d285eec

feat(Card) : 에이스인지 확인하는 기능 구현

e05bcee

test(Hand) : 초기 생성 카드 개수 및 카드 추가 테스트

d713507

feat(Hand) : 카드 추가 기능 구현

84144e8

test(Hand) : 총 점수 계산 테스트

8d42b23

feat(Hand) : 총 점수 계산 구현

1864a3a

test(Hand) : 버스트인 경우 총합 계산 테스트 추가

25559b2

feat(Hand) : 버스트인 경우 총합 계산 구현

3920ff9

test(Hand) : 버스트 상태 확인 테스트

20003cc

feat(Hand) : 버스트 상태 기능 구현

3f165c4

test(Player) : 카드 받기 테스트

6f601d2

feat(Player) : 카드 받기 기능 구현

7d2dce6

test(Dealer) : 딜러 카드 받기 테스트

2ae1b5b

feat(Dealer) : 딜러 카드 받기 구현

17960bb

test(GameResult) : 게임결과 생성 테스트

db030da

feat(GameResult) : 게임 결과 생성 구현

24e0562

feat(InputView): 플레이어 이름 입력 구현

a958072

feat : 초기 카드 두장 지급 기능 구현

5bf4110

feat(OutputView) : 딜러와 플레이어 초기 카드 출력

8ccd089

refactor: 22점 경계값으로 버스트인 테스트 코드 추가

88910bd

verus-j requested changes Mar 10, 2026

View reviewed changes

Chaerin Lee added 26 commits March 11, 2026 12:53

refactor: Hand를 private으로 리팩토링

b44bc25

refactor: 딜러의 손패 17 이상인 경우 테스트 추가 및 리팩토링

4a681d1

refactor: 불필요한 상수 및 메서드 정리

9dc5914

refactor: BlackJackController에서 불필요한 주석 제거

e5a76dd

refactor: while문 조건 정리 및 불필요한 주석 제거

3aedae1

refactor: getTotalPoint() 메서드 추출

04264aa

refactor: 에이스 개수 반환 메서드 스트림 사용하여 리팩토링

8f10c00

fix: while 조건문 실수 수정

8ec78ce

refacotr: 불필요한 테스트코드 삭제

b92127f

refactor: 불필요한 메서드 제거 및 이름 대신 카드 리스트 리턴하도록 수정

e01be06

test(Player): 승패 결정 책임을 사용자에게

cb45307

feat: 딜러와 결과 비교 기능 추가

9c17238

refactor: BlackJackJudge 삭제 후 BlackJackGame에서 결과 반환

715dfab

refactor: DealerTest에서 불필요한 테스트 코드 삭제

4fd8efe

refactor: 플레이어들 카드 받기 테스트 수정

8de5e66

refactor: PlayerGameResultDto 활용

df1114f

refactor: ParticipantDto 추가하여 리팩토링

b438c96

refactor: 메서드 추출로 인덴트 줄이기

fa22013

refactor: 불필요한 메서드 삭제

6615ee9

refactor: view private final로 수정

2e9f925

refactor: 불필요한 매개변수 삭제

3ed6046

refactor: Scanner private final로 수정

2dc6d4d

refactor: 미사용 메서드 삭제 및 boolean 메서드 정리

ade7d40

refactor: 메서드명 오타 수정

2cc9b13

refactor: 테스트 코드 수정

54ecbd6

refactor: 다른 테스트 코드와 동일한 형태로 수정

cce2d5f

verus-j approved these changes Mar 11, 2026

View reviewed changes

verus-j merged commit d40ac81 into woowacourse:rix01 Mar 11, 2026

Conversation

Rix01 commented Mar 7, 2026

🃏 블랙잭

체크 리스트

객체지향 생활체조 요구사항을 얼마나 잘 충족했다고 생각하시나요?

1~5점 중에서 선택해주세요.

선택한 점수의 이유를 적어주세요.

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

Uh oh!

verus-j left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

verus-j left a comment

Choose a reason for hiding this comment

Uh oh!

verus-j Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

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

Uh oh!

Choose a reason for hiding this comment

Uh oh!

verus-j Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

Rix01 Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

verus-j Mar 11, 2026

Choose a reason for hiding this comment

Uh oh!

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

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Rix01 Mar 11, 2026 •

edited

Loading

Rix01 Mar 11, 2026 •

edited

Loading