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

sjjeong · 2025-01-07T05:36:43Z

마지막 과제에서 배팅 금액과 수익을 어떻게 처리해야할지 고민이 있었습니다.
마지막까지 리뷰 해주셔서 감사합니다 🙏

기존에 승패가 나왔지만 이제는 최종 수익이 나오도록 함

Rok93

석준님 최근에 이사 준비로 정신이 없어서 리뷰가 늦었습니다. 🥲
4단계도 구현 잘 해주셨네요. 👍

몇몇 코멘트 남겨두었으니 확인 부탁드릴게요. 🙏

Rok93 · 2025-01-10T14:29:08Z

src/main/kotlin/blackjack/view/InputView.kt

+        participants.forEach { participant ->
+            println("${participant.name}의 배팅 금액은?")
+            val bettingMoney = readlnOrNull()?.toIntOrNull() ?: error("배팅은 ")
+            participant.bettingMoney = bettingMoney


Participant가 View에서 베팅을 하고있군요? 🤔
이 기능은 도메인 로직으로 볼 수 있지 않을까요? 😃

Rok93 · 2025-01-10T14:29:32Z

src/main/kotlin/blackjack/view/InputView.kt

+        participants.forEach { participant ->
+            println("${participant.name}의 배팅 금액은?")
+            val bettingMoney = readlnOrNull()?.toIntOrNull() ?: error("배팅은 ")
+            participant.bettingMoney = bettingMoney


Participant 객체에게 베팅을하라고 메시지를 보내보면 어떨까요? 😃

Rok93 · 2025-01-10T14:33:15Z

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

+
+    constructor(vararg participantNames: String): this(participantNames.map { name -> Participant(name.trim()) })
+
+    fun setupCard(blackjackShoe: BlackjackShoe) {


setupCard 라는 표현보다는 조금 더 (능동적인) 객체의 입장에서 행위를 하는 것 같은(?) 네이밍을 고려해보면 어떨까 싶어요.

참가자의 입장에서 초기 패를 뽑는다 와 같은 행위적 관점으로 표현하면

참가자.초기 패를 뽑는다 와 같이 표현하면 직관적으로 이해하기가 조금 더 쉽지 않을까요? 😃

Rok93 · 2025-01-10T14:37:33Z

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

@@ -0,0 +1,10 @@
+package blackjack.domain
+
+class Participants(private val participants: List<Participant>): List<Participant> by participants {


by 키워드 활용 👍💯

현재 Participants 일글컬렉션을 by 키워드를 활용해서 구현해주셨네요.
by 키워드를 사용했을 때의 장점과 단점을 알고 계신지 궁금합니다. :)

가볍게 코멘트로 남겨주시면 좋을 것 같아요. 😁

Rok93 · 2025-01-10T14:51:40Z

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

@@ -19,6 +19,10 @@ abstract class Player(
            return score
        }

+    var money: Int = 0


꼭 순서가 정해져있는 것은 아니지만, 개인적인 생각으로는 읽기전용 프로퍼티보다는 실제 Palyer의 상태값인 money가 더 상위에 있어야하지 않을까 싶어요. 🤔

Rok93 · 2025-01-10T14:52:58Z

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

@@ -3,12 +3,21 @@ package blackjack.domain
 data class Participant(
    override val name: String


name 프로퍼티는 오버라이드 할 필요 없이, 단순히 Player의 생성자 파라미터로 전달만해주면 될 것 같은데요.
오버라이드하신 이유가 있으실까요? 😃

Rok93 · 2025-01-10T15:07:12Z

src/main/kotlin/blackjack/BlackjackApplication.kt


        outputView.showReady(dealer = dealer, participants = participants)
        return dealer to participants
    }

-    private fun play(dealer: Dealer, participantList: List<Participant>, blackjackShoe: BlackjackShoe) {
-        participantList.forEach { participant ->
+    private fun play(dealer: Dealer, participants: Participants, blackjackShoe: BlackjackShoe) {


ready()나 play()의 경우 dealer�, participants를 프로퍼티로 가진 BlackjackGame 객체에게 역할을 위임해볼 수도 있을 것 같아요. 😃
물론 중간에 입력받는 부분들이 있어서, 조금 어렵게 느껴지실 수도 있을 것 같네요.

이 방향으로 한번 고민해보시면 좋을 것 같아요!

sjjeong added 4 commits January 2, 2025 14:37

refactor: List<Participant>를 일급 컬렉션으로 만들기

165ad5a

feat: 21점이 초과하는 Bust 케이스 추가

a5954de

feat: Participant가 betting 금액을 받을 수 있도록 추가

129b572

feat: BlackjackResults가 최종 수익을 출력하도록 변경

6f81637

기존에 승패가 나왔지만 이제는 최종 수익이 나오도록 함

Rok93 requested changes Jan 10, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

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

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

sjjeong commented Jan 7, 2025

Rok93 left a comment

Rok93 Jan 10, 2025

Rok93 Jan 10, 2025

Rok93 Jan 10, 2025

Rok93 Jan 10, 2025

Rok93 Jan 10, 2025

Rok93 Jan 10, 2025

Rok93 Jan 10, 2025


		constructor(vararg participantNames: String): this(participantNames.map { name -> Participant(name.trim()) })

		fun setupCard(blackjackShoe: BlackjackShoe) {

		@@ -0,0 +1,10 @@
		package blackjack.domain

		class Participants(private val participants: List<Participant>): List<Participant> by participants {

		@@ -3,12 +3,21 @@ package blackjack.domain
		data class Participant(
		override val name: String

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

Are you sure you want to change the base?

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

Conversation

sjjeong commented Jan 7, 2025

Rok93 left a comment

Choose a reason for hiding this comment

Rok93 Jan 10, 2025

Choose a reason for hiding this comment

Rok93 Jan 10, 2025

Choose a reason for hiding this comment

Rok93 Jan 10, 2025

Choose a reason for hiding this comment

Rok93 Jan 10, 2025

Choose a reason for hiding this comment

Rok93 Jan 10, 2025

Choose a reason for hiding this comment

Rok93 Jan 10, 2025

Choose a reason for hiding this comment

Rok93 Jan 10, 2025

Choose a reason for hiding this comment