[둘리] 1단계 블랙잭 제출합니다. #21
Conversation
…Player pick 함수 추가
| fun `카드의 합이 16을 초과하지 않으면 hit 한다`() { | ||
| val dealer = Dealer( | ||
| Cards( | ||
| listOf( | ||
| Card(Rank.JACK, Suit.DIAMOND), | ||
| Card(Rank.SIX, Suit.CLOVER), | ||
| ), | ||
| ), | ||
| ) | ||
| val dealer = hitDealer() | ||
| assertThat(dealer.isHit()).isTrue | ||
| } | ||
|
|
||
| @Test | ||
| fun `카드의 합이 16을 초과하면 stay 한다`() { | ||
| val dealer = Dealer( | ||
| Cards( | ||
| listOf( | ||
| Card(Rank.JACK, Suit.DIAMOND), | ||
| Card(Rank.SEVEN, Suit.CLOVER), | ||
| ), | ||
| ), | ||
| ) | ||
| val dealer = stayDealer() | ||
| assertThat(dealer.isHit()).isFalse | ||
| } |
There was a problem hiding this comment.
이 테스트 케이스에서 중요한 정보는 Rank입니다.
테스트 케이스를 처음 보는 사람이 테스트 본문만 보아도 합이 16이 넘거나 넘지 않았을 때, hit or stay 한다는 것을 알 수 있어야 합니다.
하지만 이미 함수 이름에서부터 hitDealer(), stayDealer()라고 확정되어있고 어떤 때에 hit 되는지는 테스트 본문을 떠나서 해당 함수를 들여다 보아야합니다.
dealer(rank1, rank2, ...) 형태로 만들어보는건 어떨까요?
| private const val DIAMOND = "다이아몬드" | ||
| private const val CLOVER = "클로버" | ||
| private const val SPADE = "스페이드" | ||
| private const val HEART = "하트" | ||
| private const val ACE = "A" | ||
| private const val KING = "K" | ||
| private const val JACK = "J" | ||
| private const val QUEEN = "Q" |
There was a problem hiding this comment.
조금 더 고도화해본다면 ui layer에서만 사용하는 SuitModel을 만들고 해당 클래스에서 UI를 제공하도록 만드는 방법도 있습니다.
이 방식은 이번 미션에서는 적용하지 않아도 좋지만 추후 안드로이드에서 자주 쓰이는 방식이니 미리 인지해두시면 좋겠어요.
| private lateinit var game: CardGame | ||
| @BeforeEach | ||
| fun setUp() { | ||
| game = CardGame(CardPackGenerator().createCardPack()) | ||
| } |
There was a problem hiding this comment.
참고로 junit4, junit5에서는 각 테스트 케이스를 수행하기 전에 테스트 클래스를 새로 만드는게 기본 설정이라 실제로는 @BeforeEach도 필요없습니다.
하지만 가독성과 추후에 필요해지는 경우도 생기므로 지금처럼 초기화 코드를 분리해주시는게 좋습니다.
만약 필요하다면 @TestInstance(Lifecycle.PER_CLASS)를 통해 한 번만 인스턴스가 만들어지도록 할 수도 있습니다.
| fun pick(cardPack: Cards) { | ||
| cards.add(cardPack.pop()) | ||
| } |
| fun pop(): Card { | ||
| require(_cards.isNotEmpty()) { OUT_OF_INDEX_CARDS_CURSOR } | ||
| val card = _cards[0] | ||
| _cards.removeAt(0) | ||
| return card | ||
| } |
There was a problem hiding this comment.
LinkedList와 같이 이미 잘 만들어져있는 자료구조를 활용해봐도 좋겠네요.
안녕하세요, 리뷰어님!! 많이 배우겠습니다. 피드백 부탁드립니다! :)