[김정호] 연료 주입, 블랙잭 (Step 1) by Hoya-kim · Pull Request #34 · next-step/java-blackjack

Hoya-kim · 2022-02-17T13:54:05Z

안녕하세요!
김정호입니다🙇🏻‍♂️

저는 이지인님과 함께 연료주입, 블랙잭 미션을 수행했습니다!
리뷰 주시면 피드백 반영하겠습니다!

항상 좋은 리뷰 감사합니다!😄

연료 주입 미션 요구 사항 정리 - 기능 요구 사항 - 프로그래밍 요구 사항 - 기능 구현 사항 리스트 정리

Classes - Car - RentCompany

- Avante, K5, Sonata 클래스 생성 - 제공된 테스트 코드 추가

- constructor - getDistancePerLiter() - getTripDistance() - getName()

- 기능 구현 사항 리스트 완료 처리

- 블랙잭 게임 기능 구현 사항 작성

classes - BlackjackGame - Dealer - Deck - Denomination - Gambler - Judgement - InputView - OutputView - Player - Suit

classes - DealerTest - DeckTest - DenominationTest - GamblerTest - JudgementTest - PlayerTest - SuitTest

classes -Card

- allocateCard()

- Denomination

- 기능 구현 완료 사항 체크 표시

-Suit

- enum타입의 suit, denomination값을 가지는 Card class 설계

- deck class 구현 - field : List card - method: setupCard, setDenomination, shuffle, popCard

- 중복 코드 제거를 위함 - Dealer#allocateCard 메서드 구현

- calculateScore - adjustScore

-addOneMoreCard

- findWinner()

- findWinner() - checkWinOrLost() - getDealerResult() - getCount()

- testFindWinner

joswlv

안녕하세요 정호님!
유연하고 읽기 좋은 코드를 위해 객체 관계를 고민해주세요 🙇
몇 가지 코멘트를 남겼습니다. 확인부탁드립니다.

joswlv · 2022-02-19T03:04:17Z

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

