[1단계 - 블랙잭 게임 실행] 재즈(함석명) 미션 제출합니다. #4

seokmyungham · 2024-03-08T09:46:41Z

현구막 안녕하세요~ 이번 블랙잭 게임 리뷰를 부탁드리게 된 재즈라고 합니다😊
세 번째 미션이라 이젠 조금 익숙해지려 하다가도, 사다리 미션보다 난이도가 확 올라간 느낌이라서 또 새롭네요 ㅎㅎ
객체의 역할이 다양해지니까 점점 더 재미있어지는 것 같아요 🔥

게임 로직을 구현하는 방법이 정말 다양할 것 같아서
리뷰 하시는 인지적 비용을 덜어드리고자.. 게임 흐름 명세와 함께 궁금한 부분들 질문드립니다!

블랙잭 게임 흐름

Dealer와 Player는 공통적으로 추상 클래스 BlackjackGamer를 상속 받습니다.
Dealer, Player 서로 카드를 받는 조건이 다르므로 각각 추상 메서드를 오버라이드합니다.
"카드 패"를 의미하는 Hand 객체에서 자신의 카드 점수를 계산하고, 딜러의 점수를 플레이어에게 넘겨주어 플레이어는 자신의 승, 패 여부를 결정합니다.
플레이어에 따른 승, 패 여부를 출력해야 하므로, Players에서 이름과 게임 결과를 매핑하여 Dto로 반환합니다.
플레이어들의 게임 결과를 Dealer 로 넘겨주어, 딜러의 게임 결과를 집계하고 Dto로 반환하도록 하였습니다.

미션을 진행하면서 궁금했던 점

출력 형식에 의존하는 Enum `CardNumber` `CardShape`?

이번 미션에서 카드 번호와 카드 문양을 상수로 활용하고자 Enum 클래스로 만들게 되었어요.
현재 두 Enum 모두 문자열 name을 필드로 가지고 있는데요, 페어와 저 모두 문자열 name은 지금 출력에 의존 하고 있는게 아닐까? 라는 생각을 가지고 있습니다.
출력 요구사항이 "J하트" 에서 "JACK하트"로 변경되면 도메인을 수정해야 하기도 하고, 도메인을 지키지 못하고 있다는 느낌이 큰 것 같아요..

페어와 의견을 나눠보면서 현재 Enum에서 문자열 필드를 제거하고, view 패키지에 Enum을 출력 형식으로 변환해줄 수 있는 Enum CardShapeToSymbol같은 중간 다리를 하나 추가하는 방법을 제가 제안해봤는데 서로 뭔가 확신이 없는 것 같습니다.

현재 CardNumber와 CardShape가 출력 형식에 의존하고 있는 것 같은지, 제가 위에 작성한 방법에 대해 어떻게 생각하시는지 현구막의 의견을 듣고 싶어요!!

Dto를 사용하는 것을 어떻게 생각하시는지, 사용한다면 Dto 생성 로직의 위치

저번 사다리 미션에서 Dto를 사용하면서 조금 번거롭기도 했고 코드를 가볍게 가져가고 싶어 다음 미션에는 Dto를 사용하지 말아야지!! 라고 다짐했는데 이번에도 Dto를 사용하게 되었어요.

도메인을 뷰에 노출시키지 않고 싶으면 컨트롤러에서 도메인의 필드 원시값을 꺼내서 넘겨도 되기는 하지만,
내가 원하는 값들만 모아서 전달할 수 있다는 장점과
Map<String, boolean> 처럼 결과를 넘기게 되었을 때, 이 코드를 처음 보는 개발자가 저 안의 엔트리 값이 뭘 의미하는지 해석하는 인지적 비용을 줄이고 Dto에 이름을 붙일 수 있다는 점이 매력적으로 다가왔던 것 같아요.
그리고 Dto를 사용하면서 getter도 거의 사용하지 않았던 점도 긍정적인 부분이었던 것 같습니다.

그런데 현재 Dto 생성 로직이 도메인 내에 존재하는데,
Dto개수가 늘어남에 따라 중요하지 않은 Dto 생성 로직이 불필요하게 많아지는 것 같다고 느껴요.

Dto 생성 로직을 Dto 내부로 옮기고 컨트롤러에서 getter를 사용해서 생성한다면 걱정을 해결할 수 있지만 지금도 100줄이나 넘는 컨트롤러 로직이 더 복잡해질 것을 우려해서 도메인 내부에서 생성하도록 결정하게 되었습니다.

은총알은 없고 트레이드 오프를 항상 고려해야 하지만
현재 제 코드는 미숙한 부분들이 너무 많다고 느끼고
이번 미션을 진행하면서 개선해야 할 부분들에 대해 현구막께 많은 것을 배우고 싶어요!! 😀

긴 글과 코드 읽어주셔서 감사하고
리뷰 기다리고 있겠습니다!!
감사합니다 😄

Co-authored-by: seokmyungham <[email protected]>

HaiSeong

컨트롤러 부분이 아주 깔끔하네요.
코맨트 조금 남겼습니다!

HaiSeong · 2024-03-09T03:34:31Z

src/main/java/blackjack/domain/gamer/Dealer.java

+	public DealerInitialHandDto convertDealerToInitialHandDto() {
+		String dealerName = getName().value();
+		Card first = getFirstCard();
+
+		return new DealerInitialHandDto(dealerName, first.convertCardToDto());
+	}


Entity를 DTO로 변환하는 로직이 Entity에 있으면 View의 출력 요구조건에 따라 Domain 코드가 바뀌게 된다고 생각합니다.

저도 기존에 동일한 생각을 가지고 있었는데, 확신이 없었던 것 같습니다.
처음에는 dto 변환 로직을 dto가 책임지도록 했다가, 컨트롤러 코드가 너무 길어져서 중간에 수정하게 되었어요.

그런데 이번 리뷰어분도 같은 부분을 지적해주셨고
계속 고민해본 결과, dto 변환 로직을 스스로 책임질 수 있도록 해서
dto가 변경되거나 삭제되어도 도메인은 영향을 받지 않도록 하는 게 맞는 것 같아요
수정해보겠습니다.

HaiSeong · 2024-03-09T04:06:51Z

src/test/java/blackjack/domain/card/DeckTest.java

+	void setUP() {
+		deck = new Deck();
+	}


UP 는 어떤 뜻인가요?
어떤 단어의 약자인가요?

테스트를 위한 객체를 공통으로 사용하기 위해 준비하다 set up 의 의미의 네이밍이었습니다!

HaiSeong · 2024-03-09T04:20:52Z

src/main/java/blackjack/domain/gamer/GameResult.java

