[🚀 사이클2 - 미션 (블랙잭 게임 실행)] 아론 미션 제출합니다. by sun007021 · Pull Request #1109 · woowacourse/java-blackjack

sun007021 · 2026-03-13T08:48:53Z

체크 리스트

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

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

이번 기능을 구현하면서 기존 코드들의 구조를 리팩터링 한 부분들이 많았습니다. 현재 각 객체의 책임이 잘 분리된 것인지 궁금합니다.
ENUM 사용시 책임 분리
- 현재 push는 안하였지만 Rank와 Shape를 Enum으로 변환하는 작업을 진행중입니다. 이때,
  기존에는 ACE의 값을 11로 고정하였고 나중에 Cards에서 총합 계산시에 10씩 뺄셈을 하는 방향으로 진행을 하였는데 이게 올바른 책임 분리가 맞을지 잘 모르겠습니다.
  만약 ACE값 처리에 대한 규칙이 바뀐다면 ENUM만 수정해주도록 구조를 만드는 편이 책임이 확실하다고 생각하였는데 현재 점수에 따른 계산이 필요해서 Enum에서 처리하도록 하지는 못 할 것 같습니다.

hyeonic

안녕하세요 아론~ 빠르게 구현해주셨네요! 궁금한 부분 간단히 코멘트 남겼으니 확인해주세요!

현재 push는 안하였지만 Rank와 Shape를 Enum으로 변환하는 작업을 진행중입니다. 이때,
기존에는 ACE의 값을 11로 고정하였고 나중에 Cards에서 총합 계산시에 10씩 뺄셈을 하는 방향으로 진행을 하였는데 이게 올바른 책임 분리가 맞을지 잘 모르겠습니다.
만약 ACE값 처리에 대한 규칙이 바뀐다면 ENUM만 수정해주도록 구조를 만드는 편이 책임이 확실하다고 생각하였는데 현재 점수에 따른 계산이 필요해서 Enum에서 처리하도록 하지는 못 할 것 같습니다.

10씩 뺄셈하는 연산의 책임을 누구에게 두냐에 따라 달라지지 않을까 싶네요 ㅎㅎ 아론이 생각하는 올바른 책임 분리는 무엇인가요??

hyeonic · 2026-03-13T13:13:55Z

src/main/java/blackjack/domain/Card.java

-    @Override
-    public boolean equals(Object o) {
-        if (this == o) {
-            return true;
-        }
-        if (!(o instanceof Card card)) {
-            return false;
-        }
-        return rank.equals(card.rank) &&
-                shape.equals(card.shape);
-    }
-


equals 구현은 왜 제거하였나요?

기존에 Card가 일반 클래스였는데 상태가 변하지 않는 객체인것 같아 Record로 변경하였습니다.
이때, Record 사용시 equals, hashCode 등의 메서드를 자동으로 생성해주기에 불필요하다고 판단되어 제거하였습니다.

hyeonic · 2026-03-13T13:15:04Z

src/main/java/blackjack/domain/Cards.java

-            }
-        }
-        return count;
+        return (int) cards.stream().filter(card -> card.rank().equals("A")).count();


card에게 ace 여부를 물어보는 건 어떻까요?

외부에서 직접 비교하는 것 보다 card자체에서 여부를 반환해주면 훨씬 깔끔해지겠네요!

hyeonic · 2026-03-13T13:16:19Z

src/main/java/blackjack/service/Game.java

+        for (Map.Entry<Player, ScoreCompareResult> entry : playerResults.entrySet()) {
+            Player player = entry.getKey();
+            ScoreCompareResult result = entry.getValue();
+
+            double profit = player.calculateProfit(result);
+            dealerProfit += profit * -1.0;
+            playerProfits.put(player, profit);
+        }


double을 기반으로 한 실수 연산은 안전할까요? 값이 큰 경우 어떤 문제가 발생할 수 있나요?

정밀도 문제 때문에 정확한 계산이 어렵고 == 같은 연산이 예상과 다르게 동작 할 수 있습니다!
이번에 찾아보니 BigDecimal을 사용해 이러한 문제를 해결할 수 있는 방법이 있네요. 이 방식으로 바꿔보겠습니다!

hyeonic · 2026-03-13T13:17:39Z

README.md

+- [x] 수익 계산
+    - [x] 플레이어는 승리시 배팅금액 만큼 수익을 얻는다.
+        - [x] 플레이어가 블랙잭인지 판단할 수 있다.
+            - 처음 받은 2개의 카드가 21인 경우 블랙잭으로 판단
+        - [x] 참가자가 블랙잭인 경우 배팅금액의 1.5배 수익을 얻는다.
+    - [x] 플레이어는 패배시 배팅금액 만큼 손해를 본다
+    - [x] 플레이어는 무승부시 배팅금액 만큼 수익도 손해도 보지 않는다.
+    - [x] 딜러는 플레이어들이 패배한 금액 만큼 수익을 얻는다.
+    - [x] 딜러는 플레이어들이 승리한 금액 만큼 손해를 본다.


신규 요구사항을 잘 정리하였네요 😃

hyeonic · 2026-03-13T13:18:57Z

src/main/java/blackjack/domain/ProfitResults.java

+public record ProfitResults(Double dealerProfit, Map<Player, Double> playerProfit) {
+}


별다른 행위가 없지만 domain 패키지에 위치하여 있네요! 어떤 역할을 하게 되나요?

