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

Eian1106 · 2026-03-12T16:15:44Z

체크 리스트

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

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

안녕하세요 초코칩!
많이 부족한 제 코드를 봐주시고 리뷰 해 주셔서 정말 감사합니다.. 🥺 이번에도 부탁드립니다!

사이클1 마지막 피드백은 아직 적용하지 않았습니다.
금요일 피드백 활동인 변경 범위 비교: 사이클1 vs 사이클2 을 우선적으로 확인하고 싶었기 때문입니다.

Controller에서 승무패 로직을 의도적으로 주석처리 했습니다.

// ResultView.printResult(gameService.result());

지금은 여러가지 질문을 조금 하고 싶습니다..
초코칩의 생각이 궁금해요!

메서드명과 변수명을 짓는 게 어렵습니다.
- AI를 사용하지 않고 해보려 노력중입니다. 하지만 메서드명과 변수명 작성은 추후 나를 위해서나, 내 코드를 보는 모든 이들을 위해 잘 지어야 한다고 생각합니다. 지금은 구글 번역기나 단어 검색을 통해 작성하곤 합니다..
- Q) 네이밍 같은 경우는 AI를 사용해도 무방할까요?
블랙잭 완료 후 어떤 공부를 하면 좋을까요?
- 다른 크루의 코드와 비교해보면 제 코드는 정돈되지 않은 느낌입니다. 노력은 했다고 자신할 수 있으나, 결과물은 🤔 ..
  그래서 좋은 코드, 나쁜 코드를 읽어볼까 싶기도 합니다. 모든 공부는 진행 할 예정이지만 공부 순서에 고민이 생깁니다.
  객체지향의 사실과 오해는 읽어보았으나 체화 하기엔 부족했던 것 같습니다. 고민중인 도서(공부) 후보는 다음과 같습니다.
  - 좋은 코드, 나쁜 코드
  - 클린 코드
  - 클린 아키텍처
  - 오브젝트
  - TDD 관련
  - JAVA 관련
  해야 할 공부는 많지만, 방향성에 대해서 갈피를 못 잡고 있는 것 같습니다!.. 조언 해주실 수 있을까요?

Chocochip101

안녕하세요, 이안! 리뷰어 초코칩입니다 🍪
피드백과 변경점이 많아져서 나눠서 리뷰하겠습니다.
커맨트로 리뷰 남겼으니 확인 후 재요청주세요~
그리고 사이클1 마무리에 남긴 리뷰도 반영해주세요!

Chocochip101 · 2026-03-12T20:35:58Z

src/main/java/service/GameService.java

