[박찬우] 연료주입, 블랙잭 구현 #28

chanuuuuu · 2022-02-17T12:16:14Z

No description provided.

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>

- 게임 진행시 최초로 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>

hyunniiii

안녕하세요 찬우님!
몇가지 코멘트 남겼습니다.
확인 부탁드립니다 😄

src/test/java/fuelInjection/RentCompanyTest.java

src/main/java/blackJack/domain/CardType.java

src/main/java/blackJack/domain/CardNumber.java

src/main/java/blackJack/domain/Deck.java

src/main/java/blackJack/domain/Player.java

src/main/java/blackJack/util/Util.java

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

src/main/java/blackJack/domain/Card.java

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

- Intellij 제안에 따라 수정

- CardType의 Suit 사용 - CardNumber의 TrumpNumber 사용

- Deck의 추가 메서드 삭제 - Deck 메서드 순서 변경

- Card의 score 계산 로직 Card로 이동

- draw 여부 로직 이동

hyunniiii

안녕하세요 찬우님!
피드백 반영 잘해주셨습니다. 👍
몇가지 코멘트 남겼으니 확인부탁드립니다 ″̮

hyunniiii · 2022-02-23T12:56:20Z

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

+
+public class Game {
+
+    private static Game game = null;


이 부분이 필요한 이유는 무엇일까요?
없어도 되지 않을까요? ″̮

또한 Controller는 상태를 가지고 있지 않아도 될 것 같습니다 ″̮

hyunniiii · 2022-02-23T12:56:25Z

src/main/java/blackJack/domain/CardType.java

@@ -0,0 +1,18 @@
+package blackJack.domain;
+
+public class CardType {


CardType 객체를 생성하는 대신 Suit를 바로 사용해보는건 어떨까요?

hyunniiii · 2022-02-23T12:56:27Z

src/main/java/blackJack/domain/Dealer.java

+    }
+
+    public boolean isCardDraw() {
+        return deck.getScore() <= ADDITIONAL_CARD_DRAW_SCORE_THRESHOLD;


기준값을 파라미터로 넘겨주면 되지 않을까요?
객체는 자율적인 존재가 되어야합니다.
Dealer가 deck에게 너의 점수를 알려줘 하기보다는 deck이 스스로 판단하도록 하는게 자율적인 행동이 아닐까요?

또한 메서드 이름이 getScore에서 변경된다면 Dealer와 Player, Deck 총 세번의 변경이 일어나야합니다.
이렇게 여러번 변경되면 SOLID 원칙을 위반하게 되겠네요 ″̮

hyunniiii · 2022-02-23T12:56:38Z

src/main/java/blackJack/domain/Deck.java

+        return new Deck(deck);
+    }
+
+    public String convertDeckFormat() {


toString을 이용해보면 어떨까요?

hyunniiii · 2022-02-23T12:57:11Z

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

+public class GameUser {
+
+    private final Dealer dealer;
+    private final List<Player> players;


List<Player> players 를 일급컬렉션으로 만들어보는건 어떨까요?

hyunniiii · 2022-02-23T12:57:16Z

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

+    }
+
+    public List<Player> getPlayers() {
+        return players;


이렇게 players를 바로 넘겨주면 getPlayers().add()가 가능하게 됩니다.
방어적 복사를 이용해보는건 어떨까요?

https://tecoble.techcourse.co.kr/post/2021-04-26-defensive-copy-vs-unmodifiable/

hyunniiii · 2022-02-23T12:57:22Z

src/main/java/blackJack/domain/TrumpNumber.java

+    ACE("A"), TWO("2"), THREE("3"), FOUR("4"), FIVE("5"), SIX("6"), SEVEN("7"),
+    EIGHT("8"), NINE("9"), TEN("10"), JACK("J"), QUEEN("Q"), KING("K");
+


Suggested change

ACE("A"), TWO("2"), THREE("3"), FOUR("4"), FIVE("5"), SIX("6"), SEVEN("7"),

EIGHT("8"), NINE("9"), TEN("10"), JACK("J"), QUEEN("Q"), KING("K");

ACE("A"),

TWO("2"),

THREE("3"),

FOUR("4"),

FIVE("5"),

SIX("6"),

SEVEN("7"),

EIGHT("8"),

NINE("9"),

TEN("10"),

JACK("J"),

QUEEN("Q"),

KING("K");

길어지더라도 enum 정렬은 이렇게 해주는게 어떨까요?

hyunniiii · 2022-02-23T12:57:25Z

src/main/java/blackJack/domain/User.java

+    }
+
+    public String convertStatus() {
+        return String.format("%s 카드: %s", userName.getName(), deck.convertDeckFormat());


이 부분은 view의 책임이지 않을까요?

hyunniiii · 2022-02-23T12:57:29Z

src/main/java/blackJack/domain/UserStats.java

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


여기 있는 대부분 로직이 view와 관련되어 보여요.
view와 관련된 부분은 view로 옮겨주는건 어떨까요?

hyunniiii · 2022-02-23T12:57:37Z

src/main/java/blackJack/domain/UserStats.java

+        if (player.getScore() > Deck.MATCH_SCORE) {
+            return true;
+        }
+        return player.getScore() < gameUser.getDealer().getScore();


디미터 법칙을 위반하고 있네요.
위반하지 않도록 수정해주세요 ″̮

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 16 commits February 16, 2022 21:56

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>

chanuuuuu changed the title ~~연료주입, 블랙잭 구현~~ [박찬우] 연료주입, 블랙잭 구현 Feb 17, 2022

hochan222 and others added 4 commits February 18, 2022 17:01

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>

hyunniiii requested changes Feb 19, 2022

View reviewed changes

chanu94 added 7 commits February 21, 2022 18:28

refactor : RentCompanyTest 리팩토링

981ae7d

- Intellij 제안에 따라 수정

refactor : Enum 사용

1d218f0

- CardType의 Suit 사용 - CardNumber의 TrumpNumber 사용

refactor : Deck 리팩토링

08d0258

- Deck의 추가 메서드 삭제 - Deck 메서드 순서 변경

refactor : Card 리팩토링

c82c962

- Card의 score 계산 로직 Card로 이동

refactor : Dealer, Player 리팩토링

2f3bfd5

- draw 여부 로직 이동

refactor : 불필요한 메서드 제거

d2ece34

refactor : View 로직 Controller로 이동

d7e3bef

hyunniiii requested changes Feb 23, 2022

View reviewed changes

		@@ -0,0 +1,18 @@
		package blackJack.domain;

		public class CardType {

		ACE("A"), TWO("2"), THREE("3"), FOUR("4"), FIVE("5"), SIX("6"), SEVEN("7"),
		EIGHT("8"), NINE("9"), TEN("10"), JACK("J"), QUEEN("Q"), KING("K");

[박찬우] 연료주입, 블랙잭 구현 #28

Are you sure you want to change the base?

[박찬우] 연료주입, 블랙잭 구현 #28

Uh oh!

Conversation

chanuuuuu commented Feb 17, 2022

Uh oh!

hyunniiii left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

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!

Uh oh!