[🚀 사이클2 - 미션 (블랙잭 게임 실행)] 피노 미션 제출합니다.

README.md

@@ -139,5 +139,49 @@
 
 - 객체지향의 5원칙
 - SOLID
-- stream, record가 뭔지.
-- exception
+
+---
+
+## 두번째 사이클
+
+#### 미션 중 기록
+
+- 기능 추가로 인해 수정한 위치 개수
+    - main(9개) test(4개)
+
+- 사이클1 때보다 수정 범위가 줄었는가/늘었는가
+    - 수정 범위가 준 것 같습니다. 도메인에서 관리하던 string format들을 view에서 관리할 수 있도록 했습니다.
+    - 각 도메인들이 해야할 책임들을 잘 분배한 것 같습니다.
+
+- 규칙 적용으로 변경한 코드 1곳
+    - 이것은 생각이 좀 필요합니다.
+
+- 테스트가 설계를 도운 순간 1회
+    - 기능을 변경하면서 처음 테스트 코드를 생성한 곳은 BettingMoneyTest입니다.
+    - 당장 뭐부터 해야할 지 설계가 막막할 때 1000이하의 숫자를 입력해선 안된다는 규칙읋 세워 적용하였더니 수월하게 설계를 할 수 있었습니다.
+
+### 베팅 기능 추가 시 기능 요구사항
+
+- 겜블러들은 베팅을 할 수 있다.
+- 딜러와 견주어 승리 시 배팅금액의 1배 이다.
+- 블랙잭으로 승리 시 배팅금액의 1.5배 이다.
+- 버스트나 패배 시 배팅금액의 -1배 이다.
+- 무승부 시 수익은 0이 된다.
+- 최소 배팅 금액은 1000 이다.
+- 손해는 -로 표시한다. ex) -10000
+- 딜러의 수익은 **-(모든 겜블러들의 총 수익)** 이다.
+
+### 추가 도메인
+
+- **BettingMoney**
+- [x] 사용자가 0이나 음수를 입력하면 [ERROR] 메세지를 보낸다.
+- [x] 단위는 1000단위로 한다.
+
+- **MatchResult**
+- [x] 승/패/무 에 따라 수익률이 정해진다
+
+- **Gambler**
+- [x] 자신의 배팅 금액을 가진다.
+- [x] 자신의 수익 구한다.
+
+

src/main/java/BlackJack.java

@@ -1,8 +1,10 @@
+import domain.BettingMoney;
 import domain.deck.Deck;
 import domain.deck.StandardDeck;
 import domain.player.Dealer;
 import domain.player.Gambler;
 import domain.player.Gamblers;
+import java.util.ArrayList;
 import java.util.List;
 import parser.AnswerParser;
 import parser.PlayerNameParser;
