[블랙잭 1단계] 뭉치 미션 제출합니다. #122

m6z1 · 2025-03-07T08:21:07Z

안녕하세요 둘리!
모바일 안드로이드 크루 뭉치입니다 ㅎㅎ
이렇게 만나뵙게 되어 영광입니다 😄🍀

블랙잭 미션을 돌아가게하자! 라는 부분에 초점을 두어, 코드가 질서정연하지 않고 가독성이 떨어지는 것 같습니다..
따끔하게 피드백 주시면 감사하겠습니다!!

이번 블랙잭 미션 잘 부탁드려요!!🙇‍♀️

셀프 체크리스트

프로그램이 정상적으로 작동하는가?
모든 테스트가 통과하는가?
이전에 받은 피드백을 모두 반영하였는가?
코딩 스타일 가이드를 준수하였는가?
- IDE 코드 자동 정렬을 적용하였는가?
- 린트 검사를 통과하였는가?
프로그래밍 요구 사항을 준수하였는가?
README.md에 기능 목록을 정리하고 명확히 기술하였는가?

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

1. 모든 객체를 설계하고 컨트롤러 및 뷰를 생성하다보니 생각보다 컨트롤러가 하는 역할이 많다? 라는 생각이 들었습니다.

저는 기존에 controller에서 객체를 생성해도 되며, 서비스의 흐름을 가지는 것이 컨트롤러의 역할이라고 생각해 블랙잭 게임의 흐름을 모두 컨트롤러에서 구현했습니다. 하지만, 이번 블랙잭 미션에서 느낀 문제점이 있었습니다.

단위 테스트만 진행하다보니 여러 가지의 분기에 대해서 테스트를 할 수 없다는 문제점을 깨달았습니다. 특히나 블랙잭 미션의 경우에는 딜러가 ~하면, 플레이어가 ~한 경우의 분기가 굉장히 많다보니 실제로 그 플로우를 테스트하기까지 application을 계속 돌려야 하는 문제가 있었습니다.

그렇다면 블랙잭 게임을 주관하는 새로운 도메인 객체를 만들어야 할까? 라는 생각도 들었는데 이것에 대해서는 어찌 생각하시는지 궁금합니다!
이번 피드백에서 만들어보라고 하시면 반영하고 둘의 장단점을 느껴보도록 하겠습니다!! ㅠㅠ

다른 내용은 코멘트로 달겠습니다!

- 카드 모양 추가

- 카드는 모양과 끗수를 가진다.

- 카드의 끗수에 대한 요구 사항 추가

- 응답에 대한 플레이어 행동 반환

…kjack into step1

- 이름, 카드 리스트 가짐

- 기존) List<Card> - 변경) Cards

- 딜러의 hit 과 stay 로직 제외 구현

m6z1

안녕하세요 둘리!

고민이 많았던 부분에 대해서 코멘트를 남겨두었습니다.

이번 미션 잘 부탁드립니다 : )

m6z1 · 2025-03-07T08:39:16Z

src/main/kotlin/blackjack/model/Cards.kt

+package blackjack.model
+
+class Cards(
+    value: List<Card>,


기존의 cards의 생성자 네이밍은 cards 였습니다.
일급컬렉션을 만들어 외부에서 사용하게 되면 , List<Card>에 접근하기 위해서는
cards.cards.~~로 사용을 했어야 했는데, 미관상으로도 그렇고 카드들의 카드들이라는 이름이 어색하다고 느꼈습니다.
이에 value라는 네이밍을 사용했는데, 조금 추상적인가? 라는 생각도 들었습니다.
둘리는 어떻게 생각하시는지 궁금합니다..!!

취향차이라고 생각하는데, 저는 cards.cards를 더 많이 사용하긴 합니다. 정답은 없어요!

m6z1 · 2025-03-07T08:44:29Z

src/main/kotlin/blackjack/model/Card.kt

+) {
+    fun isDenominationAce(card: Card): Boolean = card.denomination == Denomination.ACE
+
+    fun combine(): String = denomination.title + shape.koreanName


combine이란 함수가 String 을 반환하는데, 다시 생각해보니 view에서 결합하여 출력을 하는 게 더 적절할 것이라는 생각이 들었습니다.

A다이아몬드에서 다이아몬드A로 출력 요구 사항이 변경될 경우, model이 아닌 view에서 변경을 해야 하지 않나?
하지만, 반대로 생각하면 card에서 메시지를 받아 반환 할 수 있는 것은 아닐까? 라는 생각이 들었습니다...

고민의 끝에 현재 코드는 card 에서 직접 제작해서 제출하고 있습니다.

combine 함수의 로직은 뷰의 관심사가 맞습니다.
그럼 이렇게 생각해보면 어떨까요?

이 도메인 로직을 여러 플랫폼에서 사용한다

combine() 함수로 뷰에 출력할 String값을 받는다.

A플랫폼에서는 A다이아몬드, B플랫폼에서는 B다이아몬드 라고 출력해야한다.

뷰의 요구사항이 바뀌면 뷰를 수정하는 것이 자연스럽지 않을까요?

m6z1 · 2025-03-07T08:48:52Z

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

현재 Dealer는 Player를 상속받고 있습니다.
하지만, dealer가 아닌 player들은 딜러 라는 이름을 가질 수 없다고 판단했습니다.

하지만, init을 통해 이름에 대한 검증을 진행하게 되면, 딜러 또한 이 에러에 막히는 것을 깨달았습니다.

그래서 dealer 가 아닌 player의 경우에는 팩토리 함수를 통해 객체를 생성하며 이름을 검증할 수 있도록 했고,
dealer의 경우에는 player의 생성자를 통해 직접적으로 객체를 생성할 수 있도록 했습니다.

하지만, 생성자는 public 으로 열려있어서 누구든지 Player("이름")을 통해 객체를 생성할 수 있습니다.
dealer 가 아닌 player 는 생성자로 접근해 Player 객체를 만들 수 있으니 그렇다면 팩토리 메서드가 의미가 있는 것일까? 라는 생각이 들었습니다.

어떻게 문제를 해결해야 할 지 방향성을 제시해주시면 감사하겠습니다 ㅠㅠ!!

생성자가 public으로 열려있더라도, 생성자 외에 다른 의도를 주고 싶다면 팩토리 함수를 사용해도 이상하지 않다고 생각해요.
하지만 요구사항에 대한 검증은 init에 두는 것이 자연스럽다고 생각합니다. 말씀하신대로 누구든지 Player("이름")을 통해 객체를 생성할 수 있기 때문이에요.

플레이어와 딜러에게 다른 요구사항을 검증하고 싶다면, 딜러가 플레이어를 상속하는 것이 아니라
딜러-플레이어를 공통화할 수 있는 새로운 상위 클래스를 생성하고, 딜러와 플레이어가 그 클래스를 상속받는 형태는 어떨까요?

추가로 플레이어 이름이 겹치면 안된다는 요구사항은 없는데요. 🤔 다른 이야기일 수 있겠지만 동명이인이라면요..?🥹

hyemdooly

안녕하세요, 뭉치! 이번 미션을 함께 하게 된 둘리입니다. 반가워요! 😄
객체를 많이 나눠주셨어요! 여기서 좀 더 어떤 객체가 어떤 정보를 가지고 있어야 할지, 어떤 행위를 해주어야할지 고민해보면 좋겠어요.
질문에 대한 답과 함께 고민해볼만한 내용들을 코멘트로 달아드렸습니다.
더 궁금한 점이 있으시면 코멘트 달아주시거나 DM 주세요! 💪

hyemdooly · 2025-03-08T08:14:55Z

src/main/kotlin/blackjack/controller/BlackjackController.kt

+import blackjack.view.InputView
+import blackjack.view.OutputView
+
+class BlackjackController(


딜러가 ~하면, 플레이어가 ~한 경우의 분기가 굉장히 많다보니

혹시 이런 경우의 테스트가 어떤 것이 있을까요?
보통은 단위 테스트면 충분하다고 생각하고, 전제조건을 초기값으로 제공해서 테스트하는 함수들이 지금도 보여서요!

별개로 지금도 컨트롤러가 하고있는게 많아보이긴하는데... 우선 다른 것에 집중해서 수정해보고, 그 이후에 컨트롤러를 살펴보죠!

hyemdooly · 2025-03-08T08:18:19Z

src/main/kotlin/blackjack/model/Card.kt

+) {
+    fun isDenominationAce(card: Card): Boolean = card.denomination == Denomination.ACE
+
+    fun combine(): String = denomination.title + shape.koreanName


combine 함수의 로직은 뷰의 관심사가 맞습니다.
그럼 이렇게 생각해보면 어떨까요?

이 도메인 로직을 여러 플랫폼에서 사용한다

combine() 함수로 뷰에 출력할 String값을 받는다.

A플랫폼에서는 A다이아몬드, B플랫폼에서는 B다이아몬드 라고 출력해야한다.

뷰의 요구사항이 바뀌면 뷰를 수정하는 것이 자연스럽지 않을까요?

hyemdooly · 2025-03-08T08:23:19Z

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

생성자가 public으로 열려있더라도, 생성자 외에 다른 의도를 주고 싶다면 팩토리 함수를 사용해도 이상하지 않다고 생각해요.
하지만 요구사항에 대한 검증은 init에 두는 것이 자연스럽다고 생각합니다. 말씀하신대로 누구든지 Player("이름")을 통해 객체를 생성할 수 있기 때문이에요.

플레이어와 딜러에게 다른 요구사항을 검증하고 싶다면, 딜러가 플레이어를 상속하는 것이 아니라
딜러-플레이어를 공통화할 수 있는 새로운 상위 클래스를 생성하고, 딜러와 플레이어가 그 클래스를 상속받는 형태는 어떨까요?

추가로 플레이어 이름이 겹치면 안된다는 요구사항은 없는데요. 🤔 다른 이야기일 수 있겠지만 동명이인이라면요..?🥹

hyemdooly · 2025-03-08T08:24:58Z

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

+    open fun appendCard(card: Card) {
+        cards.add(card)
+    }


Controller에서 deck에서 카드를 뽑아서 주지말고, player가 직접 deck에서 카드를 뽑는다면 어떨까요?

hyemdooly · 2025-03-08T08:26:19Z

src/main/kotlin/blackjack/model/Cards.kt

+    fun calculateScore(): Int {
+        val aceCount: Int = value.count { card -> card.isDenominationAce(card) }
+        val score: Int = value.sumOf { card -> card.denomination.number }
+        return when (aceCount) {
+            1 -> if (score < 11) score + 11 else score + 1
+            2 -> if (score < 10) score + 11 + 1 else score + aceCount * 1
+            3 -> if (score < 9) score + 11 + 1 + 1 else score + aceCount * 1
+            4 -> if (score < 8) score + 11 + 1 + 1 + 1 else score + aceCount * 1
+            else -> score
+        }
+    }


이 계산 로직이 이렇게 길지 않아도 해결할 수 있을 것 같아요. 🤔

hyemdooly · 2025-03-08T08:27:41Z

src/main/kotlin/blackjack/model/Cards.kt

+    private val _value: MutableList<Card> = value.toMutableList()
+    val value: List<Card> get() = _value.map { card -> card.copy() }
+
+    var size: Int = this.value.size


이 변수는 테스트를 위함일까요? 🤔

hyemdooly · 2025-03-08T08:30:03Z

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

+    open val name: String,
+    open val cards: Cards = Cards(emptyList()),
+) {
+    lateinit var result: GameResult


사실 저는 이 result가 왜있는지 잘 모르겠어요!
result는 결과 계산이 필요할 때만 계산해서 보여주면 될 것 같은데, 게임 진행 내내 갱신을 하고 있더라구요.
진행 로직 정리하면서 이 result가 꼭 필요한지 고민해보면 좋겠어요.

hyemdooly · 2025-03-08T08:31:29Z

src/main/kotlin/blackjack/model/PlayerBehavior.kt

+                "Y" -> HIT
+                "N" -> STAY


어차피 하드코딩된 "Y"와 "N"에 따라 결과값을 리턴할거라면 enum 내의 answer는 정의하지 않아도 괜찮아보여요.
파라미터 answer가 지금 String인데, 진행 여부를 Boolean으로 받아도 충분하지 않을까요?

hyemdooly · 2025-03-08T08:34:09Z

src/main/kotlin/blackjack/model/Cards.kt

+    fun isBlackjack(firstTurn: Boolean): Boolean = CardsStatus.from(calculateScore(), firstTurn) == CardsStatus.BLACKJACK
+
+    fun isBust(): Boolean = CardsStatus.from(calculateScore()) == CardsStatus.BUST


상태를 활용하겠다면, Cards가 상태를 가지고, 카드가 추가될 때마다 상태를 업데이트하는 것이 자연스럽지 않을까요?

hyemdooly · 2025-03-08T08:35:44Z

src/main/kotlin/blackjack/model/CardsMaker.kt

+    private val DENOMINATIONS: List<Denomination> = Denomination.entries
+    private val SHAPES: List<CardShape> = CardShape.entries
+
+    val CARDS: List<Card> =
+        DENOMINATIONS
+            .flatMap { denomination ->
+                SHAPES.map { shape ->
+                    Card(shape, denomination)
+                }
+            }.shuffled()


Kotlin에서 이렇게 전부 대문자로 이름을 짓는 경우는 어떤 경우일까요?

hyemdooly · 2025-03-08T08:41:05Z

추가로 전체적인 블랙잭 로직을 확인해보면 좋겠어요.

딜러카드: 3다이아몬드, A하트, Q다이아몬드, 2하트, 3스페이드 - 결과: 19
a카드: A클로버, 5클로버, K다이아몬드 - 결과: 16
b카드: 5하트, 10클로버 - 결과: 15
c카드: 8하트, 3하트 - 결과: 11

## 최종 승패
딜러: 3무
a: 무
b: 무
c: 무

위 경우는 딜러가 이겨야합니다. 딜러가 Bust가 아니면서 21 숫자에 가깝기 때문이에요.

제가 아는 블랙잭 규칙으로는 플레이어들이 모두 카드뽑기가 끝나면 딜러가 카드를 더 뽑는 것으로 알고 있는데요. 지금은 한 명의 플레이어가 카드를 뽑으면 딜러가 뽑고 있어요.
사실 딜러가 먼저 Bust해버린다면 플레이어들은 카드를 더 뽑을 필요 없습니다. 아직 Bust하지 않은 모든 플레이어가 승리하기 때문이에요.

전체적으로 로직을 한번 체크해보시면 좋을 것 같아요! 객체 간의 역할과 책임을 고민하다보면 컨트롤러 로직도 좀 더 단순화되지 않을까 싶네용 💪

wondroid-world and others added 30 commits March 4, 2025 17:00

docs: 기능 목록 작성

d9f4ea4

docs: 기능 요구 사항 추가

c9e3b8a

- 카드 모양 추가

feat: 카드 모양 객체 구현

2df5152

feat: 카드 객체 구현

7ceb108

- 카드는 모양과 끗수를 가진다.

docs: 기능 요구 사항 추가

d3d99c2

- 카드의 끗수에 대한 요구 사항 추가

feat: 카드 끗수를 숫자로 변환하는 로직 구현

5d8b925

feat: 플레이어 행동 객체 구현

6d8e4d1

- 응답에 대한 플레이어 행동 반환

Merge branch 'step1' of https://github.com/wondroid-world/kotlin-blac…

e902fdf

…kjack into step1

docs: 플레이어 상태 기능 목록 수정

7fe01d4

feat: 플레이어 객체 구현

61ed0d5

feat: 플레이어 카드 추가 기능 구현

9c24b94

docs: 플레이어 기능 구현 추가

90a4195

feat: 플레이어 이름만 가질 경우, 비어있는 카드 리스트를 가짐

5b79da7

feat: 게임 결과 상태 객체 구현

47dc2ec

docs: 게임 결과 기능 목록 수정

3557ebc

feat: 기준 점수와 비교 점수를 비교하여 기준 점수 기준으로 게임 결과 반환 로직 구현

48d8577

feat: 딜러 객체 구현

48e2386

- 이름, 카드 리스트 가짐

docs: 딜러 이름 기능 목록 추가

ea6300e

feat: 딜러 이름의 초기값은 '딜러'

e55f6e2

docs: 딜러 카드 리스트 기능 목록 추가

a4a29b4

feat: 딜러 카드 리스트 초기값 구현

a0ccfcc

feat: 딜러 카드 추가 기능 구현

8147bce

docs: 카드들 상태 기능 목록 수정

bbce347

feat: 게임 진행 중 카드들의 상태 구현

fd9af80

docs: 끗수 기능 목록 추가

827acb6

feat: 끗수 객체 추가

16be104

docs: 카드 덱 기능 목록 추가

e44a767

feat: 카드 생성기 객체 구현

9100d77

docs: 카드 덱 기능 목록 수정

fa8bedd

feat: 카드 덱 객체 구현

1a91b51

m6z1 and others added 26 commits March 6, 2025 15:34

feat: 카드 점수 계산 로직 구현

fa7d4c4

refactor: 플레이어와 딜러의 카드들 타입 변경 및 카드 추가 로직 수정

d2bb180

- 기존) List<Card> - 변경) Cards

