[최현민] 연료 주입, 블랙잭 - Step1 by Gal2o · Pull Request #39 · next-step/java-blackjack

Gal2o · 2022-02-17T16:03:30Z

이현호 님과 페어프로그래밍을 통해 미션을 진행했습니다.

추상 클래스를 많이 사용해보지 않아서 배울 수 있었던 기회였던 것 같습니다.

블랙잭 미션은 너무 어려워서 코드가 상당히 난잡해졌는데

피드백 주시면 빠르게 고쳐나가도록 하겠습니다.

감사합니다

- 기능 요구 사항 작성

- 오타 제거

- 팩토리 매세드 적용

- addCar 기능 추가

- generateReport 기능 추가

…메서드 생성

Rok93

현민님 안녕하세요! 블랙잭 미션을 함께하게 된 김경록입니다. 🙇‍♂️
블랙잭 미션 잘 구현해주셨네요! 👍
몇몇 피드백 남겨두었으니 확인해서 반영해주세요! 🙏
이해가 잘 안되거나 어려운 부분은 언제든지 DM 주세요! 😉

Rok93 · 2022-02-19T02:56:39Z

src/main/java/rentcompany/Avante.java

+    private static final double DISTANCE_PER_LITER = 15;
+    private double distance;


상수와 인스턴스 변수는 한 줄 띄어서 가독성을 높여주세요! 🙏

Rok93 · 2022-02-19T02:58:46Z

src/main/java/rentcompany/K5.java

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


Car의 모든 하위 구현체들이 distance를 상태로 가지고 getTripDistacne() 또한 동일한 결과 값을 반환하는 것 같아요. 🤔
추상 클래스인 Car에게 이 인스턴스 변수를 할당하고 메서드도 정의하면 반복되는 코드를 줄일 수 있을 것 같아요. 😃

Rok93 · 2022-02-19T03:00:15Z

src/main/java/rentcompany/RentCompany.java

+        for(Car car : cars){
+            report.append(car.getName()+" : "+(int)car.getChargeQuantity()+ LITER + NEW_LINE);
+        }


IDE의 코드 자동 정렬 기능(mac 기준 ⌘ + ⌥ + L)을 습관적으로 실행시켜주세요! 🙏

Suggested change

for(Car car : cars){

report.append(car.getName()+" : "+(int)car.getChargeQuantity()+ LITER + NEW_LINE);

}

for (Car car : cars) {

report.append(car.getName() + " : " + (int) car.getChargeQuantity() + LITER + NEW_LINE);

}

Rok93 · 2022-02-19T03:11:11Z

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

+    private static List<Card> makeBlackJackCards() {
+        return Arrays.stream(CardNumber.values())
+            .flatMap(number ->
+                Arrays.stream(CardType.values())
+                    .map(type -> new Card(number, type)))
+            .collect(Collectors.toList());
+    }


Deck으로 쓰이는 카드들은 카드의 형태와 갯수들이 정해져있고 재사용할 확률이 높다고 생각해요. 😃
미리 만들어뒀다가 필요한 시점에 재사용해보면 어떨까요? 🤗

참고 자료

Rok93 · 2022-02-19T03:25:12Z

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

+
+    public Card cardDraw() {
+        if (cards.isEmpty()) {
+            throw new RuntimeException(INVALID_DRAW);


RuntimeException의 하위 (예외) 클래스는 100개가 넘는다고 해요. 👀
지금처럼 예외 처리로 RuntimeException을 던지게 되면, 이 예외를 catch하는 경우 해당 예외뿐만이 아니라 예상치 못했던 에러까지 잡히는 문제가 발생해요.🙂
즉, 예상치 못한 에러가 예외 처리돼버려서 애플리케이션에 결함을 발견하지 못할 수도 있어요. 🥲
구체적인 예외 클래스를 사용해서 예외 처리하도록 변경해주세요! 🙏

Rok93 · 2022-02-19T06:27:15Z

src/main/java/blackjack/domain/GameResult.java

+        if (playerDeck.getTotalScore() == dealerDeck.getTotalScore()) {
+            return DRAW;
+        }
+
+        if (playerDeck.getTotalScore() > dealerDeck.getTotalScore()) {
+            return WIN;


총점수를 계산하는 불필요한 연산이 반복되고 있어요! 🥲

playerDeck과 dealerDeck의 getTotalScore 메서드를 매번 호출하지 말고 필요한 시점에 한 번 호출할 때, 변수로 추출해둔 이후에 재사용하도록 변경해주세요! 🙏

Rok93 · 2022-02-19T06:29:31Z

src/main/java/blackjack/domain/GameResult.java

+        return LOSE;
+    }
+
+    public List<String> getDealerGameResult() {


get, set으로 시작하는 메서드 네이밍은 지양하는 것이 좋아요. 😃
자바 진영에서는 관습적으로 객체의 상태를 외부로 꺼내는 메서드를 getter, 객체의 상태를 변경하는 메서드를 setter로 정의하기 때문에 일반적인 getter 메서드와 혼동될 수 있어요. 🙂

Rok93 · 2022-02-19T06:29:42Z

src/main/java/blackjack/domain/GameResult.java

+            .entrySet()
+            .stream()
+            .sorted(Map.Entry.comparingByKey())
+            .map(m -> m.getValue() + m.getKey().getName())


람다식 변수도 네이밍에 신경써주세요! 🙏

Rok93 · 2022-02-19T07:23:04Z

src/main/java/blackjack/domain/OwnCards.java

+    }
+
+    public int getTotalScore() {
+        int total = ownCards.stream().mapToInt(Card::getCardNumber).sum();


stream은 한줄에 하나의 메서드만 호출하도록 해서 가독성도 높이고 인텔리제이의 편의 기능도 이용하도록 변경해주세요! 🙏

Rok93 · 2022-02-19T07:23:30Z

src/main/java/blackjack/domain/OwnCards.java

+        if (total <= BLACKJACK - ACE_NUMBER_TEN && ownCards.stream().anyMatch(Card::isAceCard)) {
+            return total + ACE_NUMBER_TEN - ACE_NUMBER_ONE;
+        }


아래와 같이 변경해보면 어떨까요? 🤗

Suggested change

if (total <= BLACKJACK - ACE_NUMBER_TEN && ownCards.stream().anyMatch(Card::isAceCard)) {

return total + ACE_NUMBER_TEN - ACE_NUMBER_ONE;

}

boolean hasAceCard = ownCards.stream().anyMatch(Card::isAceCard);

if (total + ACE_NUMBER_TEN <= BLACKJACK && hasAceCard) {

return total + ACE_NUMBER_TEN - ACE_NUMBER_ONE;

}

hyeonho8453 and others added 30 commits February 11, 2022 16:28

docs(rentcompany): README.md 작성

7f13e05

- 기능 요구 사항 작성

docs(rentcompany): 불필요한 파일 삭제

af63466

feat(rentcompany): Car 클래스 생성

d0ece66

refactor(rentcompany): Car 클래스 수정

11efcf5

- 오타 제거

feat(rentcompany): Sonata, Avante, K5 클래스 생성

c91d116

feat(rentcompany): RentCompany 클래스 생성

c27bd16

feat(rentcompany): RentCompany 클래스 수정

aee5205

- 팩토리 매세드 적용

feat(rentcompany): RentCompany 클래스 기능 추가

4f354df

- addCar 기능 추가

feat(rentcompany): RentCompany 클래스 기능 추가

03bda82

- generateReport 기능 추가

feat(rentcompany): RentCompanyTest 생성

0470c59

docs(Blackjack): 블랙잭 README 추가

8f83230

feat(Blackjack): 블랙잭 패키지, 클래스 생성

86813d5

feat(Blackjack): 블랙잭 게임을 하는 플레이어 추상 클래스 생성

cdadf6d

feat(Blackjack): GamePlayers를 상속받는 Dealer, Player 생성

b241605

feat(Blackjack): GamePlayers에 필요한 사용자들이 사용하는 OwnCards 생성

f288843

feat(Blackjack): 카드의 번호, 이름을 가지는 CardNumber 생성

b1233be

feat(Blackjack): 카드의 문양을 가지는 CardType 생성

ec4503d

feat(Blackjack): CardNumber, CardType을 갖는 Card 생성

5d92ef8

feat(Blackjack): 게임에 사용할 52장의 덱을 만드는 Deck 생성

0f92554

feat(Blackjack): 게임 플레이어가 처음 두장의 카드 뽑는 메소드 생성

28228aa

feat(Blackjack): 사용자가 가진 카드를 통해 카드들의 총 합을 구하는 메서드, 블랙잭인지, 초과했는지 확인하는 …

a26e00a

…메서드 생성

feat(Blackjack): 딜러, 플레이어가 카드를 뽑을 수 있는지 확인하는 오버라이딩 메서드 생성

c552ba4

test(Blackjack): CardNumber 테스트 생성

a9dbeb2

test(Blackjack): CardType 테스트 생성

53e1520

test(Blackjack): Card 테스트 생성

38de5d3

test(Blackjack): Deck 테스트 생성

497ab0a

test(Blackjack): OwnCardTest 테스트 생성

d6233dc

feat(Blackjack): 딜러, 플레이어가 카드 한장 뽑는 메서드 생성

e8ac38b

test(Blackjack): Gameplayers, 상속 클래스들 테스트 생성

dee7ffc

feat(Blackjack): player를 담는 Players 클래스 생성

597f4c3

Gal2o added 9 commits February 18, 2022 00:31

test(Blackjack): Players 테스트 생성

88febd1

feat(Blackjack): 처음 딜러, 플레이어 생성하는 InputView, Parser 생성

b2e2f12

feat(Blackjack): 처음 게임 시작에 필요한 ResultView-inputPlayersPrint 메서드 생성

8de7a71

feat(Blackjack): Application에서 BlackJackController 실행 메서드 생성

9e8dc3a

feat(Blackjack): 딜러, 플레이어가 처음 카드를 받고 추가 카드 받는 InputView, OutputView 생성

b637290

feat(Blackjack): 게임 결과에 사용할 GameResult, ResultValue 클래스 생성

2c62248

test(Blackjack): GameResult, ResultValue 테스트 생성

6dcec25

feat(Blackjack): 게임 결과를 출력하는 메서드 생성

3dcb065

docs(Blackjack): 블랙잭 README 수정

abeae48

Rok93 requested changes Feb 19, 2022

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[최현민] 연료 주입, 블랙잭 - Step1#39

[최현민] 연료 주입, 블랙잭 - Step1#39
Gal2o wants to merge 39 commits intonext-step:gal2ofrom
Gal2o:step1

Gal2o commented Feb 17, 2022

Uh oh!

Rok93 left a comment

Uh oh!

Rok93 Feb 19, 2022

Uh oh!

Rok93 Feb 19, 2022

Uh oh!

Rok93 Feb 19, 2022

Uh oh!

Rok93 Feb 19, 2022

Uh oh!

Rok93 Feb 19, 2022

Uh oh!

Rok93 Feb 19, 2022

Uh oh!

Rok93 Feb 19, 2022

Uh oh!

Rok93 Feb 19, 2022

Uh oh!

Rok93 Feb 19, 2022

Uh oh!

Rok93 Feb 19, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		private static final double DISTANCE_PER_LITER = 15;
		private double distance;

Conversation

Gal2o commented Feb 17, 2022

Uh oh!

Rok93 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