+        return denomination;
+    }
+
+    public String getDenomiationCountstring() {


Suggested change

public String getDenomiationCountstring() {

public String getDenomiationCountString() {

joswlv · 2022-02-19T03:26:19Z

src/main/java/blackjack/controller/BlackjackGame.java

+        List<Player> players = playersName.stream().map(Player::new)
+            .collect(Collectors.toList());


Suggested change

List<Player> players = playersName.stream().map(Player::new)

.collect(Collectors.toList());

Players players = playersName.stream().map(Player::new)

.collect(Collectors.toList());

로또 미션에서 학습한 것 처럼 일급컬렉션을 만들어 객체의 행위를 정해보면 어떨까요?

joswlv · 2022-02-19T03:26:23Z

src/main/java/blackjack/domain/Denomination.java

+    private final int count;
+    private final String countString;


Suggested change

private final int count;

private final String countString;

private final int score;

private final String cardName;

의미를 명확하게 변수명을 변경해보는 것은 어떨까요?

joswlv · 2022-02-21T11:11:09Z

src/main/java/blackjack/domain/Player.java

+import java.util.ArrayList;
+import java.util.List;
+
+public class Player {


플레이어에게 카드를 받을 수 있는지 여부를 판단하는 역할을 추가하면 어떨까요?

joswlv · 2022-02-21T11:13:05Z

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

@@ -0,0 +1,29 @@
+package blackjack.domain;
+
+public class Card {


매 게임마다 카드를 새로 생성해야하는 것 보다 카드 static하게 만들어 보는 것은 어떨까요?

joswlv · 2022-02-21T11:15:34Z

src/main/java/blackjack/controller/BlackjackGame.java

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


컨트롤러에 카드점수 상태를 판단하는 로직이 있는 것 보다 적절한 객체에 책임을 부여해보는 것은 어떨까요?
딜러와 플레이어가 카드를 들고 있는 손을 객체로 만들어 카드를 추가 여부와 카드를 카드 점수 상태를 관리해 보는 것은 어떨까요?

joswlv · 2022-02-21T11:18:44Z

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

+    public Deck() {
+        cards = new ArrayList<>();
+        setupCard();
+        shuffle();


덱을 구성하는 방식을 주입받아 생성하는 것은 어떨까요?

joswlv · 2022-02-21T11:21:30Z

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

+    private static void printFinalResult(Map<String, String> results) {
+        System.out.println(FINAL_RESULT);
+        results.forEach((name, result) -> {
+            System.out.println(name + COLON + result);


Suggested change

System.out.println(name + COLON + result);

privat static final String FINAL_RESULT_MESSAGE = "%s : %s";

...

System.out.println(String.format(FINAL_RESULT_MESSAGE, name, result));

+연산자 보다 String.format을 활용해 보는 것은 어떨까요?

joswlv · 2022-02-21T11:31:15Z

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

@@ -0,0 +1,39 @@
+package blackjack.domain;
+
+public class Dealer extends Player {


딜러와 플레이어의 공통행위를 추상화해보는 방식으로 수정하면 어떨까요?
딜러와 플레이어 모두 카드를 받는 행위와 이름을 가지고 있습니다.

Hoya-kim and others added 30 commits February 11, 2022 16:15

docs: Update README.md

3b755f3

연료 주입 미션 요구 사항 정리 - 기능 요구 사항 - 프로그래밍 요구 사항 - 기능 구현 사항 리스트 정리

chore: 프로젝트 수행을 위한 기본 구성 파일 추가

ff92f1c

Classes - Car - RentCompany

feat: 요구 class 생성 및 제공된 템플릿 코드 작성

7ab8c03

- Avante, K5, Sonata 클래스 생성 - 제공된 테스트 코드 추가

feat: 테스트 코드 구현을 위한 메서드 시그니처 생성

95fbf47

- constructor - getDistancePerLiter() - getTripDistance() - getName()

feat: Car의 파생클래스(Sonata, Avante, K5) 오버라이딩 메서드 구현

b2c2920

feat: RentCompany.generateReport 메서드 구현

679cd13

refactor: 파생 클래스 car name 상수처리

1501e4f

docs: Update README.md

eab1872

- 기능 구현 사항 리스트 완료 처리

refactor: 사용하지 않는 import문 제거

eb94b7d

docs: update README.md

f7ee342

- 블랙잭 게임 기능 구현 사항 작성

김정호 님이 변경한 내용

361bb4e

chore: 프로젝트 수행을 위한 기본 구성 파일 추가

3a7fe0c

classes - BlackjackGame - Dealer - Deck - Denomination - Gambler - Judgement - InputView - OutputView - Player - Suit

chore: 프로젝트 테스트를 위한 테스트 코드 추가

648a2dc

classes - DealerTest - DeckTest - DenominationTest - GamblerTest - JudgementTest - PlayerTest - SuitTest

chore: 프로젝트 수행을 위한 기본 구성 파일 추가

8ac61f1

classes -Card

feat: Dealer 객체 메소드 추가

2e16386

- allocateCard()

feat: enum 객체 작성

b8bc27f

- Denomination

feat: Gambler 객체 기본 템플릿 작성

c808b5f

feat: Player 객체 메소드 작성

faa242d

docs: Update README.md

18e892c

- 기능 구현 완료 사항 체크 표시

feat: enum 객체 작성

17e065d

-Suit

feat: Implement Card class

6de51e6

- enum타입의 suit, denomination값을 가지는 Card class 설계

feat: Implement Deck class

ebb2b54

- deck class 구현 - field : List card - method: setupCard, setDenomination, shuffle, popCard

feat: Gambler 추상 클래스 제거 및 Player, Dealer 재설계

5a6a77f

- 중복 코드 제거를 위함 - Dealer#allocateCard 메서드 구현

feat: Player 점수 계산 메소드 구현

42c5843

- calculateScore - adjustScore

test(PlayerTest) : Player 테스트 코드 작성

1f1471e

feat: Player 점수 계산 메소드 구현

678b79a

docs(README): 구현 완료한 기능 체크 표시

24cd7d0

feat: Dealer의 카드 합이 16이하일 때 카드를 추가하는 메소드 구현

c10faf7

-addOneMoreCard

feat: Dealer의 카드 합이 16이하일 때 카드를 추가하는 메소드 구현

75b59e2

-addOneMoreCard

test: Dealer의 addOneMoreCard 기능 테스트 작성

c2e535f

LeahJiinLee and others added 13 commits February 16, 2022 17:50

feat: Dealer의 카드 합이 16이하일 때 카드를 추가하는 메소드 구현

7f565e1

-addOneMoreCard

feat:Judgement 우승자 찾기 메소드 구현

8441a30

- findWinner()

test: Judgement 우승자 찾기 기능 테스트 코드 작성

259ea44

feat:Judgement 승패 찾기 메소드 수정 및 승패계산

e079604

- findWinner() - checkWinOrLost() - getDealerResult() - getCount()

test: Judgement 우승자 찾기 기능 테스트 코드 작성

fa79f66

test(JudgementTest): JudgeMent 테스트 코드 작성

0b6eb9a

- testFindWinner

test: Judgement 우승자 찾기 기능 테스트 코드 작성

f2575e4

docs(README): 구현 완료한 기능 체크 표시

3ce951c

test: Judgement 우승자 찾기 기능 테스트 코드 작성

232fc23

feat: Implement InputView and OutputView

2839a04

fix: static Scanner 객체로 인한 테스트 실패 이슈 해결

bebd646

feat: Controller 및 블랙잭 application 구현완료

e8f4aed

chore: google convention에 따른 코드 정렬

07c2a26

joswlv requested changes Feb 21, 2022

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[김정호] 연료 주입, 블랙잭 (Step 1)#34

[김정호] 연료 주입, 블랙잭 (Step 1)#34
Hoya-kim wants to merge 43 commits intonext-step:hoya-kimfrom
Hoya-kim:step1

Hoya-kim commented Feb 17, 2022

Uh oh!

joswlv left a comment

Uh oh!

joswlv Feb 19, 2022

Uh oh!

joswlv Feb 19, 2022

Uh oh!

joswlv Feb 19, 2022

Uh oh!

joswlv Feb 21, 2022

Uh oh!

joswlv Feb 21, 2022

Uh oh!

joswlv Feb 21, 2022

Uh oh!

joswlv Feb 21, 2022

Uh oh!

joswlv Feb 21, 2022

Uh oh!

joswlv Feb 21, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

	public String getDenomiationCountstring() {
	public String getDenomiationCountString() {

		List<Player> players = playersName.stream().map(Player::new)
		.collect(Collectors.toList());

		@@ -0,0 +1,29 @@
		package blackjack.domain;

		public class Card {

-            System.out.println(name + COLON + result);
+privat static final String FINAL_RESULT_MESSAGE = "%s : %s";
+      ...
+            System.out.println(String.format(FINAL_RESULT_MESSAGE, name, result));

		@@ -0,0 +1,39 @@
		package blackjack.domain;

		public class Dealer extends Player {

Conversation

Hoya-kim commented Feb 17, 2022

Uh oh!

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

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants