[이호찬] 연료 주입, 블랙잭 (Step 1) #29

hochan222 · 2022-02-17T13:14:08Z

안녕하세요.

리뷰 주셔서 감사드립니다.

Step1 @chanuuuuu 님과 페어로 진행했습니다.

좋은 하루 보내세요 :)

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

- 거리를 가져야한다. - 거리는 `0` 이상 정수이어야 한다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

- Avante - `Avante` String Type 이름을 가져야한다. - `15km/리터`의 연비 가져야한다. - 거리 가져야한다. - 거리는 `0` 이상 정수이어야한다. - - 주입해야할 연료량을 구할 수 있다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

- K5 - `K5` String Type 이름을 가져야한다. - `13km/리터`의 연비 가져야한다. - 거리 가져야한다. - 거리는 `0` 이상 정수이어야한다. - 주입해야할 연료량을 구할 수 있다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

변경된 클래스 - Avante - K5 Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

- Deck을 가지고 있다. - UserName을 가지고 있다. - name은 `dealer`라는 기본 이름을 가진다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

- GameUser - BlackJack을 수행하는 User의 집합 - List<User>를 가지고 있다. - User를 추가할 수 있다. - List<User>를 반환할 수 있다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

- GameUser 인스턴스 생성방식 변경 Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

- GameUser를 가지고 있다. - GameCard를 가지고 있다. - 게임 진행시 최초로 GameCard를 셔플할 수 있다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

- 게임 진행시 최초로 GameCard를 셔플할 수 있다. - User별로 초기 카드를 나눠준다. - GameCard에서 Card를 두 장 뽑기 - User에게 Card 전달하기 Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

- BlackJack - 게임 로직 수행 - GameUser를 가지고 있다. - GameCard를 가지고 있다. - 게임 진행시 최초로 GameCard를 셔플할 수 있다. - User별로 초기 카드를 나눠준다. - GameCard에서 Card를 두 장 뽑기 - User에게 Card 전달하기 - Dealer는 총 score가 16이하일 경우, 1장의 카드를 추가로 받는다. - User별로 턴을 진행할 수 있게 한다. - GameCard에서 Card를 한 장 뽑기 - User에게 Card 전달하기 - Player는 총 score가 21이하일 경우, 1장의 카드를 추가로 받을 수 있다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

- UserStats - User별 승/패에 대한 정보 - GameUser를 인자로 받아 User별, 승/패 정보를 계산한다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

- UserStats - User별 승/패에 대한 정보 - GameUser를 인자로 받아 User별, 승/패 정보를 계산한다. - User별 Deck의 score를 출력하기 위한 String을 반환할 수 있다. convertTotalScore() - User별 최종 승/패를 출력하기 위한 String을 반환할 수 있다. convertTotalResult() - User별 Deck의 score가 21을 초과할 경우, 승/패를 계산할 수 있다. - Dealer의 score가 21을 초과할 경우, Player가 승리한다. - Dealer의 score가 21을 초과하지 않고, Player의 score가 21을 초과하는 경우, Dealer가 승리한다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

liquidjoo

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

liquidjoo · 2022-02-21T10:38:40Z

src/main/java/blackJack/controller/Game.java

+    public static Game getInstance() {
+        if (game == null) {
+            game = new Game(getPlayerName());
+        }
+        return game;
+    }
+


싱글톤 패턴을 사용하신 이유가 있을까요?
싱글톤 패턴에는 장단점이 있는데요~ 한번 알아보는 것도 좋을 것 같네요

liquidjoo · 2022-02-21T10:41:14Z

src/main/java/blackJack/controller/Game.java

+    private void init() {
+        blackJack.initCardDraw();
+        OutputView.printInitCardDrawFormat(blackJack.getGameUser().convertPlayersName());
+        OutputView.printUserStatus(blackJack.getGameUser());
+    }


블랙잭을 초기화 하는 도메인 로직과 화면 로직이 같이 있어서 테스트하기가 너무 어려울 것 같아요
도메인 로직과 화면을 분리해 테스트 할 수 있도록 작성해보는 것이 어떨까요?

liquidjoo · 2022-02-21T10:41:28Z

src/main/java/blackJack/controller/Game.java

+    private void summarize() {
+        UserStats userStats = UserStats.of(blackJack.getGameUser());
+        OutputView.printTotalScore(userStats.convertTotalScore());
+        OutputView.printTotalResult(userStats.convertTotalResult());
+    }


도메인 로직과 화면을 분리해 보아요~

liquidjoo · 2022-02-21T10:48:40Z

src/main/java/blackJack/controller/Game.java

+import blackJack.view.OutputView;
+import java.util.List;
+
+public class Game {


테스트 코드가 필요해 보여요~

liquidjoo · 2022-02-21T10:50:07Z

src/main/java/blackJack/domain/BlackJack.java

+    private void cardShuffle() {
+        gameCard.shuffle();
+    }


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

liquidjoo · 2022-02-21T12:10:41Z

src/main/java/blackJack/domain/GameUser.java

+    public GameUser append(Player user) {
+        players.add(user);
+        return this;
+    }


다른 곳에서 사용하지 않는다면 접근 제어자를 활용해보는 것이 어떨까요?

liquidjoo · 2022-02-21T12:11:40Z

src/main/java/blackJack/domain/GameUser.java

+        return this;
+    }
+
+    public List<User> getUsers() {


GameUser 에서 User 를 생성하는 것이 어색해 보여요

liquidjoo · 2022-02-21T12:11:59Z

src/main/java/blackJack/domain/GameUser.java

+    }
+
+    public List<User> getUsers() {
+        List<User> user = new ArrayList<>();


컬렉션을 사용할 때 변수명은 복수로 표기해보아요~

liquidjoo · 2022-02-21T12:13:01Z

src/test/java/blackJack/domain/CardNumberTest.java

+            .isInstanceOf(RuntimeException.class);
+
+    }
+}


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

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

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

liquidjoo · 2022-02-21T12:14:13Z

src/test/java/blackJack/domain/CardTest.java

+    @DisplayName("카드 숫자와 카드 타입을 입력할때, 주어진 숫자와 타입으로 카드가 생성된다.")
+    @ParameterizedTest
+    @MethodSource("generateCardTypeAndCardNumber")
+    void givenNumberWithStringAndTypeWithString_WhenGetCardNumberAndGetCardType_ThenEqualToGivenNumberAndType(


이름이 너무 긴 것 같아요~

카드_숫자와_카드_타입을_입력할때_주어진_숫자와_타입으로_카드가_생성된다 와 같이 한글로 메서드 명을 적어도 괜찮아요~

hochan222 and others added 30 commits February 11, 2022 17:02

docs: TDD 기능 구현 사항 정의

194bd15

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

test: RentCompanyTest 테스트 케이스 추가

2132cfe

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

feat: 초기 구조.

452e494

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

test: Sonata String Type 이름을 가져야한다.

af3b6ac

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

test: Sonata String Type 이름을 가져야한다.

a508794

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

feat: Sonata String Type 이름을 가져야한다.

115b872

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

docs: Sonata String Type 이름을 가져야한다.

afb8d8b

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

test: 10km/리터의 연비 가져야한다.

27431e9

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

feat: 10km/리터의 연비 가져야한다.

bed1c08

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

docs: 10km/리터의 연비 가져야한다.

33d93c5

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

test: 거리

7071676

- 거리를 가져야한다. - 거리는 `0` 이상 정수이어야 한다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

test: 거리

4d48981

- 거리를 가져야한다. - 거리는 `0` 이상 정수이어야 한다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

feat: 거리

49214d3

- 거리를 가져야한다. - 거리는 `0` 이상 정수이어야 한다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

docs: 거리

e5bcbc7

- 거리를 가져야한다. - 거리는 `0` 이상 정수이어야 한다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

test: 주입해야할 연료량을 구할 수 있다.

9e5d638

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

feat: 주입해야할 연료량을 구할 수 있다.

7aab241

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

docs: 주입해야할 연료량을 구할 수 있다.

6081d09

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

test: 메서드 이름 리펙토링

4f596ac

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

docs: git pull pair

0501b46

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

docs : 기능 구현 사항 checkbox 제거

7ad9a5f

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

test : Class명에 따른 테스트 메소드명 변경

4d22b97

변경된 클래스 - Avante - K5 Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

docs : RentCompany에 대한 기능 구현 추가

94c57b9

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

feat : RentCompany factory 메소드 구현

3fbb733

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

test : RentCompanyTest - Car의 List를 가져야한다.

ffc7c7f

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

chanu94 and others added 29 commits February 16, 2022 17:07

docs : Dealer 클래스 구현

35ca712

- Deck을 가지고 있다. - UserName을 가지고 있다. - name은 `dealer`라는 기본 이름을 가진다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

docs : Dealer 클래스 구현 checkbox 추가

64ce0e8

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

feat : GameUser 클래스 구현

d93d497

- GameUser - BlackJack을 수행하는 User의 집합 - List<User>를 가지고 있다. - User를 추가할 수 있다. - List<User>를 반환할 수 있다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

docs : GameUser 클래스 구현

78af8ab

- GameUser - BlackJack을 수행하는 User의 집합 - List<User>를 가지고 있다. - User를 추가할 수 있다. - List<User>를 반환할 수 있다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

refactor : GameUser 리팩토링

19f4ef3

- GameUser 인스턴스 생성방식 변경 Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

refactor : Dealer 정적 상태 변수명 변경

cd83748

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

feat : GameCard shuffle 기능 추가

a761d7c

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

feat : BlackJack 클래스 구현

c5a791d

- GameUser를 가지고 있다. - GameCard를 가지고 있다. - 게임 진행시 최초로 GameCard를 셔플할 수 있다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

refactor : GameUser의 Playter, Dealer 책임 분리

452aa4c

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

refactor : 메소드 참조로 변경

359ff2f

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

feat: Game - 전체적인 게임을 진행할 Controller 구현 및 InputView, OutputView 로직 추가

2c71cbf

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

docs: Game - 전체적인 게임을 진행할 Controller 구현 및 InputView, OutputView 로직 추가

d0e97ac

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

feat: phase 출력 구현

03ea0d5

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

feat: convertTotalScore 메서드 구현, UserStats

7908ef9

- UserStats - User별 승/패에 대한 정보 - GameUser를 인자로 받아 User별, 승/패 정보를 계산한다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

test: convertTotalScore 메서드 구현에 따른 dealer이름 수정

d7a5c87

- UserStats - User별 승/패에 대한 정보 - GameUser를 인자로 받아 User별, 승/패 정보를 계산한다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

docs: convertTotalScore 메서드 구현

37c524e

- UserStats - User별 승/패에 대한 정보 - GameUser를 인자로 받아 User별, 승/패 정보를 계산한다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

docs: convertTotalScore 메서드 구현

44216e4

- UserStats - User별 승/패에 대한 정보 - GameUser를 인자로 받아 User별, 승/패 정보를 계산한다. Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

refactor : User 추상 클래스로 리팩토링

352de29

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

refactor: getScore within 10 line

83bcacb

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

refactor: remove not use Score

52fe5e1

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

style: UserStats

0f7ea39

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

refactor: Game's function within 10 lines

4e5dc51

Co-authored-by: hochan222 <hochan049@gmail.com> Co-authored-by: chanuuuuu <dn0208@gmail.com>

liquidjoo requested changes Feb 21, 2022

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[이호찬] 연료 주입, 블랙잭 (Step 1) #29

[이호찬] 연료 주입, 블랙잭 (Step 1) #29

Uh oh!

hochan222 commented Feb 17, 2022

Uh oh!

liquidjoo left a comment

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!

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!

2 participants

[이호찬] 연료 주입, 블랙잭 (Step 1) #29

Are you sure you want to change the base?

[이호찬] 연료 주입, 블랙잭 (Step 1) #29

Uh oh!

Conversation

hochan222 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!

2 participants