[이현주] 연료 주입, blackjack (Step1) by hyoenju · Pull Request #38 · next-step/java-blackjack

hyoenju · 2022-02-17T15:30:16Z

안녕하세요

연료 주입과 블랙잭 미션 구현 완료하여 과제 제출합니다.

감사합니다 :)

- Sonata, Avante, K5

- CarTest.java - RentCarTest.java

…o step1

- Player 와 Dealer에게 카드 2장을 배분 프로세스 구현

- Dealer의 이름은 "딜러"로 고정 - Dealer의 카드를 출력을 OutputView로 묶음 - 매직넘버 & 리터럴 추가

hyunniiii

안녕하세요~ 현주님
몇가지 코멘트 남겼습니다. ″̮
확인부탁드립니다!

hyunniiii · 2022-02-20T03:17:02Z

src/main/java/blackjack/controller/Controller.java

+    public void receive(Player player) {
+        Gameable gameable = player.getCards();
+        String yesOrNo = "";
+        do {
+            yesOrNo = InputView.inputYesOrNo(player.getName());
+            if (yesOrNo.equals("y")) {
+                gameable.addCard(CardDeck.pop());
+                OutputView.printCurrentCardsState(player.getName(), player.getCards());
+                gameable = gameable.judge();
+            }
+            if (yesOrNo.equals("n")) {
+                gameable = new State(gameable.cards(), false);
+            }
+        } while (gameable.isEnd());
+    }


함수(또는 메서드)의 길이가 10라인을 넘어가지 않도록 구현한다.
요구사항에 맞도록 구현해보면 어떨까요?

hyunniiii · 2022-02-20T03:17:15Z

src/main/java/blackjack/domain/Game.java

+    private static final String DEALER_NAME = "딜러";
+    private final static int DEALER_THRESHOLD = 16;
+
+    private final List<Player> players;


players를 일급컬렉션으로 분리해주는건 어떨까요?
책임을 조금 더 분리할 수 있을 것 같습니다.

hyunniiii · 2022-02-20T03:17:28Z

src/main/java/blackjack/domain/Game.java

+    }
+
+    public boolean giveCardToDealer() {
+        if (dealer.getCards().cards().sumScore() <= DEALER_THRESHOLD) {


디미터 법칙을 위반하고 있네요.
dealer.getCards().cards().sumScore() <= DEALER_THRESHOLD 이 부분에 대한 책임을 넘겨주고 결과만 받아서 사용하도록 하면 어떨까요?

hyunniiii · 2022-02-20T03:17:49Z

src/main/java/blackjack/domain/Winner.java

+public class Winner {
+
+    private final int BLACKJACK = 21;
+    private final Game game;


Winner 라는 클래스가 상태값으로 Game을 가지고 있네요.
Winner는 우승자들을 상태값으로 가지고 있는게 맞지 않을까요? ″̮

아니면 조금 더 적절한 이름으로 변경해주는건 어떨까요?
한번 고민해보면 좋을 것 같습니다.

hyunniiii · 2022-02-20T03:18:16Z

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

+
+public class Dealer extends Person {
+
+    private final Gameable cards;


cards도 Person에 구현해서 중복을 줄여보는건 어떨까요?

hyunniiii · 2022-02-20T03:18:41Z

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

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


디미터 법칙을 위반하는 곳이 많네요.
디미터 법칙을 위반하지 않도록 수정해주세요 : )
https://mangkyu.tistory.com/147

hyunniiii · 2022-02-20T03:18:53Z

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

+    }
+
+    public static void printGameWinOrLose(Person person, List<Integer> gameResult) {
+        System.out.printf("%s : %d승 %d패%n", person.getName(), gameResult.get(0), gameResult.get(1));


a,b 가 2패가 아니라 각각 1패가 되야하지 않을까요? 확인 부탁드립니다.

hyunniiii · 2022-02-20T03:19:07Z

src/main/java/rentcar/domain/Avante.java

+
+    private static final String CAR_NAME = "Avante";
+    private static final double DISTANCE_PER_LITER = 15;
+    private final double distance;


상수와 변수는 개행을 통해 구분해주세요 : )

hyunniiii · 2022-02-20T03:19:18Z

src/main/java/rentcar/domain/Car.java

@@ -0,0 +1,27 @@
+package rentcar.domain;
+
+public abstract class Car {


각 자동차들의 distance들도 추상클래스에 구현해주면 어떨까요?
중복을 줄일 수 있을 것 같습니다.

hyunniiii · 2022-02-20T03:19:28Z

src/main/java/rentcar/domain/K5.java

+
+    private static final String CAR_NAME = "K5";
+    private static final double DISTANCE_PER_LITER = 13;
+    private final double distance;


여기도 상수와 변수는 개행을 통해 구분해주세요 : )

hyoenju and others added 30 commits February 11, 2022 16:02

docs(rentCar): 연료주입 기능요구사항 정의

cb6c3ad

feat(main): Application 클래스 생성

c0947c3

feat(car): Car 추상 클래스 생성

527c83f

feat(car): Car 상속 클래스 생성

7b498fa

- Sonata, Avante, K5

feat: empty.txt 생성

9f5362d

feat(company): RentCompany 클래스 생성

3498e4c

test(company): RentCompany 클래스 생성 테스트

ebcb132

feat(company): Car 객체 추가

d8cb870

test(company): Car 객체 추가 테스트

80dd03e

feat(car): getter 정의

8cafc20

feat(company): 레포트 생성 기능 구현

0687344

test(company): 레포트 생성 기능 테스트

19286f8

docs: Update README.md

d8cc419

feat(main): main method 정의

cf06f9e

feat: empty.txt 삭제

a06b01f

remove: empty.txt 삭제

ebcaf92

feat(view): inputView 클래스 생성

f4c30c4

remove: 불필요한 파일 삭제

bfded44

test: 테스트 코드 작성

e518287

- CarTest.java - RentCarTest.java

feat(view): OutputView 클래스 생성, InputView 로직 수정

8c65c2f

feat(rentCar): enum으로 rentCar 타입 결정

1fa56ff

feat(Application): Application에서 Controller 호출

e43b483

Merge branch 'step1' of https://github.com/hyoenju/java-blackjack int…

512b778

…o step1

docs: 연료 주입 기능 구현 완료

aeaab59

docs: 블랙잭 기능 명세서 작성

bbfecb5

feat(participant): 부모 클래스(Person) 생성

9146c02

feat(participant): Participant 클래스 생성

875efb0

test(participant): Participant 클래스 생성 테스트

165f03c

feat(card): cardNumber enum 클래스 생성

f1b626d

feat(card): cardPattern enum 클래스 생성

fd92fd7

hyoenju and others added 29 commits February 16, 2022 01:47

feat(view) : InputView, OuputView 클래스 생성, 기능 구현

00f6f1d

feat(card) : 게임 진행 위한 카드 객체 생성

dfa02ad

feat(dealer) : Dealer 클래스 생성

0646780

feat(main) : blackjack 기능 구현

13cc902

remove(test) : Remove Test

5403ec2

refactor(view): InputView, OutputView 정적 팩토리 메소드로 변경

6c4694f

feat(game): game 클래스 생성

72bb5f9

- Player 와 Dealer에게 카드 2장을 배분 프로세스 구현

refactor(view): OutputView 클래스 수정

db82882

- Dealer의 이름은 "딜러"로 고정 - Dealer의 카드를 출력을 OutputView로 묶음 - 매직넘버 & 리터럴 추가

test(player): PlayerTest 코드 작성

089ff30

test(dealer): DealerTest 코드 작성

181a0e4

test(dealer): Game 테스트 코드 작성

d08b32a

refactor(main): Controller new 연산자로 생성

f86f7d4

refactor(main): 구현된 기능 분리 작업

ec8a9c6

feat(card) : Card 객체 생성 && test(card)

2e180aa

feat(cards) : Card 객체 생성, 계산 && test(card)

a7bb315

refactor(cardDeck) : 파일 위치 이동

57fb0f5

refactor(cardDeck) : 매직넘버 생성, final 정의

7f6b559

refactor(number) : 파일 위치 이동

db9372d

refactor(number) : final 정의

4f8dbcd

refactor(person): 파일 이동, final 정의

aa60018

refactor(state): 공통 기능 분리

1b5e63c

delete & remove

bf9ab2c

feat(view): 결과 출력 메소드 구현

dccbe46

feat(winner): 게임 결과 메소드 구현 && test(winner)

ce019ba

feat(control): Controller 기능 구현

5558990

refactor(game): game 기능 구현 && test(game)

ffb0e10

refactor: reformat Codes

d04be38

Update README.md

aca0ca1

refactor: 디버깅용 print 삭제

b83a693

hyunniiii requested changes Feb 20, 2022

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[이현주] 연료 주입, blackjack (Step1)#38

[이현주] 연료 주입, blackjack (Step1)#38
hyoenju wants to merge 67 commits intonext-step:hyoenjufrom
hyoenju:step1

hyoenju commented Feb 17, 2022

Uh oh!

hyunniiii left a comment

Uh oh!

hyunniiii Feb 20, 2022

Uh oh!

hyunniiii Feb 20, 2022

Uh oh!

hyunniiii Feb 20, 2022

Uh oh!

hyunniiii Feb 20, 2022

Uh oh!

hyunniiii Feb 20, 2022

Uh oh!

hyunniiii Feb 20, 2022

Uh oh!

hyunniiii Feb 20, 2022

Uh oh!

hyunniiii Feb 20, 2022

Uh oh!

hyunniiii Feb 20, 2022

Uh oh!

hyunniiii Feb 20, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants


		public class Dealer extends Person {

		private final Gameable cards;

		@@ -0,0 +1,27 @@
		package rentcar.domain;

		public abstract class Car {

Conversation

hyoenju commented Feb 17, 2022

Uh oh!

hyunniiii 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