[김태준] 연료 주입, 블랙잭 - Step1 #35

kimtaejun97 · 2022-02-17T14:44:39Z

안녕하세요 성주님!! 😀
이번 미션인 연료 주입, 블랙잭 구현 해 보았습니다.
네이밍에 신경을 쓰면서 작성을 해보았는데 어떨지 모르겠네요!

불변객체가 되도록 신경쓰면서 작성하였습니다.
그러나 Deck 같은 경우는 불변으로 만들지 않았는데요, Deck을 불변으로 만들게 되면
Application단에서 카드를 뽑고, Deck에서 카드를 제거하는 부분이 밖으로 나가야해 좋아보이지 않았습니다.
이 부분에 대한 리뷰어님의 의견이 궁금합니다!!

먼저 리뷰 감사합니다!! 잘부탁드립니다. 🙇‍♂️

- 추상클래스 Car에 존재하는 메서드 오버라이딩

1. 정상적인 입력이 들어오는 경우 2. 빈 값이 들어오는 경우, 예외 발생 3. 입력 형식에 유효하지 않은 경우, 예외 발생

1. Avante 반환 확인 2. K5 반환 확인 3. Sonata 반환 확인 4. 해당되는 이름이 없는 경우, exception 발생

1. 클래스 이름 변경 2. 메서드 변경 적절한 이름으로 변경

1. Participant 2. Player 3. Dealer

1. 카드 한 장 분배 2. 참가자의 초기 카드 분배

1. 전체 카드를 관리하는 덱 생성 2. 카드 분배 3. 참가자들에게 초기 카드 분배

- 불필요한 Wrapping 제거

1. Participant#getCardNames() 2. Hands#getCardNames()

1. 참가자 카드 출력

1. Deck > Cards 2. Game > Deck

1. 플레이어 : 21을 넘지 않는 경우, 의사를 물어봄 2. 딜러 : 16을 넘지 않는다면, 무조건 받음

승패의 경우의 수 테스트

Hands -> List<Card>로 변경

- 요구 사항 변경. - 딜러는 처음에 한장의 카드만을 보여준다.

- 플레이어가 21점이면 패배한 것으로 간주되는 버그, 21이 최대 점수가 되도록 수정.

liquidjoo

안녕하세요 이번 블랙잭 리뷰어를 맡은 김성주 입니다
미션 구현 잘해주셨어요 👍
몇 가지 코멘트 남겼어요 확인하고 다시 요청 주세요 🙇

liquidjoo · 2022-02-18T16:21:12Z

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

+    public Cards() {
+        this(Arrays.stream(Pattern.values())
+            .flatMap(pattern ->
+                Arrays.stream(Rank.values())
+                    .map(rank -> new Card(pattern, rank))
+                    .collect(Collectors.toList()).stream())
+            .collect(Collectors.toList()));
+    }


만약 게임이 한판만에 끝나는게 아니라 계속 게임을 다시 할 수 있다면
Card 의 객체는 계속 생성될 것 같아요
객체의 생성을 어떻게 줄여볼 수 있을까요?

liquidjoo · 2022-02-18T16:22:34Z

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

+    public Card draw() {
+        checkDeckIsEmpty();
+        Collections.shuffle(cards);
+        return cards.get(SELECTED_CARD);
+    }


랜덤적인 요소가 강해 테스트하기 어려운 메서드를 어떻게 테스트를 진행해 볼 수 있을까요?

상태 값 사이즈 확인은 제외

liquidjoo · 2022-02-18T16:23:20Z

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

+    public Card draw() {
+        final Card card = cards.draw();
+        cards.remove(card);
+        return card;
+    }


해당 메서드의 테스트를 어떻게 해볼 수 있을까요?

liquidjoo · 2022-02-18T16:23:52Z

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

+import java.util.List;
+import java.util.stream.Collectors;
+
+public class Hands {


카드를 관리하기 위한 객체 👍

liquidjoo · 2022-02-18T16:24:17Z

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

+    private static final int ACE_VALUE_1 = 1;
+    private static final int ACE_VALUE_11 = 11;
+
+    private final List<Card> cards;


Cards 라는 일급컬렉션이 있는데 활용해보면 좋지 않을까요?

liquidjoo · 2022-02-21T04:48:33Z

src/main/java/blackjack/domain/judge/Judge.java

+    private static final String PLAYER_LOSE = "패";
+    private static final String DEALER_WIN_OR_LOSE_FORMAT = "%d승 %d패";
+
+    public Map<String, String> getWinOrLose(final Participant dealer, final List<Participant> players) {


Map 을 일급컬렉션으로 활용해보면 어떨까요?

liquidjoo · 2022-02-21T04:51:38Z

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

+import org.junit.jupiter.api.DisplayName;
+import org.junit.jupiter.api.Test;
+
+class CardTest {


매 요청마다 카드 객체가 생성되는 것을 최소화 해보고 이를 테스트 코드로 나타내보면 좋을 것 같아요 :)

liquidjoo · 2022-02-21T04:52:16Z

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

+    @DisplayName("카드 한장을 뽑는다.")
+    @Test
+    void GivenNothing_When카드_뽑기_Then카드() {
+        // When
+        final Card card = sut.draw();
+
+        // Then
+        assertThat(card).isNotNull();
+    }


내가 어떤 카드를 뽑았는지 테스트를 해보려면 어떻게 해보는 것이 좋을까요?

liquidjoo · 2022-02-21T04:54:34Z

src/test/java/oilinjection/domain/car/SonataTest.java

+        final Car sonata = new Sonata(200D);
+        assertThat(sonata.getChargeQuantity()).isEqualTo(20D);
+    }
+}


파일 마지막에 엔터(개행문자)를 넣어주세요!!
이유는 리뷰를 진행할 때 깃허브에서 경고메시지를 지우기 위함입니다.

좀 더 알고 싶으시면 재미삼아 아래의 링크를 보시는 것도 추천 드립니다.
https://minz.dev/19

Intellij 를 사용하실 경우엔
Preferences -> Editor -> General -> Ensure line feed at file end on save 를 체크해주시면
파일 저장 시 마지막에 개행문자를 자동으로 넣어줍니다!

liquidjoo · 2022-02-21T04:56:47Z

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

+    private static final int ACE_VALUE_11 = 11;
+
+    private final List<Card> cards;
+    private Score score;


Score 라는 객체와 결합을 갖는 것도 좋지만
Hands 와 Score 두 객체의 메시지 전달로 표현을 해보아도 좋을 것 같아요

0JUUU added 30 commits February 13, 2022 16:10

docs: 연료 주입 구현 사항 목록 작성

794c525

feat: 불필요한 파일 제거

d75a5f4

feat: 사용자 입력을 관리하는 InputView 클래스 구현

4fea65a

feat: 자동차 추상화 클래스 Car 구현

ff68c0f

feat: 소나타에 관한 테스트 SonataTest 구현

977b285

feat: Sonata 클래스 구현

ab5a54e

- 추상클래스 Car에 존재하는 메서드 오버라이딩

feat: 아반떼에 관한 테스트 AvanteTest 구현

b83e167

feat: Avante 클래스 구현

6b90041

- 추상클래스 Car에 존재하는 메서드 오버라이딩

feat: K5에 대한 테스트 K5Test 클래스 구현

619bc0b

feat: K5 클래스 구현

42bc897

- 추상클래스 Car에 존재하는 메서드 오버라이딩

feat: add ignore

2ea68bc

refactor: tripDistance 부모 필드에서 상속 받도록 변경.

a52796e

feat: 입력을 저장하는 Rent VO 객체 추가

5fb53cf

feat: Rent 클래스 equals & hashCode 재정의

37c0b78

feat: InputView 클래스 생성자 제한접근자 변경

0440900

feat: 사용자 입력을 파싱하여 Rent 객체 리스트를 반환

7603b88

test: Parser에 대한 테스트코드 작성

f822ac0

1. 정상적인 입력이 들어오는 경우 2. 빈 값이 들어오는 경우, 예외 발생 3. 입력 형식에 유효하지 않은 경우, 예외 발생

docs: 연료 주입 구현 사항 목록 수정

e9205ee

feat: 이름에 따라 Car 객체를 반환하는 CarFactory enum 클래스 생성

8b2cd26

test: CarFactory 클래스에 대한 테스트코드 작성

98acb61

1. Avante 반환 확인 2. K5 반환 확인 3. Sonata 반환 확인 4. 해당되는 이름이 없는 경우, exception 발생

feat: Car 객체 내 equals & hashcode 오버라이드

f408a4c

docs: 연료 주입 구현 사항 목록 수정

88fb5d3

feat: 여행거리가 1보다 작은 경우에 대한 예외처리 적용

ccff25c

feat: 1보다 작은 여행거리에 대한 테스트케이스 작성

3daf149

docs: 연료 주입 구현 사항 목록 수정

fa50f21

feat: 대소문자 구분없이 Car 객체를 반환하는지 확인하는 테스트코드 작성