docs: 플레이어 행동 기능 목록 수정

eb8b486

feat: 딜러 점수에 따른 플레이어 행동 반환 로직 구현

d395b56

feat: 카드 상태 반환 로직 구현

bb1979e

feat: 플레이어 카드 입출력 로직 구현

701086a

feat: dsl 실습 구현

18f3c58

docs: DSL 노트 정리 수정

5aa7ecd

feat: 입출력 구현

feda5e3

feat: 카드들의 블랙잭, 버스트 반환 로직 추가

5390252

feat: 딜러 결과 반환 및 업데이트 로직 추가

14d5ccb

feat: 게임 결과의 반댓값 반환 로직 구현

dd2cd76

feat: 플레이어의 블랙잭 여부 및 결과 업데이트 로직 추가

60ff224

feat: 플레이어 이름 검증 로직 추가

b4a8fbc

feat: 블랙잭 로직 구현

2cce28d

- 딜러의 hit 과 stay 로직 제외 구현

feat: 딜러의 점수 분기 로직 추가

a60f022

feat: 플레이어 이름 검증 로직 추가

3ff8699

refactor: 컨틀롤러 내부의 함수 순서 변경

196df71

refactor: 사용하지 않는 생성자 제거

05980db

refactor: 카드들의 파라미터 네이밍 변경

9024c63

test: 카드들 테스트 추가

09916bb

refactor: 플레이어에서 bust, blackjack 판단하도록 수정

69e1097

test: 카드 테스트 추가

e1779e9

test: 딜러 테스트 추가

87b91af

test: 게임 결과 테스트 추가

08cba24

test: 플레이어 테스트 추가

70c7164

m6z1 changed the base branch from main to m6z1 March 7, 2025 08:40

m6z1 commented Mar 7, 2025

View reviewed changes

hyemdooly requested changes Mar 8, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[블랙잭 1단계] 뭉치 미션 제출합니다. #122

[블랙잭 1단계] 뭉치 미션 제출합니다. #122

m6z1 commented Mar 7, 2025 •

edited

Loading

m6z1 left a comment

m6z1 Mar 7, 2025

hyemdooly Mar 8, 2025

m6z1 Mar 7, 2025

hyemdooly Mar 8, 2025

m6z1 Mar 7, 2025

hyemdooly Mar 8, 2025

hyemdooly left a comment

hyemdooly Mar 8, 2025

hyemdooly Mar 8, 2025

hyemdooly Mar 8, 2025

hyemdooly Mar 8, 2025

hyemdooly Mar 8, 2025

hyemdooly Mar 8, 2025

hyemdooly Mar 8, 2025

hyemdooly Mar 8, 2025

hyemdooly Mar 8, 2025

hyemdooly Mar 8, 2025

hyemdooly Mar 8, 2025

hyemdooly commented Mar 8, 2025

		fun isBlackjack(firstTurn: Boolean): Boolean = CardsStatus.from(calculateScore(), firstTurn) == CardsStatus.BLACKJACK

		fun isBust(): Boolean = CardsStatus.from(calculateScore()) == CardsStatus.BUST

[블랙잭 1단계] 뭉치 미션 제출합니다. #122

Are you sure you want to change the base?

[블랙잭 1단계] 뭉치 미션 제출합니다. #122

Conversation

m6z1 commented Mar 7, 2025 • edited Loading

셀프 체크리스트

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

1. 모든 객체를 설계하고 컨트롤러 및 뷰를 생성하다보니 생각보다 컨트롤러가 하는 역할이 많다? 라는 생각이 들었습니다.

m6z1 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

hyemdooly 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

hyemdooly commented Mar 8, 2025

m6z1 commented Mar 7, 2025 •

edited

Loading