+
+        for (Map.Entry<String, WinStatus> playerWinStatus : playerGroups.getGameResult().entrySet()) {
+            String userName = playerWinStatus.getKey();
+            int userCost = (int) (userBetInfo.get(userName) * playerWinStatus.getValue().getEarningsRate());


int로 형변환하면 소수점은 전부 잘리겠군요.

Chocochip101 · 2026-03-12T20:46:30Z

src/main/java/service/GameService.java

#1014 (comment)

Chocochip101 · 2026-03-12T20:48:44Z

src/main/java/controller/GameController.java

+
+    private Map<String, Integer> bettingRound() {
+        List<Player> players = gameService.getPlayers();
+        Map<String, Integer> userBetCost = new HashMap<>();


자료형보다는 객체로 표현하는 것은 어떤가요?

Chocochip101 · 2026-03-12T20:51:18Z

src/main/java/controller/GameController.java

    public void run() {
        startGame();
+
+        Map<String, Integer> userBetInfo = bettingRound();


금액이 음수도 가능하네요. 배팅 금액에도 필요한 검증 로직이 필요해보이네요.

배팅 금액도 원시 타입이 아닌 객체로 관리할 수 있을까요?

Chocochip101 · 2026-03-12T20:54:05Z

src/main/java/service/GameService.java

        return playerGroups.getGameResult();
    }

+    public List<ParticipantBetResult> bettingResult(Map<String, Integer> userBetInfo) {


배팅 관련 테스트가 없어요. 블랙잭일 때의 금액 로직, 딜러가 졌을 때 돈을 잃는지 등등이요. 많은 테스트 케이스가 있을 것 같아요.

모든 유즈 케이스 테스트로 검증되도록 추가해주세요.

README.md

src/main/java/service/GameService.java

Chocochip101 · 2026-03-12T21:29:07Z

src/main/java/service/GameService.java


+    public List<ParticipantBetResult> bettingResult(Map<String, Integer> userBetInfo) {
+        List<ParticipantBetResult> profitResults = new ArrayList<>();
+        int dealerCost = 0;


dealerCost는 서비스가 관리하고 있네요. dealer도 금액이 있고, participant도 금액이 있는데 굳이 서비스가 관리할 필요가 있을까요?

Eian1106 · 2026-03-14T19:50:24Z

int로 형변환하면 소수점은 전부 잘리겠군요.

초기 베팅 금액 입력을 100 이상으로 제한하도록 변경함으로써,
블랙잭 승리시 1.5배 하더라도, 소수점이 생기지 않도록 하였습니다!

dealerCost는 서비스가 관리하고 있네요. dealer도 금액이 있고, participant도 금액이 있는데 굳이 서비스가 관리할 필요가 있을까요?

프로그래밍 요구 사항인 3개 이상의 인스턴스 변수를 가진 클래스를 쓰지 않는다. 가 아니였다면 participant가 관리하도록 할 것 같습니다... dealerCost는 베팅 결과 계산 전 까지 사용되지 않는다고 판단해서 bettingResult() 내부에서 관리하도록 했습니다.🤔
dealerCost는 바로 다시 읽어와야 하는데, 다른 객체에 계산값을 넣었다가, 바로 다시 읽어오는 게 괜찮을까? 라는 고민을 했습니다.
베팅 금액을 관리하는 클래스를 만들고, 계산하는 책임도 관리하는 편이 제일 나을까요?
이 부분에 대해서 조금 더 고민해보겠습니다!..

책을 읽으려고 하는 목적이 무엇인지 한 번 생각해보셔도 좋을 것 같습니다.

책을 읽으려는 이유는 비유를 하자면 이렇게 느껴졌습니다.
간식 여러개로 배 채우는 것보다 든든한 한 끼 먹자! 😆
아직 여러 방법으로 공부해본 것은 아니지만, 모르는 것을 인터넷으로 검색해서 공부할 땐 이어지지 않고, 부분부분 파편화 된 기억으로 남는 것 같았어요. 책은 조금 달랐거든요
그렇지만 책은 읽는 데 속도가 느린만큼 잘 조절해 봐야겠어요! 약간의 완독병(?)이 있어서 불필요한 부분도 읽거든요.
아직까진 책으로 학습이 최고였지만, 다양한 방법을 우테코 기간에 시도해보며 장단점을 비교하고 적용해보겠습니다 😄

블랙잭 미션을 끝낸 뒤 처음부터 다시 구현해보기
지금은 이게 제일 하고싶어요!

항상 좋은 피드백 감사합니다 초코칩! 😆

Chocochip101

안녕하세요, 이안!
사이클 1의 피드백을 잘 수정해주셨네요 👍🏻
변경해주신 점들에 대해 커맨트로 피드백 남겼으니 확인해주세요!

Chocochip101 · 2026-03-14T21:18:08Z

src/main/java/domain/card/NoShuffleStrategy.java

+
+import java.util.List;
+
+public class NoShuffleStrategy implements ShuffleStrategy {


테스트에서만 쓰이는데, 테스트 패키지로 옮겨보는건 어떨까요?

Chocochip101 · 2026-03-14T21:26:40Z

src/main/java/domain/card/RandomShuffleStrategy.java

+public class RandomShuffleStrategy implements ShuffleStrategy {
+
+    @Override
+    public void shuffle(List<Card> cards) {


인자로 들어온 cards가 immutable한 자료형이라면 어떤 일이 발생할까요? 또한 원본 cards가 섞이고 난 후에 변경이 존재할까요? 그렇지 않다면 방어적 복사로 void가 아닌 immutable한 랜덤 List<Card>를 반환해도 되지 않을까요?

Chocochip101 · 2026-03-14T21:30:00Z

src/main/java/domain/card/TrumpNumber.java

+        return value.getFirst();
+    }
+
+    public int getUpgradeValue() {


순서로 ACE의 값의 1 또는 11 여부를 결정하네요. 만약 누군가가 ACE("A", List.of(11, 1)),로 바꿔서 배포하면 바로 서비스에 장애가 날수도 있겠네요 😨

'Upgrade'보다는 isConvertibleTo 같은 메서드 명은 어떠신지요 ㅎㅎ

Chocochip101 · 2026-03-14T21:30:25Z

src/main/java/domain/player/HandCards.java


-    private boolean canPlusBonusScore(int sum) {
-        return containsAce() && sum + ACE_BONUS_SCORE <= MAX_SCORE;
+    public int quantity() {


메서드는 동사형으로 바꾸는건 어떨까요?

Chocochip101 · 2026-03-14T21:33:48Z

src/main/java/domain/card/CardDeck.java

+        for (TrumpSuit suit : TrumpSuit.values()) {
+            for (TrumpNumber number : TrumpNumber.values()) {
+                cards.add(new Card(suit, number));
+            }
+        }


인덴트가 2네요. 분리해볼까요?

Chocochip101 · 2026-03-14T21:36:44Z

src/main/java/domain/vo/Cost.java

+    }
+
+    private void validateRange(int cost) {
+        if (cost > MAX_COST || cost < MIN_COST) {


저는 Bigdecimal이나 double로 소수점을 챙기는 방식으로 접근했는데, 도메인 제약으로 해결하는 것도 훌륭하네요 👍🏻 예외 처리도 좋습니다!

Chocochip101 · 2026-03-14T21:40:34Z

src/main/java/service/GameService.java

        for (Map.Entry<String, WinStatus> playerWinStatus : playerGroups.getGameResult().entrySet()) {
            String userName = playerWinStatus.getKey();
-            int userCost = (int) (userBetInfo.get(userName) * playerWinStatus.getValue().getEarningsRate());
+            int userCost = (int) (userBetInfo.get(userName).getCost() * playerWinStatus.getValue().getEarningsRate());


int 형변환 때문에 100단위로 받는다는 것을 모르는 누군가는 이 코드만을 보고 오해할 수도 있을 것 같아요.

해당 로직을 서비스가 아닌 도메인으로 옮길 수 있을까요? (int로 100단위로 다루기한게 저희 도메인 규칙이 되었으니까요) 기존 도메인에 옮기기 힘들다면 새로운 일급 컬렉션 만들어도 되겠네요.

Chocochip101 · 2026-03-14T21:52:49Z

src/main/java/domain/player/Participant.java

@@ -7,6 +7,7 @@

 public class Participant {


프로그래밍 요구 사항인 3개 이상의 인스턴스 변수를 가진 클래스를 쓰지 않는다. 가 아니였다면 participant가 관리하도록 할 것 같습니다

맞습니다. name, handCards, cost를 모두 넣으면 3개가 되어 제약을 어기게 되네요! 하지만 이 제약의 핵심 의도는 '로직을 서비스로 밀어내라'는 것이 아니라, '객체를 더 작고 의미있는 단위로 쪼개라'는 것으로 이해하고 있어요.

저희는 요구사항을 만족해야하니, HandCards와 Cost를 묶는 새로운 객체(예: PlayerStatus)를 도입한다면, 각 객체는 변수를 2개 이하로 유지하면서도 '자신의 수익을 스스로 계산하는 로직'을 가질 수 있게 되겠네요!
서비스는 그저 이 객체들에게 메시지만 던지면 되고요. 이렇게 리팩토링해보면 어떨까요?

public class Participant { private final Name name; private final PlayerStatus status; ... } public class PlayerStatus { private final HandCards handCards; private final Cost cost; public int calculateProfit(WinStatus winStatus) { ... // 어떤 아무개 로직... } }

Eian1106 added 13 commits March 12, 2026 13:37

fix: 점수가 21일 때 뽑지 않도록 변경

e73c66c

feat(Participant): BlackJack 판단 로직 추가

99cc005

feat(WinStatus): 배율 추가

a0ca40e

feat: 참가자 배팅 결과 DTO(ParticipantBetResult) 생성

b6273ea

feat: BLACKJACK_WIN 판별 로직 추가

b9f74c4

cleanup: 사용하지 않는 메서드 제거

a114fb9

feat: 배팅금액 입력 기능 구현

b118d0a

feat(GameService): 수익 계산 로직 구현

0772e4e

feat: 최종 수익 출력 구현

10bbef0

test: 블랙잭 판별 테스트 추가

7272882

feat: 숫자 검증 로직 추가

e9c7af4

docs: readme 업데이트

6a4648e

cleanup(controller): 승무패 로직 주석화

98cf2a3

Chocochip101 requested changes Mar 12, 2026

View reviewed changes

Eian1106 added 7 commits March 15, 2026 01:20

refactor: shuffle 전략패턴 사용하도록 변경

06e7277

feat: Cost vo 생성

14ddb85

refactor: Cost 객체 사용하도록 변경

3b1f650

refactor(ParticipantBetResult): 금액관련 변수명 통일 profit -> cost

b42e7b0

test(GameService): 배팅 결과 검증 테스트 추가

9b93503

refactor: 에이스 점수 관리 위치 변경

efcb079

test: ACE가 두장일 경우 테스트 추가

2783c1f

Chocochip101 requested changes Mar 14, 2026

View reviewed changes


		import java.util.List;

		public class NoShuffleStrategy implements ShuffleStrategy {

Conversation

Eian1106 commented Mar 12, 2026

체크 리스트

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

Uh oh!

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

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Eian1106 commented Mar 14, 2026

Uh oh!

Chocochip101 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Chocochip101 Mar 14, 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!

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

Chocochip101 Mar 14, 2026 •

edited

Loading