b803fc1

refactor: 추상 메서드 접근제한자 default에서 public으로 변경

4a56d05

refactor: CarFactory -> RentCar 네이밍 변경

738c9a7

1. 클래스 이름 변경 2. 메서드 변경 적절한 이름으로 변경

rename: Rent -> RentInfo 네이밍 변경

933475e

test: 렌트 예약 기능에 대한 테스트코드 작성

d73ed2b

0JUUU and others added 29 commits February 16, 2022 22:00

refactor: InputView 유틸 클래스로 변경.

f75bac5

feat: 참가자 이름 추가

d938a77

1. Participant 2. Player 3. Dealer

docs: 블랙잭 참가자 이름에 대한 구현 사항 목록 수정

df47d5c

style: 구글 코드 컨벤션 적용

2df412f

test: Game 관련 테스트 구현

a40bd3d

1. 카드 한 장 분배 2. 참가자의 초기 카드 분배

feat: 카드를 분배하는 Game 구현

b0b47a4

1. 전체 카드를 관리하는 덱 생성 2. 카드 분배 3. 참가자들에게 초기 카드 분배

refactor: Cards -> Deck 변경, 기존 Deck 제거

330913e

- 불필요한 Wrapping 제거

refactor: default -> public 접근제한자 변경

df0be65

feat: 참가자가 가지고 있는 카드 이름들을 반환하는 메서드 구현

348507e

1. Participant#getCardNames() 2. Hands#getCardNames()

feat: 초기 분배 결과 출력 구현

300e059

test: 중복 없이 모든 카드가 뽑힌다

318d553

feat: 뽑힌 카드를 제거하도록 구현

696cd0b

docs: 블랙잭 OutputView 구현 사항 수정

c77f874

1. 참가자 카드 출력

refactor: 기존 Deck, Game 이름 변경

ce71f13

1. Deck > Cards 2. Game > Deck

refactor: final 키워드 추가

22150b7

refactor: 팀 코드 컨벤션 적용

7a5aed7

fix: EIGHT의 이름이 7이던 걸 8로 변경

fc6043b

refactor: 불필요한 import문 제거

2f94a6e

feat: 추가 카드 Draw 구현

330c4c9

1. 플레이어 : 21을 넘지 않는 경우, 의사를 물어봄 2. 딜러 : 16을 넘지 않는다면, 무조건 받음

style: 공백 및 줄바꿈 수정

460f2c4

test: Judge 관련 테스트 구현

930eb9a

승패의 경우의 수 테스트

feat: 참가자들의 승패를 판단하는 Judge 구현

c982483

docs: 블랙잭 구현 사항 수정

3835cb6

refactor: 네이밍 변경

9f64fad

refactor: 변수, 메서드 네이밍 변경

2b18ffa

refactor: Dealer, Player 생성자 파라미터 변경

525c261

Hands -> List<Card>로 변경

refactor: getWinOrLose 로직 메서드 추출 및 네이밍 변경.

9ee25c2

feat: getStartCardNames 추가.

3bd09b5

- 요구 사항 변경. - 딜러는 처음에 한장의 카드만을 보여준다.

fix: 플레이어 Score limit 설정 변경.

36f6d1d

- 플레이어가 21점이면 패배한 것으로 간주되는 버그, 21이 최대 점수가 되도록 수정.

liquidjoo requested changes Feb 21, 2022

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[김태준] 연료 주입, 블랙잭 - Step1 #35

[김태준] 연료 주입, 블랙잭 - Step1 #35

Uh oh!

kimtaejun97 commented Feb 17, 2022

Uh oh!

liquidjoo left a comment

Uh oh!

liquidjoo Feb 18, 2022

Uh oh!

liquidjoo Feb 18, 2022

Uh oh!

liquidjoo Feb 18, 2022

Uh oh!

liquidjoo Feb 18, 2022

Uh oh!

liquidjoo Feb 18, 2022

Uh oh!

liquidjoo Feb 21, 2022

Uh oh!

liquidjoo Feb 21, 2022

Uh oh!

liquidjoo Feb 21, 2022

Uh oh!

liquidjoo Feb 21, 2022

Uh oh!

liquidjoo Feb 21, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

[김태준] 연료 주입, 블랙잭 - Step1 #35

Are you sure you want to change the base?

[김태준] 연료 주입, 블랙잭 - Step1 #35

Uh oh!

Conversation

kimtaejun97 commented Feb 17, 2022

Uh oh!

liquidjoo left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants