숫자 야구게임 구현 by DevRunner21 · Pull Request #2 · Run-Dev-Run/rdr-jihoon-java-baseball-playground

DevRunner21 · 2022-01-13T07:11:05Z

readme 작성 후 개발진행
TDD로 개발 진행
재귀용법 사용

재성님의 플이영상을 한번 봤더니
코드가 비슷해지는군요.. ㅜㅜ
자꾸 그게 생각이나다보니... ㅜㅜ
다음부터는 풀이강의는 최대한 늦게 봐야겠어요 ㅜㅜ

부족한 소스지만 시간 되실 때 리뷰 한번만 부탁드립니다!

제가 생각하기에 "좀 특이한 것 같다~" 라는 사항은 코멘트에 달아놨습니다.
감사합니다.

- 1~ 9까지의 숫자 체크 - 서로 다른 숫자 체크

- 위치, 숫자 모두 맞춘경우 -> 스트라이크 - 위치는 틀리고 숫자만 맞춘 경우 -> 볼 - 위치, 숫자 모두 틀린 경우 -> 낫싱

DevRunner21 · 2022-01-13T07:13:25Z

src/main/java/GameInput.java

+
+public abstract class GameInput {
+
+    public static final String REQUEST_RE_INPUT_MESSAGE = "다시 입력해주세요.";
+
+    public InputNumbers getUserInputNumbers() {
+
+        String inputStr = inputStr = input();
+        List<Integer> inputs = convertToIntegerList(inputStr);
+
+        InputNumbers inputNumbers = null;
+        try {
+            inputNumbers = new InputNumbers(inputs);
+
+        } catch (IllegalArgumentException e) {
+            System.out.println(REQUEST_RE_INPUT_MESSAGE);
+            getUserInputNumbers();
+        }
+
+        return inputNumbers;
+    }
+
+    public ConfirmCommand inputConfirmCommand() {
+        ConfirmCommand confirmCommand = null;
+
+        try {
+            confirmCommand = ConfirmCommand.findConfirmCommand(input());
+        } catch (IllegalArgumentException e) {
+            System.out.println(REQUEST_RE_INPUT_MESSAGE);
+            inputConfirmCommand();
+        }
+
+        return confirmCommand;
+    }
+
+    private List<Integer> convertToIntegerList(String inputStr) {
+        return Arrays.stream(inputStr.split(""))
+            .map(Integer::parseInt)
+            .collect(Collectors.toList());
+    }
+
+    protected abstract String input();
+
+}


입력 UI를 구현할 때, TemplateMethodPattern을 적용해봤습니다.

DevRunner21 · 2022-01-13T07:15:53Z

src/main/java/Game.java

+            ConfirmCommand confirmCommand = input.inputConfirmCommand();
+            if (confirmCommand.isExit()) {
+                return;
+            }
+
+            start(); // 재귀사용
+            return;
+        }


재원님이 말씀해주셨던 피드백중에 "재귀식"에 대한 내용이 기억나서
이번에 반복되는 처리에 대해서 재귀식으로 구성해봤습니다.

DevRunner21 · 2022-01-13T07:16:43Z

src/main/java/RandomNumberGenerator.java

+public class RandomNumberGenerator {
+
+    private final int MAX = 9;
+
+    private final int MIN = 1;
+
+    private final Random random;
+
+    public RandomNumberGenerator() {
+        this.random = new Random();
+    }
+
+    public int generate() {
+        return random.nextInt((MAX - MIN) + 1) + MIN;
+    }
+
+}


RandomNumber에 대한 생성의 책임은 RandomNumberGenerator라는 객체에게 부여했습니다.

darkant99 · 2022-01-13T10:41:36Z

src/main/java/Ball.java

+        this.number = number;
+    }
+
+    public int getPosition() {


getter가 특별한 이유 없이 public으로 노출되고 있어요 🤔

darkant99 · 2022-01-13T10:45:38Z

src/main/java/Balls.java

+    private final List<Ball> balls;
+
+    public Balls(InputNumbers inputNumbers) {
+        balls = new ArrayList<>();


생성자에서는 단순한 값의 초기화만 이루어져야 좋다고 생각해요.
생성자에서 단순히 List을 받고, static 팩토리 메서드로 다양한 생성 방법을 노출하면 어떨까요? 🤔

darkant99 · 2022-01-13T10:46:59Z

src/main/java/InputNumbers.java

+        numbers.forEach(this::addInputNumber);
+    }
+
+    // TODO : getter가 있어도 될까?


getter로 노출 하는건 굉장히 좋지 못하다고 생각해요!

darkant99 · 2022-01-13T10:49:57Z

src/main/java/Score.java

+        this.ballCount = ballCount;
+    }
+
+    public int getStrikeCount() {


strikeCount를 알려주지 않고 Score에게 strikeCount값을 물어보면 어떨까요?

darkant99 · 2022-01-13T10:50:45Z

src/main/java/Score.java

+    }
+
+    public String makeResultMessage() {
+        StringBuilder stringBuilder = new StringBuilder();


ResultMessage를 만들어 주는건 도메인의 역할이 아니라고 생각해요

darkant99 · 2022-01-13T10:52:01Z

src/main/java/RandomNumberGenerator.java

@@ -0,0 +1,19 @@
+import java.util.Random;
+
+public class RandomNumberGenerator {


추상화 할 수 있을것 같아요🤔

darkant99 · 2022-01-13T10:57:02Z

고생 하셨습니다.
충분히 깔끔한 코드 이지만 아쉬운 점은 객체들간의 역할이 모호한것 같아요😭
또한 정말 특별한 이유가 아니라면 getter의 노출은 지양 되었으면 해요.
미션을 진행 하시면서 getter를 1개도 사용하지 않는다는 극단적인 방법을 사용 해보셔도 인사이트를 얻으실것 같아요 😃

DevRunner21 added 5 commits January 13, 2022 12:38

[doce] ReadMe에 구현사항을 정리하라

373ff0e

[feat] 랜덤 숫자 입력 로직을 구현하라

4a92ccb

- 1~ 9까지의 숫자 체크 - 서로 다른 숫자 체크

- 입력받은 숫자에 대한 비교 로직을 구현하라

a8fabde

- 위치, 숫자 모두 맞춘경우 -> 스트라이크 - 위치는 틀리고 숫자만 맞춘 경우 -> 볼 - 위치, 숫자 모두 틀린 경우 -> 낫싱

[feat] 입출력 및 UI 로직을 구현하라

8cc3954

[feat] 게임 실행 로직을 구현하라

deac9e0

DevRunner21 added the 리뷰요청 label Jan 13, 2022

DevRunner21 commented Jan 13, 2022

View reviewed changes

darkant99 reviewed Jan 13, 2022

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

숫자 야구게임 구현#2

숫자 야구게임 구현#2
DevRunner21 wants to merge 5 commits intomainfrom
mission02-feedback-mine

DevRunner21 commented Jan 13, 2022 •

edited

Loading

Uh oh!

DevRunner21 Jan 13, 2022

Uh oh!

DevRunner21 Jan 13, 2022

Uh oh!

DevRunner21 Jan 13, 2022

Uh oh!

darkant99 Jan 13, 2022

Uh oh!

darkant99 Jan 13, 2022

Uh oh!

darkant99 Jan 13, 2022

Uh oh!

darkant99 Jan 13, 2022

Uh oh!

darkant99 Jan 13, 2022

Uh oh!

darkant99 Jan 13, 2022

Uh oh!

darkant99 commented Jan 13, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		@@ -0,0 +1,19 @@
		import java.util.Random;

		public class RandomNumberGenerator {

Conversation

DevRunner21 commented Jan 13, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

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!

darkant99 commented Jan 13, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

DevRunner21 commented Jan 13, 2022 •

edited

Loading