[1주차] 정재우

README.md

@@ -43,7 +43,8 @@ BUILD SUCCESSFUL in 0s
         - 123을 제시한 경우 : 1스트라이크
         - 456을 제시한 경우 : 1볼 1스트라이크
         - 789를 제시한 경우 : 낫싱
-- 위 숫자 야구 게임에서 상대방의 역할을 컴퓨터가 한다. 컴퓨터는 1에서 9까지 서로 다른 임의의 수 3개를 선택한다. 게임 플레이어는 컴퓨터가 생각하고 있는 서로 다른 3개의 숫자를 입력하고, 컴퓨터는 입력한 숫자에 대한
+- 위 숫자 야구 게임에서 상대방의 역할을 컴퓨터가 한다. 컴퓨터는 1에서 9까지 서로 다른 임의의 수 3개를 선택한다. 게임 플레이어는 컴퓨터가 생각하고 있는 서로 다른 3개의 숫자를 입력하고, 컴퓨터는 입력한
+  숫자에 대한
   결과를 출력한다.
 - 이 같은 과정을 반복해 컴퓨터가 선택한 3개의 숫자를 모두 맞히면 게임이 종료된다.
 - 게임을 종료한 후 게임을 다시 시작하거나 완전히 종료할 수 있다.
@@ -126,13 +127,13 @@ BUILD SUCCESSFUL in 0s
 #### 사용 예시
 
 ```java
-List<Integer> computer = new ArrayList<>();
-while (computer.size() < 3) {
-    int randomNumber = Randoms.pickNumberInRange(1, 9);
-    if (!computer.contains(randomNumber)) {
+List<Integer> computer=new ArrayList<>();
+        while(computer.size()< 3){
+        int randomNumber=Randoms.pickNumberInRange(1,9);
+        if(!computer.contains(randomNumber)){
         computer.add(randomNumber);
-    }
-}
+        }
+        }
 ```
 
 ---

docs/README.md

@@ -0,0 +1,60 @@
+# 미션 - 숫자 야구
+
+## ✅ 기능 구현 목록
+
+### 게임 준비
+
+- [x] 컴퓨터의 숫자를 랜덤하게 정한다.
+
+### 게임 진행
+
+- [x] 플레이어의 숫자를 입력받는다.
+    - [x] 숫자를 입력받는다.
+    - [x] 문자열 타입의 숫자를 리스트로 변환한다.
+    - [x] 입력값을 검증한다.
+        - [x] 입력된 문자열이 비어있지 않은지 검증한다.
+        - [x] 입력된 문자열의 길이가 3인지 검증한다.
+        - [x] 입력된 문자열이 정수인지 검증한다.
+        - [x] 입력된 문자열이 양의 정수인지 검증한다.
+        - [x] 입력된 정수들의 범위가 1부터 9인지 검증한다.
+        - [x] 입력된 정수들이 서로 다른 수인지 검증한다.
+    - [x] 입력값이 부적절한 경우, 예외를 반환한다.
+
+
+- [x] 플레이어가 입력한 숫자와 컴퓨터의 숫자를 비교한다.
+    - [x] 플레이어가 입력한 각 숫자가 컴퓨터의 숫자에 있는지 확인한다.
+        - [x] 플레이어가 입력한 숫자와 위치를 비교한다.
+        - [x] 숫자 비교에 따라 힌트를 결정한다.
+
+
+- [x] 결정된 힌트를 출력한다.
+- [x] 플레이어가 숫자를 맞추지 못한 경우, 다시 `게임 준비`로 돌아가 숫자를 입력받는다.
+- [x] 플레이어가 3개의 숫자를 모두 맞췄다면 게임을 다시 시작할 지, 종료할 지 결정한다.
+    - [x] "1"을 누르면 게임을 다시 시작, "2"를 누르면 게임을 종료한다.
+
+---
+
+## 🛠️ 리팩토링 목록
+
+### 실행 로직 간소화 및 성능 향상
+
+- [x] 숫자 비교에 따른 힌트 설정 로직을 간소화하고 성능을 향상시킨다.
+- [x] for문을 stream 메소드를 사용하여 간소화한다.
+
+### 일부 변수들의 상수 변환
+
+- [x] Hint 클래스 관련 힌트 목록을 Enum으로 분리한다.
+- [x] 입력값 검증 시의 검증 조건들을 Enum으로 분리한다.
+- [x] 입력값 검증 시 발생하는 예외의 목록을 Enum으로 분리한다.
+- [x] View 계층에서 출력하는 메시지의 목록을 Enum으로 분리한다.
+
+### getter 및 setter 제거
+
+- [x] Player 클래스의 getter 및 setter를 제거하고 비즈니스 로직을 도메인 내에 구현한다.
+    - [x] Player 클래스의 setter를 제거하고 생성 시에만 필드 변수값을 정할 수 있도록 한다.
+    - [x] Player 클래스의 getter를 제거하고 비즈니스 로직을 도메인 내에 구현한다.
+
+### 기타
+
+- [x] ViewService의 기능을 InputView와 OutputView로 각각 분리한다.
+- [x] NumberService와 InputView, OutputView에 싱글톤 패턴을 적용한다.

src/main/java/baseball/Application.java

@@ -1,7 +1,10 @@
 package baseball;
 
+import baseball.controller.GameController;
+
 public class Application {
     public static void main(String[] args) {
-        // TODO: 프로그램 구현
+        GameController game = new GameController();
+        game.run();
     }
 }

src/main/java/baseball/application/NumberService.java

@@ -0,0 +1,57 @@
+package baseball.application;
+
+import baseball.domain.Hint;
+import baseball.domain.Player;
+import camp.nextstep.edu.missionutils.Randoms;
+import java.util.ArrayList;
+import java.util.HashSet;
+import java.util.List;
+import java.util.Objects;
+import java.util.Set;
+
+public class NumberService {
+
+    private static final NumberService instance = new NumberService();
+
+    public static NumberService getInstance() {
+        return instance;
+    }
+
+    public List<Integer> getRandomNumber() {
+        Set<Integer> numberSet = new HashSet<>();
+        while (numberSet.size() < 3) {
+            numberSet.add(Randoms.pickNumberInRange(1, 9));
+        }
+
+        return new ArrayList<>(numberSet);
+    }
+
+    public Hint generateHint(Player computer, Player player) {
+        List<Integer> computerNumber = computer.getNumberList();
+        List<Integer> playerNumber = player.getNumberList();
+        Hint hint = new Hint();
+
+        compareNumber(computerNumber, playerNumber, hint);
+
+        return hint;
+    }
+
+    private static void compareNumber(List<Integer> computerNumber, List<Integer> playerNumber, Hint hint) {
+        for (int i = 0; i < playerNumber.size(); i++) {
+            Integer myNumber = playerNumber.get(i);
+            if (isStrike(computerNumber, i, myNumber)) {
+                hint.increaseStrike();
+            } else if (isBall(computerNumber, myNumber)) {
+                hint.increaseBall();
+            }
+        }
+    }
+
+    private static boolean isBall(List<Integer> computerNumber, Integer myNumber) {
+        return computerNumber.contains(myNumber);
+    }
+
+    private static boolean isStrike(List<Integer> computerNumber, int i, Integer myNumber) {
+        return Objects.equals(myNumber, computerNumber.get(i));
+    }
+}

src/main/java/baseball/constant/ErrorMessage.java

@@ -0,0 +1,23 @@
+package baseball.constant;
+
+public enum ErrorMessage {
+
+    INPUT_EMPTY("입력값은 비어있지 않아야 합니다."),
+    INPUT_LENGTH_INVALID("입력값은 3자리이어야 합니다."),
+    INPUT_NUMBER_NOT_INTEGER("입력값은 정수이어야 합니다."),
+    INPUT_NUMBER_NOT_POSITIVE_INTEGER("입력값은 양의 정수이어야 합니다."),
+    INPUT_NUMBER_NOT_IN_RANGE("입력되는 각 숫자는 1 이상 9 이하의 정수로 이루어져야 합니다."),
+    INPUT_NUMBER_NOT_UNIQUE("입력되는 각 숫자는 서로 다른 숫자이어야 합니다.");
+
+
+    private final String message;
+
+
+    ErrorMessage(String message) {
+        this.message = message;
+    }
+
+    public String getMessage() {
+        return message;
+    }
+}

src/main/java/baseball/constant/HintMessage.java

@@ -0,0 +1,19 @@
+package baseball.constant;
+
+public enum HintMessage {
+
+    ONLY_BALL("%d볼"),
+    ONLY_STRIKE("%d스트라이크"),
+    NOTHING("낫싱");
+
+    private final String message;
+
+
+    HintMessage(String message) {
+        this.message = message;
+    }
+
+    public String getMessage() {
+        return message;
+    }
+}

src/main/java/baseball/constant/SystemMessage.java

