[이호찬] 연료 주입, 블랙잭 (Step 1) by hochan222 · Pull Request #29 · next-step/java-blackjack

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

안녕하세요.

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

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

좋은 하루 보내세요 :)

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

- 거리를 가져야한다. - 거리는 `0` 이상 정수이어야 한다. Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

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

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

변경된 클래스 - Avante - K5 Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

- Deck을 가지고 있다. - UserName을 가지고 있다. - name은 `dealer`라는 기본 이름을 가진다. Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

- GameUser - BlackJack을 수행하는 User의 집합 - List<User>를 가지고 있다. - User를 추가할 수 있다. - List<User>를 반환할 수 있다. Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

- GameUser 인스턴스 생성방식 변경 Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

- GameUser를 가지고 있다. - GameCard를 가지고 있다. - 게임 진행시 최초로 GameCard를 셔플할 수 있다. Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

- 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 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

- UserStats - User별 승/패에 대한 정보 - GameUser를 인자로 받아 User별, 승/패 정보를 계산한다. Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

- 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 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

test: RentCompanyTest 테스트 케이스 추가

2132cfe

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

feat: 초기 구조.

452e494

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

af3b6ac

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

a508794

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

115b872

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

afb8d8b

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

27431e9

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

bed1c08

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

33d93c5

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

test: 거리

7071676

- 거리를 가져야한다. - 거리는 `0` 이상 정수이어야 한다. Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

test: 거리

4d48981

- 거리를 가져야한다. - 거리는 `0` 이상 정수이어야 한다. Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

feat: 거리

49214d3

- 거리를 가져야한다. - 거리는 `0` 이상 정수이어야 한다. Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

docs: 거리

e5bcbc7

- 거리를 가져야한다. - 거리는 `0` 이상 정수이어야 한다. Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

9e5d638

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

7aab241

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

6081d09

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

test: 메서드 이름 리펙토링

4f596ac

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

docs: git pull pair

0501b46

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

docs : 기능 구현 사항 checkbox 제거

7ad9a5f

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

4d22b97

변경된 클래스 - Avante - K5 Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

94c57b9

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

feat : RentCompany factory 메소드 구현

3fbb733

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

ffc7c7f

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

docs : Dealer 클래스 구현

35ca712

- Deck을 가지고 있다. - UserName을 가지고 있다. - name은 `dealer`라는 기본 이름을 가진다. Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

docs : Dealer 클래스 구현 checkbox 추가

64ce0e8

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

feat : GameUser 클래스 구현

d93d497

- GameUser - BlackJack을 수행하는 User의 집합 - List<User>를 가지고 있다. - User를 추가할 수 있다. - List<User>를 반환할 수 있다. Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

docs : GameUser 클래스 구현

78af8ab

- GameUser - BlackJack을 수행하는 User의 집합 - List<User>를 가지고 있다. - User를 추가할 수 있다. - List<User>를 반환할 수 있다. Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

refactor : GameUser 리팩토링

19f4ef3

- GameUser 인스턴스 생성방식 변경 Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

cd83748

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

feat : GameCard shuffle 기능 추가

a761d7c

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

feat : BlackJack 클래스 구현

c5a791d

- GameUser를 가지고 있다. - GameCard를 가지고 있다. - 게임 진행시 최초로 GameCard를 셔플할 수 있다. Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

refactor : GameUser의 Playter, Dealer 책임 분리

452aa4c

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

refactor : 메소드 참조로 변경

359ff2f

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

2c71cbf

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

d0e97ac

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

feat: phase 출력 구현

03ea0d5

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

feat: convertTotalScore 메서드 구현, UserStats

7908ef9

- UserStats - User별 승/패에 대한 정보 - GameUser를 인자로 받아 User별, 승/패 정보를 계산한다. Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

d7a5c87

- UserStats - User별 승/패에 대한 정보 - GameUser를 인자로 받아 User별, 승/패 정보를 계산한다. Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

docs: convertTotalScore 메서드 구현

37c524e

- UserStats - User별 승/패에 대한 정보 - GameUser를 인자로 받아 User별, 승/패 정보를 계산한다. Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

docs: convertTotalScore 메서드 구현

44216e4

- UserStats - User별 승/패에 대한 정보 - GameUser를 인자로 받아 User별, 승/패 정보를 계산한다. Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

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

352de29

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

refactor: getScore within 10 line

83bcacb

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

refactor: remove not use Score

52fe5e1

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

style: UserStats

0f7ea39

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

refactor: Game's function within 10 lines

4e5dc51

Co-authored-by: hochan222 <[email protected]> Co-authored-by: chanuuuuu <[email protected]>

liquidjoo requested changes Feb 21, 2022

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

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

[이호찬] 연료 주입, 블랙잭 (Step 1)#29
hochan222 wants to merge 108 commits intonext-step:hochan222from
hochan222:step1

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

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