🚀 4단계 - 블랙잭(베팅) #826

SeolYoungKim · 2025-01-05T06:42:08Z

안녕하세요 경록님!
마지막 미션도 잘 부탁드립니다 🙇‍♂️

Rok93

설영님 리뷰 요청 주신걸 놓치고있어서 리뷰가 많이 늦었습니다. 🥲
마지막 단계이니 조금만 더 힘내서 미션 마무리 지으면 좋을 것 같습니다.
몇몇 코멘트 남겨두었는데, 확인해서 수정 부탁드릴게요. 🙏

Rok93 · 2025-02-08T03:01:24Z

README.md

+  - [] 무승부인 경우 겜블러가 배팅 금액을 돌려받는다
+  - [] 겜블러가 이기는 경우 배팅 금액을 받는다 
+    - [] 딜러가 21을 초과하면 겜블러들이 승리하므로 겜블러들은 배팅 금액을 돌려받는다
+    - [] 딜러가 21점 이하이고 겜블러가 딜러보다 점수가 높으면 승리하므로 배팅 금액을 돌려받는다 
+    - [] 겜블러의 카드가 2장이고 블랙잭(합이 21)인 경우 1.5배 받는다  


기능 구현을 하셨다면 해당 기능 구현 목록표도 ✅ 표시를 해주세요. 😃
가급적 커밋 단위로 기능을 구현하실 때, 같이 체크를 해주신다면 더 살아있는 문서가 될 것 같아요. 😉

Rok93 · 2025-02-08T03:07:02Z

src/main/kotlin/blackjack/domain/Participants.kt

        require(dealers.size == 1) { "게임 내 딜러는 한명만 참여 가능합니다." }
    }

    private fun requiresAtLeastOneGambler() {
-        val gamblers = participants.filter { participant -> !participant.isDealer() }
+        val gamblers = participants.filter { participant -> !participant.isDealer }


extractDealer, extractGamblers 메서드도 타입대신에 isDealer 필드로 필터링할 수 있을 것 같은데
가능하다면 타입 대신에 다른 방식으로 필터링하도록 변경해보면 어떨까요?

extractDealer, extractGamblers 메서드는 각각 Dealer, Gambler타입을 사용해야만 하는 경우가 있어서 filterIsInstance를 사용했는데요.

이 부분에 대해서 계속 고민을 해보고 있는데, isDealer를 사용해도 결국 타입 캐스팅을 해주어야 하는 상황이 와서, 애초에 extract를 시도하는 시점에서 타입으로 걸러주는게 좋지 않을까? 생각했습니다!

혹시 경록님께서는 Participant를 Gambler, Dealer로 캐스팅 해서 사용해야 할 경우에는 어떤 방법으로 구현을 하실까요?

Rok93 · 2025-02-08T03:07:34Z

src/main/kotlin/blackjack/domain/Participants.kt

@@ -11,12 +11,12 @@ class Participants(private val participants: List<Participant>) {
    }

    private fun requiresOnlyOneDealer() {
-        val dealers = participants.filter { participant -> participant.isDealer() }
+        val dealers = participants.filter { participant -> participant.isDealer }
        require(dealers.size == 1) { "게임 내 딜러는 한명만 참여 가능합니다." }


1이라는 숫자는 매직넘버인 것 같네요.
상수로 추출하여 의미를 드러내주시면 좋을 것 같아요. 😃

Rok93 · 2025-02-08T03:08:12Z

src/main/kotlin/blackjack/domain/Participant.kt

@@ -21,6 +23,15 @@ sealed class Participant(val name: String) {
        return score > BlackjackRule.BLACKJACK_SCORE
    }

+    fun isBlackjack(): Boolean {
+        return cards.size == 2


2라는 숫자도 매직넘버로 보이네요.
이전 코멘트와 동일하게 매직넘버 처리 적절하게 부탁드릴게요!

Rok93 · 2025-02-08T03:13:20Z

src/main/kotlin/blackjack/domain/Participant.kt

@@ -9,6 +9,8 @@ sealed class Participant(val name: String) {
    val score: Int
        get() = _cards.calculateTotalScore()

+    abstract val isDealer: Boolean


블랙잭 4단계에는 3개 이상의 인스턴스 변수를 가진 클래스를 쓰지 않는다라믄 미션 요구사항이 있는데요.

물론 읽기전용 프로퍼티의 경우에는 위에서 언급하는 인스턴스 변수 개념과는 다르다고 생각하지만, 위의 미션 요구사항은 가급적 인스턴스 변수를 불필요하게 선언하지 말라는 의도를 내포하고있다고 생각해요. 😃
개인적으로 isDealer의 경우에는 꼭 필드로 선언할 필요 없이 추상 메서드를 두고, 각 구현체에서 재정의해도 충분할 것 같다고 생각하는데 설영님은 어떻게 생각하시나요? 🤔

저는 isDealer가 상태를 표현하는 것이라고 생각해서 인스턴스 변수로 선언을 하였었는데, 지금 보니 제가 isXxx()는 모두 메서드로 선언을 해두었었군요... ㅋㅋㅋ 일관성을 맞추기 위해서라도 메서드로 변경하는 것이 좋아 보입니다. 경록님 말씀대로 추상 메서드로 변경을 하였어요 👍

추가로, 저는 isDealer를 인스턴스 변수로 선언하느냐, 메서드로 선언하느냐에 대해 정말 많이 고민이 되는 것 같아요.

코틀린에서 선언하는 인스턴스 변수(프로퍼티)들은, 자바의 인스턴스 변수와는 다르게, 값을 저장하는 필드는 backing field로 따로 두고, getter, setter 메서드로써 취급되는 걸로 알고 있습니다.

그러다 보니 인스턴스 변수로 선언하는 것과, 메서드로 따로 선언하는 것이 별 차이가 없어보이고, 결국 isDealer와 같은 것들을 필드로 쓸지? 메서드로 추출해서 쓸지? 에 대해 어떤 기준을 잡고 사용을 할지 아직도 결정을 못내렸습니다. ㅋㅋㅋ

혹시 이에 대해 경록님은 어떤 기준을 갖고계신지가 궁금합니다!!

Rok93 · 2025-02-08T03:33:11Z

src/main/kotlin/blackjack/BlackjackGame.kt

 }

-private fun dealCardToGambler(
-    gambler: Gambler,
+fun placeBet(gamblers: List<Gambler>) {


BlackjackGame 클래스 내부에서만 사용도되는 메서드같은데요.
접근제한자를 private로 두면 좋을 것 같아요. 😃

Rok93 · 2025-02-08T03:37:55Z

src/main/kotlin/blackjack/ui/BlackjackPrinter.kt

-        return gamblerResults.joinToString(separator = LINE_FEED) { gamblerResult ->
-            val gambler = gamblerResult.gambler
-            "${gambler.name}: ${createWinOrDefeatMessage(gamblerResult)}"
+    fun printAllFinalProfit(blackjackResults: BlackjackResults) {


요구사항에 따르면 소숫점은 노출하지 않는 것으로 보이네요. 🤔

Rok93 · 2025-02-08T03:39:57Z

src/main/kotlin/blackjack/domain/Gambler.kt

-        return false
-    }
+    override val isDealer: Boolean = false
+    private var betAmount: Double = 0.0


(배팅) 금액 표현은 소수점의 부정확성을 고려하여 double 보다는 BigDecimal을 사용해보면 어떨까요? 🤔

베팅금액 또한 돈이고 정확해야 함을 인지하지 못했네요 😅 경록님 말씀대로 BigDecimal을 이용하는 편이 좋을 것 같습니다!

Rok93 · 2025-02-08T03:44:29Z

src/main/kotlin/blackjack/domain/ResultStatus.kt

+                dealer.isBlackjack() && gambler.isNotBlackjack() -> DEFEAT
+                dealer.isBurst() -> WIN
+                gambler.isBurst() -> DEFEAT
+                gambler.isScoreLargerThan(dealer) -> WIN
+                gambler.isScoreEqualTo(dealer) -> DRAW


ResultStatus는 gambler의 기준에서 측정되는 결과인듯하군요?

현실 세계에서는 딜러가 게임 참가자들의 승패를 판단해주는데요.
그에 착안해서 승패를 판단하는 역할을 dealer 객체에게 위임해보면 어떨까요? 🤔

Rok93 · 2025-02-08T03:47:02Z

src/main/kotlin/blackjack/domain/ProfitMultiplier.kt

+            ResultStatus.WIN -> if (isBlackjack) {
+                1.5
+            } else {
+                1.0
+            }
+
+            ResultStatus.DRAW -> 1.0
+            ResultStatus.DEFEAT -> -1.0


ResultStatus에 블랙잭으로 승리한 케이스를 추가하고 추가적으로 1.5, 1.0, -1.0 과 같은 배팅율(?)을 ResultStatus의 프로퍼티로 둬보면 어떨까요?

위와 같이 변경한다면 지금처럼 하드코딩을 하는 요소가 줄어들 것 같고 또, ProfitMultiplier도 제거가능할 것 같아서요. 😃

오 이런 좋은 방법이..... 바로 실행에 옮겼습니다!!

SeolYoungKim added 11 commits January 4, 2025 13:04

docs: 요구사항 작성

6f10926

feat: 블랙잭 여부를 판단해서 승패를 결정하는 로직 구현

5ca754f

feat: 겜블러가 베팅을 할 수 있도록 변경

c7293fc

feat: 겜블러가 배팅을 하는 로직 구현

059f376

refactor: ResultStatus를 판단하는 로직을 ResultStatus로 이동

9e698fa

feat: 게임 결과 금액계산 로직 구현

488ab0b

feat: 겜블러 결과 결정 기능 리팩토링

0060784

feat: 딜러 결과 결정 기능 리팩토링

26f2474

test: 올바르지 않은 테스트 코드 수정

fdd0a2a

fix: Dealer의 수익금 합산로직 수정

7563e4a

feat: 딜러, 겜블러의 수익금 입출력 로직 구현

79b5710

Rok93 requested changes Feb 8, 2025

View reviewed changes

SeolYoungKim added 6 commits February 9, 2025 19:04

docs: 구현된 요구사항 체크

9cf1aeb

refactor: 매직넘버 상수화

60d68b7

refactor: isDealer 필드를 isDealer() 함수로 변경

7564aa4

refactor: 코틀린에서 제공하는 unaryMinus()를 사용하도록 변경

5370b59

refactor: Double을 BigDecimal로 변경

3aec0b3

refactor: ResultStatus에 BLACKJACK_WIN을 추가

a338897

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 4단계 - 블랙잭(베팅) #826

🚀 4단계 - 블랙잭(베팅) #826

SeolYoungKim commented Jan 5, 2025

Rok93 left a comment

Rok93 Feb 8, 2025

Rok93 Feb 8, 2025

SeolYoungKim Feb 9, 2025

Rok93 Feb 8, 2025

Rok93 Feb 8, 2025

Rok93 Feb 8, 2025

SeolYoungKim Feb 9, 2025

Rok93 Feb 8, 2025

Rok93 Feb 8, 2025

Rok93 Feb 8, 2025

SeolYoungKim Feb 9, 2025

Rok93 Feb 8, 2025

Rok93 Feb 8, 2025

SeolYoungKim Feb 9, 2025

🚀 4단계 - 블랙잭(베팅) #826

Are you sure you want to change the base?

🚀 4단계 - 블랙잭(베팅) #826

Conversation

SeolYoungKim commented Jan 5, 2025

Rok93 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment