[🚀 사이클1 - 미션 (보드 초기화 + 기물 이동)] 피노 미션 제출합니다. by haechanmoon · Pull Request #266 · woowacourse/java-janggi

haechanmoon · 2026-03-30T07:52:12Z

체크 리스트

미션의 필수 요구사항을 모두 구현했나요?
Gradle test를 실행했을 때, 모든 테스트가 정상적으로 통과했나요?
애플리케이션이 정상적으로 실행되나요?

안녕하세요 피노입니다!
페어와 머리를 맞대며 열심히 구현해 보았지만 오랜 시간이 걸렸습니다.
Depth1 이상과, get사용, else사용, 등 못지킨 요구사항들이 많습니다..
그래도 리뷰 잘 부탁드리겠습니다!

어떤 부분에 집중하여 리뷰해야 할까요?

초반 설계의 너무 어려움

제가 객체지향에 아직 익숙하지 않은 것 같습니다. 초반 설계할 때 숨이 탁 막힙니다.
저번주에 <객체지향의 사실과 오해>를 읽긴 했지만 이것을 직접 코드로 옮기려 하니 아직 너무 어려운 과정인 것 같습니다. 페어와 같이 생각하지 않았더라면 혼자서는 돌아가는 기능을 완성하지 못했을 것 입니다.

TDD를 하더라도 큰 뼈대는 만들고 시작을 해야한다고 생각합니다.
초반에 많은 어려움을 느끼는데 이것을 어떻게 극복하셨는지 궁금합니다.!

다형성

추상클래스나 인터페이스를 사용하여 조건문을 제거하고 싶었지만 TDD 로 할 때의 복잡성이 너무 커서 시도하다가 포기하고 구현하기로 했습니다. 어디서 어떻게 추상화하면 좋을지 잘 모르겠습니다.

감사합니다!!

(상차림 추후 추가 예정)

syoun602

안녕하세요 피노 🙂
이번에 장기 미션 리뷰어를 맡게된 썬입니다. 만나서 반가워요.
리뷰를 진행하며 설계적인 부분도 보고 있었으나 그 전에 우선적으로 개선해야 할 요구사항 관련 내용이 많이 보여 이번에는 설계 중심보다는 프로그래밍 요구사항과 기본 동작을 먼저 맞추는 방향으로 리뷰를 남겼습니다.

특히 getter 지양, 원시값 포장, 일급 컬렉션 사용, else 사용 금지, 메서드 길이 제한 같은 요구사항이 제대로 지켜지지 않고 있어요. 설계를 더 다듬기 전에 이 기준들을 최대한 지키는 방향으로 먼저 정리해보길 권장드릴게요. 해당 부분들이 정리되어야 이후에 객체의 책임을 나누거나 협력 구조를 개선하는 작업도 훨씬 수월해질 것 같아요.

또한 현재 PR 브랜치 기준으로는 모든 테스트가 통과하지 않고 있어요. 빠르게 리뷰를 받아보는 것도 중요하지만 그보다 먼저 테스트가 안정적으로 통과하는 상태인지, 주어진 요구사항을 최대한 지키려고 노력했는지가 더 중요하다고 생각합니다.

syoun602 · 2026-03-31T00:22:05Z

src/main/java/janggi/controller/JanggiController.java

+import java.util.List;
+import java.util.function.Supplier;
+
+public class JanggiController {


Controller란 무엇인지 어떤 역할인지 알려주세요.
단순히 Spring에서 흔히 사용되는 패턴이라서 썼다면 콘솔 어플리케이션에 꼭 컨트롤러라는 네이밍이 필요한지 고민해봅시다.

제가 생각했던 Controller의 역할은 도메인과 View를 연결하고 전체적인 애플리케이션의 흐름을 제어하고 관리하는 것이었습니다. 저는 아직 Spring을 거의 다뤄본 적이 없어서 어떠한 패턴을 의식하고 썼다기보다는, MVC 패턴 관점에서 흐름을 담당하는 객체라고 생각하여 JanggiController라는 네이밍을 사용했습니다.

하지만 코멘트를 남겨주신 후 콘솔 환경과 현재 객체의 역할을 다시 고민해 보았습니다. 단순히 입출력을 제어하는 것을 넘어 장기 게임의 전체적인 진행을 책임지는 점에서, Controller라는 이름보다는 JanggiGame이라는 네이밍이 이 Class의 책임과 성격을 더 직관적으로 보여준다고 생각합니다!

네이밍과 객체의 역할에 대해 깊게 고민해 볼 수 있었습니다. JanggiGame으로 수정하여 반영하겠습니다!

이전에 남겼던 코멘트가 있는데 참고차 한번 읽어보시라고 링크 남겨둘게요~
woowacourse/java-blackjack#975 (comment)

src/main/java/janggi/controller/JanggiController.java

src/main/java/janggi/domain/Board.java

syoun602 · 2026-03-31T00:24:37Z

src/main/java/janggi/domain/board/Board.java

+        return board;
+    }
+
+    public List<Position> findAvailablePositions(Position position) {


전체적으로 라인 수가 너무 길어요.
미션 요구사항을 지켜봅시다

함수(또는 메서드)의 길이가 10라인을 넘어가지 않도록 구현한다. 함수(또는 메소드)가 한 가지 일만 하도록 최대한 작게 만들어라.

최대한 작게 쪼개도록 노력해 보았습니다!
메서드들을 작게 쪼개다가 보니 제가 너무 복잡한 설계가 된 것이 아닌가 싶습니다. 단위테스트부터 다시 해야하나 고민중입니다.. 메서드 라인 수 줄이는 작업이 제일 오래 걸렸던 것 같습니다.

src/main/java/janggi/domain/Board.java

src/main/java/janggi/domain/PieceType.java

src/main/java/janggi/domain/Piece.java

syoun602 · 2026-03-31T00:28:51Z

src/main/java/janggi/domain/Board.java

+        Piece piece = board.get(position);
+        PieceType pieceType = piece.getPieceType();
+
+        MoveRule moveRule = pieceType.getMoveRule();
+
+        List<Route> routes = moveRule.findRoutes(piece.getTeam());


전부 데이터를 꺼내와서 조립하고 있는데 tell don't ask에 대해 공부해봅시다

이 부분에 대해서 많은 고민을 하게 됐습니다. 어디까지 get을 쓸 수 있을까에 대한 고민, 어떻게 하면 덜 쓸 수 있을까. . 아직 생각이 완전히 정리되지 않아 조금 더 생각을 해봐야 할 것 같습니다..

src/test/java/janggi/domain/PieceTest.java

haechanmoon

안녕하세요 썬 리뷰어님
리팩토링 완료한 한 후 [사이클1 2차 PR] 제출하겠습니다!
제가 처음에 이러한 코드를 짤 수 밖에 없던 이유를 적지 않았어서 조금 혼란스러우셨을 수도 있다고 생각했습니다. 저는 아직 기본기가 많이 부족했고 구현에 어려움을 겪었습니다. 너무 많은 고민은 시간을 지체하게 했습니다. 따라서 일단 돌아가는 코드를 작성하는 것이 제 목표였습니다. 또한 급히 내느라 여러 실수가 있었던 것 같습니다.
리뷰 해주시는 부분 앞으로 열심히 리펙토링 해보도록 하겠습니다!

haechanmoon · 2026-03-31T08:20:30Z

src/main/java/janggi/controller/JanggiController.java

+import java.util.List;
+import java.util.function.Supplier;
+
+public class JanggiController {


제가 생각했던 Controller의 역할은 도메인과 View를 연결하고 전체적인 애플리케이션의 흐름을 제어하고 관리하는 것이었습니다. 저는 아직 Spring을 거의 다뤄본 적이 없어서 어떠한 패턴을 의식하고 썼다기보다는, MVC 패턴 관점에서 흐름을 담당하는 객체라고 생각하여 JanggiController라는 네이밍을 사용했습니다.

하지만 코멘트를 남겨주신 후 콘솔 환경과 현재 객체의 역할을 다시 고민해 보았습니다. 단순히 입출력을 제어하는 것을 넘어 장기 게임의 전체적인 진행을 책임지는 점에서, Controller라는 이름보다는 JanggiGame이라는 네이밍이 이 Class의 책임과 성격을 더 직관적으로 보여준다고 생각합니다!

네이밍과 객체의 역할에 대해 깊게 고민해 볼 수 있었습니다. JanggiGame으로 수정하여 반영하겠습니다!

src/main/java/janggi/domain/Board.java

src/main/java/janggi/controller/JanggiController.java

haechanmoon · 2026-04-02T06:41:10Z

src/main/java/janggi/domain/board/Board.java

+        return board;
+    }
+
+    public List<Position> findAvailablePositions(Position position) {


최대한 작게 쪼개도록 노력해 보았습니다!
메서드들을 작게 쪼개다가 보니 제가 너무 복잡한 설계가 된 것이 아닌가 싶습니다. 단위테스트부터 다시 해야하나 고민중입니다.. 메서드 라인 수 줄이는 작업이 제일 오래 걸렸던 것 같습니다.

src/main/java/janggi/domain/Board.java

src/main/java/janggi/domain/PieceType.java

src/test/java/janggi/domain/PieceTest.java

syoun602

안녕하세요 피노~
요구사항을 적절히 잘 지켜주셨네요 👍
장기 미션이 워낙 어려운 미션이기 때문에,지금처럼 하나씩 정리해나가는 과정 자체가 중요하다고 생각해요. 이번에는 이전보다 훨씬 안정적으로 읽혔고 요구사항을 맞추려는 노력도 보였습니다 😀
몇 가지 개선 포인트를 남겨두었는데 확인해주시고 재요청 주세요.

syoun602 · 2026-04-03T12:30:39Z

src/main/java/janggi/controller/JanggiGame.java

+        }
+    }
+
+    private boolean playTurn(Board board) {


턴에 대한 메시지 출력은 있는데, 실제 턴 개념이 도메인에 없는 것 같아요. 한나라 차례에도 초나라 말을 움직일 수 있네요

syoun602 · 2026-04-03T12:33:35Z

src/main/java/janggi/controller/JanggiGame.java

+import java.util.Map;
+import java.util.function.Supplier;
+
+public class JanggiGame {


이 부분은 후순위로 두고 작업해주셔도 됩니다)

게임 자체도 하나의 도메인 객체로 바라보면 좋을 것 같아요.
지금 구조는 게임 진행과 입출력 강결합 되어 있는데 Application이 입력과 출력을 담당하고 Game은 게임 진행에 필요한 상태와 규칙만 관리하도록 나누면 책임이 더 선명해질 것 같아요. 이런 식으로 분리하면 이후에 턴 관리나 종료 조건 같은 규칙도 UI와 독립적으로 다루기 쉬워질 것 같아요.

syoun602 · 2026-04-03T12:34:52Z

src/main/java/janggi/controller/JanggiGame.java

+    private boolean isChoTurn = true;
+
+    public void run() {
+        Board board = initializeBoard();


JanggiGame이 게임 진행을 담당한다면,그 안에서 계속 사용하는 Board는 인스턴스 필드로 가지고 있으면 어떨까요? 현재는 여러 메서드에서 계속 Board를 인자로 전달하고 있는데, 게임이 직접 관리하는 상태라면 필드로 두는 쪽이 객체의 역할을 더 자연스러워 보여요.

syoun602 · 2026-04-03T12:48:14Z

src/main/java/janggi/domain/MoveRule.java

+
+import java.util.List;
+
+public interface MoveRule {


지금의 MoveRule은 각 기물의 이동 규칙을 직접 판단하기보다는 이동 방향 목록만 반환하고 실제 경로 계산과 장애물 검사 이동 가능 여부 판단은 대부분 Board가 담당하고 있어요. 그래서 Board에서 isPo() 같은 타입 분기가 나오고 이동 규칙이 MoveRule이나 Piece에 충분히 머물지 못하고 Board 쪽에 많이 모여 있는 듯합니다.

이런 구조라면 MoveRule이 지금 정말 의미 있는 역할을 하고 있는지 한 번 고민해보면 좋을 것 같아요. 만약 이동 규칙을 표현하기 위한 객체라면, 단순히 방향만 넘겨주기보다 현재 보드 상태에서 어디까지 이동 가능한지까지 스스로 판단하는 쪽이 더 자연스러워 보입니다. Piece가 자신의 이동 가능 위치를 직접 판단하도록 만들어보는 것도 방법일 것 같은데요. 조합을 사용한다면 Piece가 MoveRule을 가지고 있다가 이동 가능 위치 계산을 위임할 수 있고, 그렇게 하면 Board는 판의 상태를 제공하는 역할에 더 집중할 수 있을 것 같습니다.

syoun602 · 2026-04-03T12:49:44Z

src/main/java/janggi/domain/Position.java

+    private final int x;
+    private final int y;


x,y 가 좌표를 의미하는건 알지만 더 의미 있는 이름은 없을까요?
단순 수학 좌표보다 row col과 같은 이름도 있겠네요

syoun602 · 2026-04-03T12:50:03Z

src/main/java/janggi/domain/Team.java

@@ -0,0 +1,7 @@
+package janggi.domain;
+
+public enum Team {


팀 관련해서 메서드들을 응집해봅시다. 같은 팀 판단 등

syoun602 · 2026-04-03T12:50:54Z

src/main/java/janggi/view/OutputView.java

+        System.out.println();
+    }
+
+    private void printCell(Map<Position, Piece> board, List<Position> availablePositions, Position position) {


우선 요구사항이니만큼 view여도 else if/switch는 최대한 지양해주세요

syoun602 · 2026-04-03T12:52:12Z

src/main/java/janggi/domain/Direction.java

@@ -0,0 +1,29 @@
+package janggi.domain;
+
+public enum Direction {


up, down 등 보다 더 좋은 예시가 있어요. 나침반을 활용해봅시다~

syoun602 · 2026-04-03T12:54:58Z

src/main/java/janggi/domain/board/Board.java

+        this.board = initBoard;
+    }
+
+    public Map<Position, Piece> getBoard() {


메서드 순서를 신경써보아요. 단순 getter의 위치를 가장 잘보이는 최상단에 놓아야 할까요?
또한 방어적 복사에 대해 공부해봅시다~

syoun602 · 2026-04-03T12:59:35Z

src/main/java/janggi/controller/JanggiController.java

+import java.util.List;
+import java.util.function.Supplier;
+
+public class JanggiController {


이전에 남겼던 코멘트가 있는데 참고차 한번 읽어보시라고 링크 남겨둘게요~
woowacourse/java-blackjack#975 (comment)

haechanmoon added 18 commits March 25, 2026 16:15

feat(Position): Position 도메인 구현

d4bacf5

docs(readme): v1.1 팀 추가 및 기물이 규칙을 갖고있게함

cc4a35d

feat(Team): 초나라와 한나라 enum으로 관리

4d014e3

docs(readme): v1.2 구현의 어려움에 따른 도메인 변경 -> 돌아가는 코드를 우선 만들어보기

782e1dc

feat: 해당 기물은 팀과 기물 타입을 가진다. TDD 구현

68a19d2

feat: 상차림은 마상마상으로 고정하여 보드 초기화

0dac849

(상차림 추후 추가 예정)

feat: 각 기물은 고유의 규칙을 가진다.

7aade2e

refactor: MoveRule 인터페이스 생성

8e805d5

feat: 졸의 이동가능 여부를 판단

ed501e2

test: 마의 이동가능 여부를 판단

e55e727

fix(board): 생성자에서 보드초기화 제거

27ffdfc

test: 상,사,왕의 이동가능 여부를 판단

361391c

feat: 차의 이동가능 여부를 판단

a49a2c4

feat: 포의 이동가능 여부를 판단

adac323

test: DisplayName과 메서드명 변경

08f4aad

feat: 뷰와 컨트롤러 기능 구현

f7f0f3e

feat: 재입력 로직 구현

61a6903

refactor: 코드 스타일 개선

3ed382f

syoun602 requested changes Mar 31, 2026

View reviewed changes

haechanmoon added 11 commits March 31, 2026 17:07

test(BoardTest): 포 이동 불가 시 발생하는 예외 검증 테스트로 수정

6f2249e

style(BoardTest): 코드 포매팅

2e77c24

refactor: 책임과 성격에 맞게 JanggiController -> JanggiGame 으로 이름 변경경

e0ab761

refactor(JanggiGame): 메서드 10줄 이하 되도록 수정

a03d4ee

refactor: 보드 초기화 로직 BoardFactory로 이동

740cc33

refactor(Board): for문을 stream 으로 변경

52041ea

refactor(PieceType): 미사용 필드 제거

de84fcc

refactor(ZolMoveRule): 불필요 행 제거

d0a52be

style: 코드 스타일 변경

50a774f

refactor(Board): 메서드가 한가지 일을 할 수 있도록 분리

d0557f7

refactor(OutputView): 중복되는 코드 제거

7635d5f

refactor: 도메인에서 View로직 분리

1a9affe

haechanmoon commented Apr 2, 2026

View reviewed changes

syoun602 requested changes Apr 3, 2026

View reviewed changes

		@@ -0,0 +1,29 @@
		package janggi.domain;

		public enum Direction {

Conversation

haechanmoon commented Mar 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

체크 리스트

어떤 부분에 집중하여 리뷰해야 할까요?

초반 설계의 너무 어려움

다형성

Uh oh!

syoun602 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!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

haechanmoon left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

syoun602 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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

haechanmoon commented Mar 30, 2026 •

edited

Loading