@@ -0,0 +1,20 @@
+package baseball.constant;
+
+public enum SystemMessage {
+
+    TYPE_NUMBER("숫자를 입력해주세요 : "),
+    GAME_END("3개의 숫자를 모두 맞히셨습니다! 게임 종료"),
+    GAME_RESTART("게임을 새로 시작하려면 1, 종료하려면 2를 입력하세요."),
+    EXIT_FLAG("1"),
+    RESTART_FLAG("2");
+
+    private final String message;
+
+    SystemMessage(String message) {
+        this.message = message;
+    }
+
+    public String getMessage() {
+        return message;
+    }
+}

src/main/java/baseball/controller/GameController.java

@@ -0,0 +1,52 @@
+package baseball.controller;
+
+import static baseball.constant.SystemMessage.RESTART_FLAG;
+import static baseball.util.NumberUtil.convertStringToList;
+
+import baseball.application.NumberService;
+import baseball.domain.Hint;
+import baseball.domain.Player;
+import baseball.view.InputView;
+import baseball.view.OutputView;
+import java.util.Objects;
+
+public class GameController {
+
+    private final NumberService numberService = NumberService.getInstance();
+    private final InputView inputView = InputView.getInstance();
+    private final OutputView outputView = OutputView.getInstance();
+
+    public void run() {
+        do {
+            Player computer = prepareComputer();
+
+            play(computer);
+
+        } while (isGameRestarted()); // 종료(2)를 누르지 않으면 게임을 다시 시작한다.
+    }
+
+    private void play(Player computer) {
+        while (true) {
+            Player player = preparePlayer();
+
+            Hint hint = numberService.generateHint(computer, player); // 컴퓨터와 플레이어의 숫자를 비교하여 힌트를 얻는다.
+            outputView.printHintMessage(hint); // 힌트를 출력, 만약 숫자가 맞는다면 게임 종료 메시지도 출력한다.
+            if (hint.isThreeStrike()) { // 숫자를 맞췄다면 끝낸다.
+                break;
+            }
+        }
+    }
+
+    private boolean isGameRestarted() {
+        return !Objects.equals(inputView.readNumber(), RESTART_FLAG.getMessage());
+    }
+
+    private Player prepareComputer() {
+        return new Player(numberService.getRandomNumber());
+    }
+
+    private Player preparePlayer() {
+        outputView.printReadMessage();
+        return new Player(convertStringToList(inputView.readNumber()));
+    }
+}

src/main/java/baseball/domain/Hint.java

@@ -0,0 +1,71 @@
+package baseball.domain;
+
+import static baseball.constant.HintMessage.NOTHING;
+import static baseball.constant.HintMessage.ONLY_BALL;
+import static baseball.constant.HintMessage.ONLY_STRIKE;
+import static java.lang.String.format;
+import static java.lang.String.join;
+
+import java.util.ArrayList;
+
+public class Hint {
+
+    private int strike;
+
+    private int ball;
+
+    public Hint() {
+        strike = 0;
+        ball = 0;
+    }
+
+    public void increaseStrike() {
+        strike++;
+    }
+
+    public void increaseBall() {
+        ball++;
+    }
+
+    public boolean isHintNothing() {
+        return isStrikeExist() && isBallExist();
+    }
+
+    public boolean isStrikeExist() {
+        return strike > 0;
+    }
+
+    public boolean isBallExist() {
+        return ball > 0;
+    }
+
+    public boolean isThreeStrike() {
+        return strike == 3;
+    }
+
+    public String toString() {
+        ArrayList<String> resultString = new ArrayList<>();
+        if (!isBallExist() && !isStrikeExist()) {
+            return getNothingString();
+        }
+        if (isBallExist()) {
+            resultString.add(getBallString());
+        }
+        if (isStrikeExist()) {
+            resultString.add(getStrikeString());
+        }
+        return join(" ", resultString);
+    }
+
+    public String getStrikeString() {
+        return format(ONLY_STRIKE.getMessage(), strike);
+    }
+
+    public String getBallString() {
+        return format(ONLY_BALL.getMessage(), ball);
+    }
+
+    public String getNothingString() {
+        return NOTHING.getMessage();
+    }
+}

src/main/java/baseball/domain/Player.java

@@ -0,0 +1,22 @@
+package baseball.domain;
+
+import static baseball.util.NumberUtil.convertListToString;
+import static baseball.util.NumberUtil.validate;
+
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.List;
+
+public class Player {
+
+    private final List<Integer> numberList;
+
+    public Player(List<Integer> numbers) {
+        validate(convertListToString(numbers));
+        this.numberList = new ArrayList<>(numbers);
+    }
+
+    public List<Integer> getNumberList() {
+        return Collections.unmodifiableList(numberList);
+    }
+}