[2단계 - 사다리 게임 실행] 테오(최우성) 미션 제출합니다.

docs/README.md

@@ -3,23 +3,39 @@
 ### UI
 
 - 입력 
-  - [x] 참여할 사람 이름 입력 문구
-  - [x] 최대 사다리 높이 입력 문구
+  - [x] 참여할 사람 이름 입력
+  - [x] 최대 사다리 높이 입력
+  - [x] 사다리 게임 실행 결과 입력 
+  - [x] 결과를 보고 싶은 사람 입력
+
+
 - 출력
   - [x] `실행 결과` 출력
   - [x] 참여자 목록 출력
   - [x] 사다리 게임 결과 출력
+  - [x] 개인 별 결과 출력
+  - [x] 전체 결과 출력
+
 
 ### 기능
 - **플레이어**
 - [x] 플레이어의 이름을 최소 1자, 최대 5자까지 입력받을 수 있다.
 - [x] 플레이어는 최소 2명 이상 참여해야 한다.
+- [x] 플레이어 벌 사다리 게임 결과를 저장한다.
 
 
 - **사다리**
 - [x] 사다리 높이 입력을 통해 사다리를 생성한다.
   - [x] 사다리 발판을 랜덤으로 생성한다.
   - [x] 만약 만드려는 사다리 발판의 왼쪽에 발판이 있는 경우 해당 위치의 발판은 만들지 않는다.
+- [x] 사다리의 한 곳이 어디로 이어지는지 반환한다.
+
+
+- **사다리 게임**
+- [x] 플레이어 별로 게임 결과를 생성한다.
+- [x] 게임 결과를 반환한다.
+  - [x] 특정 플레이어의 게임 결과를 반환한다.
+  - [x] 전체 플레이어의 게임 결과를 반환한다.
 
 ### 예외 사항
 - **플레이어**
@@ -31,6 +47,9 @@
 - [x] 사다리의 높이가 `(플레이어 수) - 1`이상이 아닌 경우 예외가 발생한다.
 
 
+- **사다리 게임**
+- [x] 없는 플레이어의 결과를 보고싶을 때 예외가 발생한다.
+
 ---
 
 ### 1단계 변경사항
@@ -42,17 +61,29 @@
 - [x] Line, Ladder 객체가 스스로 상태를 관리하도록 변경
 
 ### 1단계 고민사항
-- [ ] Block의 status 필드가 랜덤 값만을 위한 필드였기에 제거했는데, <br>
+- [x] Block의 status 필드가 랜덤 값만을 위한 필드였기에 제거했는데, <br>
   따라서 ***랜덤하게 Block을 가져오려면*** Blocks.values() 에서 가져와야 했음. <br>
   매번 Blocks.values()로 접근할 바에, RandomBlockGenerator에 캐싱함! 괜찮은 방법인가? <br>
 <br>
-- [ ] 서브웨이가 말해주신 것처럼, 시스템이 핵심 ***도메인 로직 - Maker 로직*** 으로 분리되어 있다. <br>
+
+- [x] 서브웨이가 말해주신 것처럼, 시스템이 핵심 ***도메인 로직 - Maker 로직*** 으로 분리되어 있다. <br>
   즉, 사다리 게임의 규칙을 어기는 Line, Ladder가 언제든지 탄생할 수 있다는 점도 문제이고, 핵심 도메인 객체가 <br>
   스스로 상태를 관리하지 못하는 것도 문제이다. 그러면 Builder 안의 로직을 Line, Ladder 안으로 옮기면 안되나? <br>
   왜 난 Builder를 만들었을까 생각해보니, 랜덤이 포함되어 있기 때문이다. 랜덤을 핵심 도메인 객체가 다뤄서는 테스트가 어렵고 <br>
   결국 랜덤 값을 외부에서 주입받는 형태가 되어야 하는데 .. 핵심 도메인 객체가 프로퍼티나 파라미터로 BlockGenerator를 받는 것도 <br>
   이상한 구조가 아닐까? <br>
 <br>
 
-
-
+---
+### 2단계 고민사항
+- [x] Player의 역할이 너무 빈약하지는 않은가? 객체 분리를 떠나서.
+  - [x] Player의 존재 의의는 Name과 같다. 따라서 둘을 합치자.
+- [ ] players가 책임이 늘어남에 따라 일급 컬렉션에서 벗어남. 요구사항을 벗어나게 되었음.
+- [ ] 프로퍼티는 가능한 없는 것이 좋은가? LadderGameController에서 LadderGame을 프로퍼티로 가지면서
+  의존성 주입을 받을지, 아니면 그냥 파라미터로 계속 Passing 할지 고민.
+- [ ] try - catch 안에 if, while 등 넣으면 들여쓰기가 2를 넘어가기에 안되는걸까? 규칙에 매몰된 것은 아닐까?
+- [ ] "all"에 관한 처리는 어떻게 할까? 컨트롤러에서 원시값 문자열을 그대로 다루는게 맞을까?
+- [x] 직접적인 setter는 아니지만, Players에서 게임 기록을 저장하는 부분이 setter와 동일하다고 볼 수 있지 않을까? 
+- [ ] commit log에 누구나 이해할 수 있게 기술적인 정보는 전혀 남기지 않음. 이런 방식이 괜찮은가?
+- [ ] FrontController를 정의하면서, OutputView, LadderGameController와의 의존성이 생기고
+예외 메세지가 더 많이 추가된다면 관리하기 힘들 것으로 예상됨. 어떻게 개선할 수 있을까?

src/main/java/ladder/Application.java

@@ -1,10 +1,19 @@
 package ladder;
 
+import ladder.controller.FrontExceptionController;
 import ladder.controller.LadderGameController;
+import ladder.view.InputView;
+import ladder.view.OutputView;
+
+import java.util.Scanner;
 
 public class Application {
 
     public static void main(String[] args) {
-        new LadderGameController().run();
+        new LadderGameController(
+                new InputView(new Scanner(System.in)),
+                new OutputView(),
+                new FrontExceptionController(new OutputView())
+        ).run();
     }
 }

src/main/java/ladder/controller/FrontExceptionController.java

@@ -0,0 +1,53 @@
+package ladder.controller;
+
+import ladder.domain.exception.LadderLengthException;
+import ladder.domain.exception.NoSuchPlayerException;
+import ladder.domain.exception.PlayerNameLengthException;
+import ladder.domain.exception.PlayerNumberException;
+import ladder.view.OutputView;
+
+import java.util.HashMap;
+import java.util.Map;
+
+public class FrontExceptionController {
+
+    private static final String MESSAGE_PREFIX = "[ERROR]";
+
+    private static final Map<Class<? extends RuntimeException>, String> messageSelector = new HashMap<>();
+
+    private final OutputView outputView;
+
+    public FrontExceptionController(OutputView outputView) {
+        this.outputView = outputView;
+        init();
+    }
+
+    private void init() {
+        addMessage();
+        addPrefix();
+    }
+
+    private void addMessage() {
+        addMessageWithDomain();
+        addMessageWithView();
+    }
+
+    private void addMessageWithDomain() {
+        messageSelector.put(LadderLengthException.class, "사다리의 길이는 <플레이어 수 - 1> 이상이어야 합니다.");
+        messageSelector.put(PlayerNameLengthException.class, "플레이어 이름의 길이는 1이상 5이하 입니다.");
+        messageSelector.put(PlayerNumberException.class, "플레이어 수는 두 명 이상이어야 합니다.");
+        messageSelector.put(NoSuchPlayerException.class, "해당하는 플레이어를 찾을 수 없습니다.");
+    }
+
+    private void addMessageWithView() {
+        messageSelector.put(NumberFormatException.class, "입력된 값은 정수가 아닙니다.");
+    }
+
+    private void addPrefix() {
+        messageSelector.forEach((k, v) -> messageSelector.put(k, MESSAGE_PREFIX + v));
+    }
+
+    public void handle(RuntimeException e) {
+        outputView.printExceptionMessage(messageSelector.get(e.getClass()));
+    }
+}

src/main/java/ladder/controller/LadderGameController.java

@@ -1,9 +1,10 @@
 package ladder.controller;
 
-import ladder.domain.rule.RandomBlockGenerator;
+import ladder.domain.game.LadderGame;
+import ladder.domain.player.Player;
+import ladder.domain.rule.RandomStoolGenerator;
 import ladder.domain.ladder.Ladder;
 import ladder.domain.player.Players;
-import ladder.exception.CustomException;
 import ladder.view.InputView;
 import ladder.view.OutputView;
 