+public enum GameResult {
+
+	WIN("승"),
+	LOSE("패");


지금은 별 문제가 없을것 같지만 2차 구현에서 무승부일때 배팅 금액을 돌려받는다는 구현사항이 있어 같이 생각해보면서 구현하면 좋을것 같습니다!

HaiSeong · 2024-03-09T04:26:15Z

src/main/java/blackjack/controller/BlackjackController.java

+	private String getCommand(Player player) {
+		String command = inputView.receiveCommand(player.getName().value());
+		if (HIT_COMMAND.equals(command) || STAND_COMMAND.equals(command)) {
+			return command;
+		}
+		throw new IllegalArgumentException(HIT_COMMAND + " 또는 " + STAND_COMMAND + "만 입력 가능합니다.");
+	}


view에 책임이 아닌가 라는 생각이 듭니다.

입력 키워드를 "y" -> "hit", "n" -> "stand" 로 바꿨을때,
이 부분(controller)이 수정될것 같습니다. 어떻게 생각하시나요

로키 의견에 전적으로 동의합니다.
수정해보겠습니다

HaiSeong · 2024-03-09T04:28:27Z

src/main/java/blackjack/controller/BlackjackController.java

+	private void distributeCardToPlayer(Deck deck, Player player) {
+		while (canDistribute(player)) {
+			player.addCard(deck.draw());
+			outputView.printPlayerHand(player.convertGamerToDto());
+		}
+	}
+
+	private boolean canDistribute(Player player) {
+		return player.canReceiveCard() && HIT_COMMAND.equals(getCommand(player));
+	}


Libienz

안녕하세요 재즈 6기 크루 리비입니다 🙌
의도가 잘 드러나는 코드 잘 보았습니다 😀

재즈와 얘기해보고 싶은 부분을 코멘트로 남겨보았어요 👍
이어지는 블랙잭 미션 같이 화이팅해봐요 💪🏻

Libienz · 2024-03-09T13:45:53Z

src/main/java/blackjack/controller/BlackjackController.java

+
+	private static final String HIT_COMMAND = "y";
+	private static final String STAND_COMMAND = "n";
+


enum으로 묶는다면 커맨드 관련 책임과 로직을 응집할 수 있을 것 같습니다 💪🏻
컨트롤러가 게임 흐름의 자세한 로직을 아는 것도 완화할 수 있을 것 같고요 🤔

맞습니다! 저도 리비 의견에 전적으로 동의해요.
처음에는 컨트롤러에서 게임 흐름을 처리하니까, 저렇게 사용하고 인지하지 못했던 것 같아요
의사소통을 위한 별도의 Enum을 view패키지 안에 생성하고 view와 controller 사이에서 전달 받도록 수정해야할 것 같습니다 😄

Libienz · 2024-03-09T13:47:15Z

src/main/java/blackjack/controller/BlackjackController.java

+
+	public BlackjackController() {
+		this.inputView = new InputView();
+		this.outputView = new OutputView();
+	}
+


생성자에서 어떤 객체를 주입받을지 fix하고 있네요
DI를 통해 유연성을 확보하는 것이 더 좋지 않을까요!

맞아요 프로젝트가 나중에 확장될 가능성을 생각해 본다면
지금 Application 클래스와 같은 외부에서 주입하도록 하는 것이 나을 수도 있을 것 같네요 😊

Libienz · 2024-03-09T13:56:12Z

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

+	public CardDto convertCardToDto() {
+		return new CardDto(cardNumber.getName(), cardShape.getName());
+	}


DTO를 운용하기로 결정하셨군요
DTO의 생성 방법이 3가지 있는 것 같아요

도메인에서 ToDto 구현하기

Dto에서 from 구현하기

Mapper 이용하기

더 자주 바뀌는 객체가 무엇인지 생각해보고 의존방향 결정해봐도 좋을 것 같습니다 😀

지금 코드 구조에 조금 문제가 있다는 걸 다른 방법들과 함께 간접적으로 알려주셨군요 !!

저도 도메인에서 ToDto 구현하기 와 Dto에서 from 구현하기 사이의 장점 단점들을 인지는 하고 있었는데
확신이 없었기에 이번 미션에서 실수를 저질렀던 것 같아요.

미션 리뷰어(현구막), 로키, 리비 다 같은 부분들을 지적해 주셨고
두 방법 사이에 트레이드 오프는 존재하지만, Dto가 변환로직을 스스로 책임을 지는게 맞다는 걸 크게 느끼게 되었어요.

Mapper 이용하는 방법은 들어만 보고 모르고 있었던 부분인데
이번 기회에 학습해봐야겠네요!! 리비 알려주셔서 감사해요~ 😊

Libienz · 2024-03-09T14:00:14Z

src/main/java/blackjack/domain/card/CardNumber.java

+
+	ACE("A", 11),
+	TWO("2", 2),
+	THREE("3", 3),
+	FOUR("4", 4),
+	FIVE("5", 5),
+	SIX("6", 6),
+	SEVEN("7", 7),
+	EIGHT("8", 8),
+	NINE("9", 9),
+	TEN("10", 10),
+	JACK("J", 10),
+	QUEEN("Q", 10),
+	KING("K", 10);
+


이 부분은 재즈의 의견을 물어보고 싶네요
CardNumber의 name은 뷰로직일까요? 🤔

먼저 저도 비슷한 방식으로 구현했습니다
CardNumber는 이름을 속성으로 가지고 이름을 가져다가 출력하는 것이 위화감 없어 보인다는 생각이에요 (자동차 경주에서 Car의 이름을 가져다가 출력하듯이)

다만 저는 리뷰어에게서 도메인 속 뷰로직이 존재한다 라는 코멘트를 받았습니다 😅

재즈는 어떻게 생각하시나요!

먼저 저도 비슷한 방식으로 구현했습니다
CardNumber는 이름을 속성으로 가지고 이름을 가져다가 출력하는 것이 위화감 없어 보인다는 생각이에요 (자동차 경주에서 Car의 이름을 가져다가 출력하듯이)

앗 ㅋㅋㅋ 저도 속으로 "이정도는 괜찮나~?...." 생각했었는데 리뷰어분께 같은 코멘트를 받았어요 😭
현재 출력을 위한 이름이 도메인에 포함되어 있는지(재즈)

근데 리뷰를 받고 보니 제 페어 리뷰어분은 또 다른 의견이신 것 같아서 신기했습니다
현재 출력을 위한 이름이 도메인에 포함되어 있는지(상돌)

사실 미션 중간에 조금 꺼림칙하기도 해서,
제가 페어에게 view 패키지 안에 출력으로 변환해줄 수 있는 Enum을 만들어보는 방법도 제안해봤었는데, 제 페어는 이정도 까지는 괜찮을 것 같다라는 의견이었고 그냥 그대로 진행하게 되었습니다.

제 개인적인 생각은 여기까지는 허용할 수 있다, 이건 안된다는 너무 주관적인 영역인 것 같고 모호하다고 생각해요
그래서 저는 이번 미션도 그렇고 앞으로의 미션도 모두 별도의 변환을 하게끔 설계하기로 다짐하게 되었는데
현재 리비는 어떻게 생각하고 계신가요~??

저도 별도의 변환이 필요하다고 생각하게 되었네요 👍

숫자들을 TWO처럼 영어로 출력되도록 바꿔주세요라는 요청을 받으면 어느 패키지를 열어볼까요? 라는 질문을 받은 적 있는데 저라면 뷰를 열어볼 것 같더군요 😂

여기에 설득되어 뷰 레이어에서 따로 변환하도록 할 것 같습니다.

Libienz · 2024-03-09T14:02:32Z

src/main/java/blackjack/domain/card/Deck.java

+
+public class Deck {
+
+	private final LinkedList<Card> cards;
+


Deck에서 LinkedList를 사용하신 이유가 궁금해요 💭

저두 궁금해요! Queue나 List를 사용하는 것이 내부 구현을 바꾸는데 좀 더 용이하지 않을까요?

맨 앞쪽(위쪽)의 카드를 가져온다라는 의미와 잘 매칭되는 메서드를 먼저 생각해봤는데
Queue 자료구조의 poll() 메서드가 가장 먼저 떠올랐던 것 같아요.
큐 자료구조 특성상 데이터 변경이 일어날 때 리스트보다 성능 상 유리한 점이 많이 와닿았던 것 같습니다.

그리고 Queue 자료구조가 특정 상황과 연관되는 구현체들이 많다고 개인적으로 생각해서
다형성을 포기하고 LinkedList만 사용해! 라고 뭔가 강제하게 된 느낌인데, 이게 좋은지는 아직 잘 모르겠네요.. ㅎㅎ

Libienz · 2024-03-09T14:05:55Z

src/main/java/blackjack/domain/gamer/BlackjackGamer.java

+
+	public void initCard(Deck deck) {
+		addCard(deck.draw());
+		addCard(deck.draw());
+	}
+


게이머는 시작 카드가 두장이다라는 규칙 자체를 상수로써 표현할 수 있을 것 같아요

initCard를 실행할 때 해당 상수를 이용하도록 하면 규칙의 변경에 유연하게 대처할 수 있을 것 같습니다 😀

오!! 미처 생각지 못했는데 괜찮은 방법인 것 같습니다 😄
바로 반영해보겠습니다!

Libienz · 2024-03-09T14:17:04Z

src/main/java/blackjack/domain/gamer/Hand.java

+	public int calculateScore() {
+		int sum = cards.stream()
+			.mapToInt(Card::getNumber)
+			.sum();
+
+		if (hasAce()) {
+			return adjustSumWithAce(sum);
+		}
+		return sum;
+	}


블랙잭 룰에 맞추어 점수를 계산하는 것은 Hand 도메인의 책임일까요 🤔
현재의 Hand는 일급 컬렉션 역할을 수행함과 동시에 게임의 룰(채점 방식)을 알고 있다고 생각합니다 😀

채점 방식이 변경되면 Hand가 변경되어야 하는데 이것에 위화감이 있다는 것이 저의 생각입니다.

그래서 저는 도메인을 순수히 유지하고 블랙잭 게임의 규칙 (ex 점수 계산 로직)등을 외부로 덜어내고자 노력했어요
다만 저도 어떤 것이 정답인지는 모르겠습니다 😂
(파생되는 코드가 많다고 리뷰어에게 코멘트를 받기도 함)

재즈의 의견 궁금합니다 🤔

앗 리비 말대로 그렇게 생각해 볼 수도 있겠네요~

저는 점수 계산하는 부분에 관해선
블랙잭 게임의 채점 방식은 오랜 시간이 지나도 거의 변하지 않는 룰 처럼 느껴진 부분과
List<Card> 의 정보를 가장 잘 알고있는 Hand가 수행하는게 자연스럽다고 생각했던 것 같아요.

채점 방식이 변경되면 Hand가 변경되어야 하는데 이것에 위화감이 있다는 것이 저의 생각입니다.

저도 맞는 생각이라고 생각해요 😄
근데 저는 블랙잭 게임이 아니라 룰이 매우 복잡하고 변경될 여지가 많을 때 별도의 점수를 계산하는 Rule이나 Score 객체를 만들 것 같습니다.

근데 이거는 제가 구조를 최대한 가볍게 가져가는 걸 좋아해서 그런 걸 수도 있는 것 같아요. 😂

근데 저는 블랙잭 게임이 아니라 룰이 매우 복잡하고 변경될 여지가 많을 때 별도의 점수를 계산하는 Rule이나 Score 객체를 만들 것 같습니다.

제가 제일 많이 고민하던 부분인데 재즈의 의견 합리적이네요 👍👍

저도 불필요한 확장을 의식하는 것보다 도메인 명세에 집중해야겠어요 💪🏻

Libienz · 2024-03-09T14:18:31Z

src/main/java/blackjack/dto/CardDto.java

+package blackjack.dto;
+
+public record CardDto(String cardNumber, String cardShape) {
+}


레코드 적극 활용하셨군요 😀

leegwichan

일단 우선적으로 눈에 보이는 부분만 짚어봤어
주로 내가 코드를 보면서 궁금했던 부분, 내 생각과 조금 달랐던 부분 등 위주로 리뷰 달았어,
내 생각과 다른 부분이 있다면 코멘트 달아줘.

leegwichan · 2024-03-10T04:41:41Z

README.md

+- [x] 딜러의 점수를 입력받아 승,패를 결정한다.
+    - [x] 플레이어의 점수가 21을 초과하면 딜러의 점수와 무관하게 패배한다.
+    - [x] 플레이어의 점수가 21 이하이고, 딜러와 동점인 경우 패배한다.
+    - [x] 플레이어의 점수가 21 이하이고, 딜러의 점수가 21을 초과하면 승리한다.
+    - [x] 플레이어와 딜러의 점수가 모두 21 이하이고, 딜러의 점수보다 크면 승리한다.


나도 무승부를 구현하지 않았는데, 따로 무승부를 구현하지 않은 이유가 있을까?

그냥 최대한 현재 요구사항에 충실하려고 노력했던 것 같아 😄
2단계 요구사항을 보지않고 진행했어서, 만약 2단계에 무승부가 없다면 굳이 에너지를 낭비하게 될 수도 있다는 생각이었어

leegwichan · 2024-03-10T04:43:12Z

src/main/java/blackjack/controller/BlackjackController.java

+	private static final String HIT_COMMAND = "y";
+	private static final String STAND_COMMAND = "n";


나는 해당 커멘드가 Controller보다 InputView에 있는게 더 좋을 것 같은데, 어떻게 생각해?

앗 별도의 명령어를 view 패키지의 Enum으로 사용하도록 수정했었는데
정작 컨트롤러는 수정하지 않았었네 😭
바로 수정!!

leegwichan · 2024-03-10T04:49:28Z

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

+public class Card {
+
+	private final CardShape cardShape;
+	private final CardNumber cardNumber;


CardShape, CardNumber보다 Shape, Number가 더 간결하게 표현할 수 있지 않을까?
Card안에 쓰였다는 것만으로도 카드의 모양, 숫자를 나타낼 수 있다고 생각해.

나는 간결보다는 명확한 걸 좋아해서 이게 낫다고 생각했어
Enum이 쓰이는 상황에서도 Shape.Heart 보다는 CardShape.Heart 가 더 직관적이라고 생각해

leegwichan · 2024-03-10T04:51:25Z

src/main/java/blackjack/domain/card/Deck.java

+
+public class Deck {
+
+	private final LinkedList<Card> cards;
+


저두 궁금해요! Queue나 List를 사용하는 것이 내부 구현을 바꾸는데 좀 더 용이하지 않을까요?

leegwichan · 2024-03-10T04:56:26Z

src/main/java/blackjack/domain/gamer/Name.java

+public record Name(String value) {
+
+	private static final int MAX_NAME_LENGTH = 5;
+
+	public Name {
+		if (value.isBlank() || value.length() > MAX_NAME_LENGTH) {
+			throw new IllegalArgumentException("이름의 길이는 최소 1글자부터 최대 " + MAX_NAME_LENGTH + "글자까지 가능합니다.");
+		}
+	}
+}


타칸의 Record 테코톡 중에 "도메인 객체는 Record를 사용하는 것이 좋지 않다"라고 했었는데,
재즈는 어떻게 생각해? (지금 이걸 보니까 이 정도는 괜찮나 싶기도...?)

leegwichan · 2024-03-10T04:59:20Z

src/main/java/blackjack/view/OutputView.java

+	public void printInitialHands(DealerInitialHandDto dealerInitialHandDto, List<GamerHandDto> playersInitialHandDto) {
+		System.out.printf("\n%s와 %s에게 2장을 나누었습니다.\n", dealerInitialHandDto.name(),
+			joinPlayerNames(playersInitialHandDto));
+
+		System.out.printf("%s 카드: %s\n", dealerInitialHandDto.name(), formatCardName(dealerInitialHandDto.firstCard()));
+		printAllPlayerHands(playersInitialHandDto);
+	}


개행 문자를 강조하는 편은 아닌데, printf()를 쓸거면 \n 대신 %n을 쓰는 건 어떨까?
https://ps3940.tistory.com/65

leegwichan · 2024-03-10T05:02:24Z

src/test/java/blackjack/domain/gamer/PlayersTest.java

+	@Test
+	@DisplayName("유효한 입력에 대해서는 예외가 발생되지 않는다.")
+	void validPlayerNamesTest() {
+		assertThatCode(() -> new Players(List.of("a", "b")))
+			.doesNotThrowAnyException();
+		assertThatCode(() -> new Players(List.of("a", "b", "c", "d", "e", "f", "g", "h")))
+			.doesNotThrowAnyException();
+	}


위의 코드가 assert 문을 통과하지 않으면 아래 코드가 assert문을 통과하는 지 한번에 알 수 없을 것 같아.
assertAll()로 묶어보는 건 어떨까?

leegwichan · 2024-03-10T05:04:53Z

src/main/java/blackjack/domain/gamer/BlackjackGamer.java

+	public void addCard(Card card) {
+		hand.add(card);
+	}
+
+	public Card getFirstCard() {
+		return hand.getFirstCard();
+	}
+
+	public int getScore() {
+		return hand.calculateScore();
+	}
+
+	public Name getName() {
+		return name;
+	}


하위 클래스에서 재정의해서 사용하지 않을 메서드는 final 키워드를 붙여서 관리해보는 건 어떨끼?
하위 클래스에서 실수로 재정의하는 것을 방지해줄 수 있을 것 같아.

pricelees and others added 29 commits March 5, 2024 20:27

docs(README): 기능 요구 사항 작성

cc6c638

Co-authored-by: seokmyungham <[email protected]>

feat: 카드 문양과 숫자에 대한 Enum 클래스 생성

9c89c6f

Co-authored-by: seokmyungham <[email protected]>

feat(Deck): 모든 카드를 가지는 Deck 클래스 및 테스트 구현

b7011cb

Co-authored-by: seokmyungham <[email protected]>

style(DeckTest): 중복 코드 제거

1ab946f

Co-authored-by: seokmyungham <[email protected]>

feat(BlackJackGamer): 딜러와 플레이어의 공통 추상 클래스 생성

54db712

Co-authored-by: seokmyungham <[email protected]>

feat(Dealer): BlackJackGamer를 상속받는 딜러 클래스 생성과 메서드 재정의

1432f2d

Co-authored-by: seokmyungham <[email protected]>

feat(Player): BlackJackGamer를 상속받는 플레이어 클래스 생성과 메서드 재정의

56af966

Co-authored-by: seokmyungham <[email protected]>

feat(Name): 문자열 이름을 포장하는 클래스 생성

af51dfa

Co-authored-by: seokmyungham <[email protected]>

feat(Players): Player의 일급 컬렉션 생성

59f51bc

Co-authored-by: seokmyungham <[email protected]>

refactor: 예외 메시지 추가 및 테스트 수정

e75ea3b

Co-authored-by: seokmyungham <[email protected]>

test(NameTest): 정상 입력에 대한 테스트 추가

939882c

Co-authored-by: seokmyungham <[email protected]>

test(PlayersTest): 정상 입력에 대한 테스트 추가

f050926

Co-authored-by: seokmyungham <[email protected]>

feat(Player): 딜러의 점수를 입력받아 플레이어의 승패 여부를 결정하는 기능 추가

9b0e50d

Co-authored-by: seokmyungham <[email protected]>

feat(Hand): 참여자의 카드 리스트에 대한 일급 컬렉션 생성

ae70681

Co-authored-by: seokmyungham <[email protected]>

refactor(BlackjackGamer): Hand를 사용하도록 수정

f6afbdc

Co-authored-by: seokmyungham <[email protected]>

refactor(Dealer, Player): 생성자 및 메서드 수정

8d1b095

Co-authored-by: seokmyungham <[email protected]>

feat(GameResult): 승,패를 표현하는 Enum 클래스 생성

69e7124

Co-authored-by: seokmyungham <[email protected]>

feat: DTO 생성 및 각 도메인에 DTO 생성 로직 추가

1a9df2a

Co-authored-by: seokmyungham <[email protected]>

refactor(Hand): 카드 숫자의 합계를 구하는 메서드 세분화 및 상수 추가

d74858c

Co-authored-by: seokmyungham <[email protected]>

style(Dealer): 상수 추가 및 dto 생성 메서드명 수정

9fd5c3d

Co-authored-by: seokmyungham <[email protected]>

style(Player): 매직 넘버 상수화

e0351cf

Co-authored-by: seokmyungham <[email protected]>

feat(Players): 전체 플레이어의 패를 초기화 하는 기능 및 dto 생성 기능 추가

da6b980

Co-authored-by: seokmyungham <[email protected]>

feat(GameResult): 패배인지 확인하는 기능 및 getter 추가

6f1ef7b

Co-authored-by: seokmyungham <[email protected]>

feat(InputView): 입력을 받아들이는 InputView 클래스 생성

23a4321

Co-authored-by: seokmyungham <[email protected]>

feat(OutputView): 출력을 담당하는 OutputView 클래스 생성

38f9367

Co-authored-by: seokmyungham <[email protected]>

feat(BlackjackController): 블랙잭 게임 흐름을 제어하는 컨트롤러 클래스 생성

c3443d3

Co-authored-by: seokmyungham <[email protected]>

style: 출력 요구사항에 맞춰 빈 줄 출력 포맷 수정

74bbc4c

Co-authored-by: seokmyungham <[email protected]>

feat(Application): 컨트롤러를 생성하고 호출하여 게임을 시작하는 클래스 생성

8250c40

Co-authored-by: seokmyungham <[email protected]>

docs(README): 기능 요구 사항 정리

1b7ead1

Co-authored-by: seokmyungham <[email protected]>

HaiSeong reviewed Mar 9, 2024

View reviewed changes

Libienz reviewed Mar 9, 2024

View reviewed changes

seokmyungham closed this Mar 9, 2024

seokmyungham deleted the step1 branch March 9, 2024 16:36

seokmyungham restored the step1 branch March 9, 2024 16:39

seokmyungham added 2 commits March 10, 2024 01:57

refactor(BlackjackController): 명령어 상수를 별도의 view Enum으로 분리 및 사용

094a0bf

refactor(Card): 출력에 의존적인 CardNumber, CardShape을 수정하고 view 패키지에 Enum 추가

9397927

seokmyungham reopened this Mar 10, 2024

leegwichan reviewed Mar 10, 2024

View reviewed changes

seokmyungham added 8 commits March 11, 2024 14:45

refactor(GameResult): 기존 출력에 의존적이던 필드 문자열을 제거 및 view enum으로 분리

12715f4

refactor: 기존 도메인 내에 존재하던 Dto 변환 로직을 Dto 내부로 이동 및 리팩토링

1375923

refactor(Deck): 테스트 목적의 List<Card>를 파라미터로 받는 생성자 개방

16372e1

test(CardTest): eqauls&hashCode 테스트 추가

93c8c4b

refactor(PlayersTest): 가독성 향상을 위해 테스트 코드 리팩토링

1ff95cb

test(PlayersTest): 경계 값 테스트를 위해 테스트케이스 수정

ec048b0

feat(Command): 사용자 입력 명령어를 별도의 Enum 클래스로 분리

6659db2

style(Players): getter 메서드의 위치를 맨 밑으로 이동

8f146b6


		private static final String HIT_COMMAND = "y";
		private static final String STAND_COMMAND = "n";

[1단계 - 블랙잭 게임 실행] 재즈(함석명) 미션 제출합니다. #4

Are you sure you want to change the base?

[1단계 - 블랙잭 게임 실행] 재즈(함석명) 미션 제출합니다. #4

Conversation

seokmyungham commented Mar 8, 2024

블랙잭 게임 흐름

미션을 진행하면서 궁금했던 점

출력 형식에 의존하는 Enum CardNumber CardShape?

Dto를 사용하는 것을 어떻게 생각하시는지, 사용한다면 Dto 생성 로직의 위치

HaiSeong 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

Libienz 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

seokmyungham Mar 10, 2024 • edited Loading

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

seokmyungham Mar 10, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

leegwichan 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

출력 형식에 의존하는 Enum `CardNumber` `CardShape`?

seokmyungham Mar 10, 2024 •

edited

Loading

seokmyungham Mar 10, 2024 •

edited

Loading