@@ -20,7 +22,7 @@ public BlackJack() {
 
     public void start() {
         Dealer dealer = new Dealer();
-        Gamblers gamblers = new Gamblers(getPlayerNames());
+        Gamblers gamblers = new Gamblers(createGambler());
 
         initialDeal(dealer, gamblers);
         printInitialDealInfo(dealer, gamblers);
@@ -36,6 +38,17 @@ private List<String> getPlayerNames() {
         return PlayerNameParser.splitNames(InputView.readLine());
     }
 
+    private List<Gambler> createGambler() {
+        List<Gambler> gamblers = new ArrayList<>();
+        List<String> names = getPlayerNames();
+        for (String name : names) {
+            OutputView.requestBettingMoney(name);
+            int money = Integer.parseInt(InputView.readLine());
+            gamblers.add(new Gambler(name, new BettingMoney(money)));
+        }
+        return gamblers;
+    }
+
     private void initialDeal(Dealer dealer, Gamblers gamblers) {
         for (int i = 0; i < INITIAL_CARD_COUNT; i++) {
             dealer.deal(deck);
@@ -46,33 +59,34 @@ private void initialDeal(Dealer dealer, Gamblers gamblers) {
     private void printInitialDealInfo(Dealer dealer, Gamblers gamblers) {
         OutputView.printInitMessage(gamblers.getNames());
         OutputView.printDealerFirstCard(dealer.getFirstCard());
-
         for (Gambler gambler : gamblers.getGamblers()) {
             OutputView.printPlayerCards(gambler);
         }
+        OutputView.addNewLine();
     }
 
     private void gamblersTurn(Gamblers gamblers) {
         for (Gambler gambler : gamblers.getGamblers()) {
             gamblerTurn(gambler);
         }
+        OutputView.addNewLine();
     }
 
     private void gamblerTurn(Gambler gambler) {
         while (true) {
             OutputView.askHit(gambler.getName());
             boolean answer = AnswerParser.parse(InputView.readLine());
             if (!answer) {
+                OutputView.printPlayerCards(gambler);
                 break;
             }
 
             gambler.deal(deck);
-
+            OutputView.printPlayerCards(gambler);
             if (gambler.isBust()) {
                 OutputView.printPlayerBust(gambler.getName());
                 break;
             }
-            OutputView.printPlayerCards(gambler);
         }
     }
 
@@ -97,6 +111,7 @@ private void printFinalPlayerInfo(Dealer dealer, Gamblers gamblers) {
 
     private void printFinalResult(Dealer dealer, Gamblers gamblers) {
         OutputView.printFinalResultHeader();
-        OutputView.printResult(gamblers.getResult(dealer));
+        int dealerFinalIncome = gamblers.dealerFinalIncome(dealer);
+        OutputView.printResult(gamblers.getResult(dealer), dealerFinalIncome);
     }
 }

src/main/java/domain/BettingMoney.java

@@ -0,0 +1,30 @@
+package domain;
+
+import exception.BlackjackException;
+import exception.ExceptionMessage;
+
+public class BettingMoney {
+    private final int money;
+
+    public BettingMoney(int money) {
+        validateMoney(money);
+        validateMoneyUnit(money);
+        this.money = money;
+    }
+
+    private void validateMoneyUnit(int money) {
+        if (money % 100 != 0) {
+            throw new BlackjackException(ExceptionMessage.MONEY_UNIT_ERROR);
+        }
+    }
+
+    private void validateMoney(int money) {
+        if (money < 1000) {
+            throw new BlackjackException(ExceptionMessage.MONEY_INPUT_ERROR);
+        }
+    }
+
+    public int getMoney() {
+        return this.money;
+    }
+}

src/main/java/domain/MatchResult.java

@@ -1,17 +1,24 @@
 package domain;
 
 public enum MatchResult {
-    WIN("승"),
-    LOSE("패"),
-    DRAW("무");
+    BLACKJACK("블랙잭", 1.5),
+    WIN("승", 1.0),
+    LOSE("패", -1.0),
+    DRAW("무", 0.0);
 
     private final String name;
+    private final double ratio;
 
-    MatchResult(String name) {
+    MatchResult(String name, double ratio) {
         this.name = name;
+        this.ratio = ratio;
     }
 
     public String getName() {
         return name;
     }
+
+    public int calculateIncome(BettingMoney bettingMoney) {
+        return (int) (this.ratio * bettingMoney.getMoney());
+    }
 }

src/main/java/domain/deck/StandardDeck.java

@@ -3,8 +3,8 @@
 import domain.card.Card;
 import domain.card.CardRank;
 import domain.card.CardSuit;
-import expcetion.BlackjackException;
-import expcetion.ExceptionMessage;
+import exception.BlackjackException;
+import exception.ExceptionMessage;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collections;

src/main/java/domain/player/Gambler.java

@@ -1,9 +1,10 @@
 package domain.player;
 
+import domain.BettingMoney;
 import domain.MatchResult;
 import domain.card.Card;
-import expcetion.BlackjackException;
-import expcetion.ExceptionMessage;
+import exception.BlackjackException;
+import exception.ExceptionMessage;
 import java.util.List;
 
 public class Gambler extends Player {
@@ -13,14 +14,15 @@ public class Gambler extends Player {
     private static final String RESULT_FORMAT = "%s:%s";
 
     private final String name;
+    private final BettingMoney bettingMoney;
 
-    public Gambler(String name) {
+    public Gambler(String name, BettingMoney bettingMoney) {
         super();
+        this.bettingMoney = bettingMoney;
         validate(name);
         this.name = name;
     }
 
-
     private void validate(String name) {
         validateContainsNumber(name);
         validateLength(name);
@@ -51,18 +53,18 @@ public MatchResult getResult(Dealer dealer) {
         return MatchResult.DRAW;
     }
 
+    public int calculateFinalIncome(Dealer dealer) {
+        MatchResult matchResult = getResult(dealer);
+        return matchResult.calculateIncome(this.bettingMoney);
+    }
+
     private int normalize(int score) {
         if (score > BLACKJACK_MAX_LIMIT) {
             return 0;
         }
         return score;
     }
 
-
-    public String showResult(MatchResult result) {
-        return String.format(RESULT_FORMAT, name, result.getName());
-    }
-
     public String getName() {
         return name;
     }

src/main/java/domain/player/Gamblers.java

@@ -1,35 +1,33 @@
 package domain.player;
 
-import domain.MatchResult;
 import domain.deck.Deck;
-import dto.BlackjackResult;
-import expcetion.BlackjackException;
-import expcetion.ExceptionMessage;
-import java.util.ArrayList;
+import exception.BlackjackException;
+import exception.ExceptionMessage;
 import java.util.HashSet;
+import java.util.LinkedHashMap;
 import java.util.List;
+import java.util.Map;
 import java.util.Set;
 
 public class Gamblers {
     private final List<Gambler> gamblers;
 
-    public Gamblers(List<String> names) {
-        validateNonDuplicate(names);
-        gamblers = new ArrayList<>();
-        init(names);
+    public Gamblers(List<Gambler> gamblers) {
+        validateNonDuplicateNames(gamblers);
+        this.gamblers = gamblers;
     }
 
-    private void validateNonDuplicate(List<String> names) {
-        Set<String> namesSet = new HashSet<>(names);
-        if (namesSet.size() != names.size()) {
+    private void validateNonDuplicateNames(List<Gambler> gamblers) {
+        List<String> names = gamblers.stream()
+                .map(Gambler::getName)
+                .toList();
+
+        Set<String> targetNames = new HashSet<>(names);
+        if (targetNames.size() != names.size()) {
             throw new BlackjackException(ExceptionMessage.INPUT_ERROR);
         }
     }
 
-    private void init(List<String> names) {
-        names.stream().map(Gambler::new).forEach(gamblers::add);
-    }
-
     public List<Gambler> getGamblers() {
         return gamblers;
     }
@@ -38,21 +36,22 @@ public void dealAll(Deck deck) {
         gamblers.forEach(gambler -> gambler.deal(deck));
     }
 
-    public BlackjackResult getResult(Dealer dealer) {
-        int winCount = 0;
-        int loseCount = 0;
-        List<String> logs = new ArrayList<>();
+    public Map<String, Integer> getResult(Dealer dealer) {
+        Map<String, Integer> gamblersResult = new LinkedHashMap<>();
+        for (Gambler gambler : gamblers) {
+            String name = gambler.getName();
+            int finalIncomeMoney = gambler.calculateFinalIncome(dealer);
+            gamblersResult.put(name, finalIncomeMoney);
+        }
+        return gamblersResult;
+    }
+
+    public int dealerFinalIncome(Dealer dealer) {
+        int dealerFinalIncome = 0;
         for (Gambler gambler : gamblers) {
-            MatchResult result = gambler.getResult(dealer);
-            if (result == MatchResult.WIN) {
-                loseCount++;
-            }
-            if (result == MatchResult.LOSE) {
-                winCount++;
-            }
-            logs.add(gambler.showResult(result));
+            dealerFinalIncome += gambler.calculateFinalIncome(dealer);
         }
-        return new BlackjackResult(winCount, loseCount, gamblers.size() - winCount - loseCount, logs);
+        return dealerFinalIncome * -1;
     }
 
     public List<String> getNames() {

src/main/java/expcetion/BlackjackException.java → src/main/java/exception/BlackjackException.java

@@ -1,4 +1,4 @@
-package expcetion;
+package exception;
 
 public class BlackjackException extends IllegalArgumentException {
 

src/main/java/expcetion/ExceptionMessage.java → src/main/java/exception/ExceptionMessage.java

@@ -1,8 +1,10 @@
-package expcetion;
+package exception;
 
 public enum ExceptionMessage {
     EMPTY_CARD_DECK("[ERROR] 카드가 모두 소진되었습니다."),
-    INPUT_ERROR("[ERROR] 잘못된 입력입니다.");
+    INPUT_ERROR("[ERROR] 잘못된 입력입니다."),
+    MONEY_INPUT_ERROR("[ERROR] 1000 이상의 숫자를 입력해주시기 바랍니다."),
+    MONEY_UNIT_ERROR("[ERROR] 돈의 단위는 100입니다.");
 
     private final String message;
 

src/main/java/parser/AnswerParser.java

@@ -1,7 +1,7 @@
 package parser;
 
-import expcetion.BlackjackException;
-import expcetion.ExceptionMessage;
+import exception.BlackjackException;
+import exception.ExceptionMessage;
 
 public class AnswerParser {
     private AnswerParser() {

src/main/java/view/InputView.java

@@ -1,7 +1,7 @@
 package view;
 
-import expcetion.BlackjackException;
-import expcetion.ExceptionMessage;
+import exception.BlackjackException;
+import exception.ExceptionMessage;
 import java.io.BufferedReader;
 import java.io.IOException;
 import java.io.InputStreamReader;
@@ -19,4 +19,4 @@ public static String readLine() {
             throw new BlackjackException(ExceptionMessage.INPUT_ERROR);
         }
     }
-}
+}