기존 record들중 이걸 포함해서 단순 출력을 위한 record들이 있어서 패키지를 분리해야 할 것 같습니다.
단순 view 전달용 record들은 dto 패키지를 만들어서 분리하고자 합니다.
이때, 고민은 ProfitResults와 GameResults는 단순 출력용이 아니라 Game등 도메인 객체에서 반환하는 값이고 도메인 계층간 값을 전달할 때 사용되는데 이 경우에도 dto패키지 등으로 분리하는게 맞는지 궁금합니다!

hyeonic · 2026-03-13T13:19:18Z

src/main/java/blackjack/domain/PlayerProfitResultDto.java

+public record PlayerProfitResultDto(String playerName, double profit) {
+
+    public static PlayerProfitResultDto from(Player player, Double profit) {
+        return new PlayerProfitResultDto(player.getName(), profit);
+    }
+}


어떤 건 dto가 붙어 있고 어떤 객체는 아무것도 안 붙어 있는데 선택의 기준이 무엇인가요?

사이클2 진행하면서 기존과 기준이 섞이며 통일되지 않은 것 같습니다.
card를 제외한 나머지는 모두 데이저 전달의 역할만 하기에 dto를 붙이는게 맞다고 생각합니다.
다만, 이번에 dto패키지를 분리하며 dto패키지에 위치하기에 클래스명에도 dto를 붙이는건 중복되어 불필요하다고 생각해 모두 제거하였습니다

hyeonic · 2026-03-13T13:19:42Z

src/main/java/blackjack/domain/Player.java

+    private static final double BLACKJACK_PROFIT_RATE = 1.5;
+
    private final String name;
+    private final int bettingAmount;


돈을 다루는 도메인에서 int라는 자료형은 적합할까요?

int는 범위가 21억정도로 돈을 다루는 도메인에서는 오버플로우가 발생할 수 있어 적합하지 않은 것 같습니다. 이러한 부분들을 전혀 생각하지 못하였었네요
이부분도 마찬가지로 BigDecimal 사용하도록 변경하겠습니다

hyeonic · 2026-03-13T13:21:38Z

src/main/java/blackjack/domain/Player.java

+    public double calculateProfit(ScoreCompareResult compareResult) {
+        if (compareResult == ScoreCompareResult.PLAYER_WIN) {
+            return calculateProfitWhenPlayerWins();
+        }
+        if (compareResult == ScoreCompareResult.PLAYER_LOSE) {
+            return -bettingAmount;
+        }
+        return 0;
+    }
+
+    private double calculateProfitWhenPlayerWins() {
+        if (isBlackjack()) {
+            return (bettingAmount * BLACKJACK_PROFIT_RATE);
+        }
+        return bettingAmount;
+    }


이번 기능을 구현하면서 기존 코드들의 구조를 리팩터링 한 부분들이 많았습니다. 현재 각 객체의 책임이 잘 분리된 것인지 궁금합니다.

이런 부분들을 말씀 하시는 걸까요~? 각각의 책임은 결국 객체가 가진 상태를 외부의 의존 없이 본인이 제어하고 해소할 수 있을 때 적합한 위치에 있다고 생각해요 ㅎㅎ Player 관점에서 승패 여부 등과 같은 정보를 의존해도 괜찮을지 고민해보면 좋을 것 같네요!

sun007021 added 25 commits March 12, 2026 11:00

fix: 카드 뽑기 중 bust되면 즉시 해당 턴 종료하도록 수정

cfb5d7b

docs: 배팅 기능 목록 추가

72fbe0a

test: 플레이어 블랙잭 판단

152762a

feat: 플레이어가 블랙잭인지 판단

f7b8de7

refactor: Game내 일부 판정 메서드 static으로 수정

386cdfa

test: 플레이어가 블랙잭이 아닌 점수로 승리시 수익 계산

5a9a8d3

feat: 플레이어가 블랙잭이 아닌 점수로 승리시 수익 계산

4780292

test: 플레이어가 블랙잭으로 승리시 수익은 배팅금액의 1.5배

50455df

feat: 플레이어가 블랙잭으로 승리시 수익은 배팅금액의 1.5배

bf80e3e

style: LOSE 오타 수정

cafc932

test: 플레이어 패배시 수익 계산

193dfd2

test: 무승부시 수익은 0

0955ddb

refactor: 카드 분배 의존성 및 책임 변경

37b1b7a

refactor: 게임 객체 책임 추가

d28edad

test: 전체 참가자 수익 계산

590b602

refactor: 배팅 금액 및 수익 계산 책임 Player로 변경

b62c344

refactor: 블랙잭 판별 책임 이동

7f65b27

feat: 전체 수익 계산

e36594e

docs: 블랙잭 판단 기능 명세 추가

8fb3ad3

feat: 배팅 금액 입력 추가

feb159e

fix: 배팅금액 재입력 처리시 input 다시 안받아 무한루프 발생 수정

42c92e0

feat: 최종 수익 결과 출력

23997d6

chore: 미사용 클래스 및 import문 삭제

780658f

chore: record 사용으로 불필요 메서드 제거

4daa210

refactor: countAces 메서드 depth초과 스트림 사용으로 변경

63b8ebd

hyeonic requested changes Mar 13, 2026

View reviewed changes

sun007021 added 4 commits March 15, 2026 10:22

refactor: rank, shape String -> Enum 사용 변경

795f55f

refactor: card ace확인 책임 분리

53852e2

refactor: dto 패키지 분리

aca7d15

refactor: 배팅 및 수익금 계산 BigDecimal 사용으로 변경

0d7cf27

		public record ProfitResults(Double dealerProfit, Map<Player, Double> playerProfit) {
		}

Conversation

sun007021 commented Mar 13, 2026

체크 리스트

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

Uh oh!

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

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