@@ -12,43 +13,90 @@
 
 public class LadderGameController {
 
+    private static final String EXIST_SIGNAL = "all";
+
+    private final InputView inputView;
+    private final OutputView outputView;
+    private final FrontExceptionController frontExceptionController;
+
+    public LadderGameController(
+            InputView inputView,
+            OutputView outputView,
+            FrontExceptionController frontExceptionController
+    ) {
+        this.inputView = inputView;
+        this.outputView = outputView;
+        this.frontExceptionController = frontExceptionController;
+    }
+
     public void run() {
+        try {
+            LadderGame ladderGame = initLadderGame();
+
+            ladderGame.makeGameResult();
+            showLadder(ladderGame);
+            startGameResultLoop(ladderGame);
+        } catch (RuntimeException e) {
+            frontExceptionController.handle(e);
+        }
+    }
+
+    private LadderGame initLadderGame() {
         Players players = initPlayers();
         Ladder ladder = initLadder(players.size());
-
-        showResult(players, ladder);
+        return initLadderGame(players, ladder);
     }
 
     private Players initPlayers() {
         try {
-            List<String> playerNames = InputView.inputPlayer();
+            List<String> playerNames = inputView.inputPlayer();
             return new Players(playerNames);
-        } catch (CustomException e) {
-            OutputView.printErrorMessage(e);
+        } catch (RuntimeException e) {
+            frontExceptionController.handle(e);
             return initPlayers();
         }
     }
 
     private Ladder initLadder(int playerNumber) {
         try {
-            final int height = InputView.inputLadderHeight();
-            return new Ladder(playerNumber, height, new RandomBlockGenerator());
-        } catch (CustomException e) {
-            OutputView.printErrorMessage(e);
+            final int height = inputView.inputLadderHeight();
+            return new Ladder(playerNumber, height, new RandomStoolGenerator());
+        } catch (RuntimeException e) {
+            frontExceptionController.handle(e);
             return initLadder(playerNumber);
         }
     }
 
-    private void showResult(Players players, Ladder ladder) {
-        List<String> playersName = mapPlayersToPlayersName(players);
-        OutputView.printGameResultHeader();
-        OutputView.printPlayersName(playersName);
-        OutputView.printLadder(ladder);
+    private LadderGame initLadderGame(Players players, Ladder ladder) {
+        try {
+            List<String> destinations = inputView.inputLadderDestination();
+            return new LadderGame(players, ladder, destinations);
+        } catch (RuntimeException e) {
+            frontExceptionController.handle(e);
+            return initLadderGame(players, ladder);
+        }
+    }
+
+    private void showLadder(LadderGame ladderGame) {
+        outputView.printGameResultHeader();
+        outputView.printWithFormat(mapPlayersToPlayersName(ladderGame.getPlayers()));
+        outputView.printLadder(ladderGame.getLadder());
+        outputView.printWithFormat(ladderGame.getDestinations());
     }
 
     private List<String> mapPlayersToPlayersName(Players players) {
         return players.getPlayers().stream()
-                .map(player -> player.getPlayerName().getName())
+                .map(Player::getName)
                 .collect(Collectors.toUnmodifiableList());
     }
+
+    private void startGameResultLoop(LadderGame ladderGame) {
+        String playerName = inputView.inputPlayerWhoNeedsGameResult();
+
+        while (!playerName.equals(EXIST_SIGNAL)) {
+            outputView.printOneGameResult(ladderGame.getOneLadderGameResult(playerName));
+            playerName = inputView.inputPlayerWhoNeedsGameResult();
+        }
+        outputView.printAllGameResult(ladderGame.getAllLadderGameResult());
+    }
 }

src/main/java/ladder/domain/exception/CustomException.java

@@ -0,0 +1,4 @@
+package ladder.domain.exception;
+
+public class CustomException extends RuntimeException {
+}

src/main/java/ladder/domain/exception/LadderLengthException.java

@@ -0,0 +1,4 @@
+package ladder.domain.exception;
+
+public class LadderLengthException extends CustomException {
+}

src/main/java/ladder/domain/exception/NoSuchPlayerException.java

@@ -0,0 +1,4 @@
+package ladder.domain.exception;
+
+public class NoSuchPlayerException extends CustomException {
+}

src/main/java/ladder/domain/exception/PlayerNameLengthException.java

@@ -0,0 +1,4 @@
+package ladder.domain.exception;
+
+public class PlayerNameLengthException extends CustomException {
+}

src/main/java/ladder/domain/exception/PlayerNumberException.java

@@ -0,0 +1,4 @@
+package ladder.domain.exception;
+
+public class PlayerNumberException extends CustomException {
+}

src/main/java/ladder/domain/game/LadderGame.java

@@ -0,0 +1,53 @@
+package ladder.domain.game;
+
+import ladder.domain.ladder.Ladder;
+import ladder.domain.player.Player;
+import ladder.domain.player.Players;
+
+import java.util.ArrayList;
+import java.util.List;
+import java.util.Map;
+
+public class LadderGame {
+
+    private final Players players;
+    private final Ladder ladder;
+    private final List<String> destinations;
+
+    public LadderGame(Players players, Ladder ladder, List<String> destinations) {
+        this.players = players;
+        this.ladder = ladder;
+        this.destinations = List.copyOf(destinations);
+    }
+
+    public void makeGameResult() {
+        List<String> gameRecord = new ArrayList<>();
+
+        for (int location = 0; location < players.size(); location++) {
+            int finalDestination = ladder.getDestinationOf(location);
+            String gameResultForOneLocation = destinations.get(finalDestination);
+            gameRecord.add(gameResultForOneLocation);
+        }
+        players.recordGameResult(gameRecord);
+    }
+
+    public Map<Player, String> getAllLadderGameResult() {
+        return players.getAllGameRecords();
+    }
+
+    public Map.Entry<Player, String> getOneLadderGameResult(String playerName) {
+        return players.getGameRecordFor(playerName);
+    }
+
+    public Players getPlayers() {
+        return players;
+    }
+
+    public Ladder getLadder() {
+        return ladder;
+    }
+
+    public List<String> getDestinations() {
+        return List.copyOf(destinations);
+    }
+}