[🚀 사이클2 - 미션 (기물 확장 + DB 적용)] 에버 미션 제출합니다. by Chocoding1 · Pull Request #344 · woowacourse/java-janggi

Chocoding1 · 2026-04-09T12:49:42Z

체크 리스트

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

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

안녕하세요 백호~
사이클2도 잘 부탁드립니다!

객체의 필드로 id를 둔 것

처음에 dao를 도입하면서 저장 객체로 DTO를 만들었습니다.
DTO는 단순히 저장용 객체이기 때문에 비즈니스 로직을 둘 이유가 없었습니다.
그러나 비즈니스 로직이 있는 진짜 객체들이 프로그램을 구성하는데, 그 과정에서 DTO의 필드로 있는 id(key)가 필요했습니다.
그래서 어쩔 수 없이 진짜 객체들에게도 id를 필드로 주었는데, 그러다 보니 db에 저장하기 전에 객체를 생성하는 경우 id를 null로 설정하여 생성해야 하는 경우가 생겼습니다.
그래서

id 없는 객체(Turn) 생성 및 반환
-> DAO를 의존하는 상위 클래스에서 해당 객체를 가지고 DB에 저장 후 id 획득
-> 획득한 id를 가지고 새로운 객체를 다시 생성 및 반환

하는 순서로 코드를 짰습니다.
효율적인 코드라고는 생각이 들지 않아서 이 부분에 대해서 피드백해주시면 감사하겠습니다:)

…o step2

Arachneee

안녕하세요 에버!

사이클2 잘 작성해주셨네요! 고생하셨습니다!

코멘트 남겼어요 확인해주세요!

객체의 필드로 id를 둔 것

객체의 필드로 id를 두는 것은 객체를 식별하기 위해 필요하다고 생각합니다.
그렇다면 id는 해당 객체의 식별자를 의미하는 것이고 동등성도 id로 보장해야한다고 생각합니다.

id 없는 객체(Turn) 생성 및 반환 -> DAO를 의존하는 상위 클래스에서 해당 객체를 가지고 DB에 저장 후 id 획득 -> 획득한 id를 가지고 새로운 객체를 다시 생성 및 반환

id 생성을 DB에 의존하기 때문에 나오는 것이라고 생각해요. DB에 의존하지 않고 id를 만드는 방법은 어떤 것이 있을까요?

Arachneee · 2026-04-09T15:57:20Z

src/main/java/janggi/domain/game/GameDao.java

+
+import static janggi.domain.game.GameStatus.IN_PROGRESS;
+
+public class GameDao {


DAO가 뭔가요?

DAO는 Data Access Object로 데이터에 접근하는 객체입니다.
테이블과 직접 소통하며 SQL 쿼리를 실행하고 JDBC를 직접 다룬다는 점, 객체를 직접 다루지 않고 특정 필드만 포함하는 DTO를 다룬다는 점에서 DAO라는 역할을 주었습니다.

Arachneee · 2026-04-09T15:59:10Z

src/main/java/janggi/domain/game/GameResult.java

+public enum GameResult {
+
+    RUNNING,
+    WIN,
+    LOSE,
+}


RUNNING도 게임 결과인가요? 게임의 상태와 게임의 결과를 구분하면 좋을 것 같아요. GameStatus라는게 이미 있네요. 그럼 RUNNING은 꼭 필요한 걸까요?

처음에 Team 객체를 생성할 때 GameResult도 초기화를 해줘야 하는데, WIN이나 LOSE가 아닌 '아직 승패가 정해지지 않음'을 표현하고자 RUNNING이라고 이름을 지었습니다. 다만 RUNNING이라는 이름이 이미 GameStatus에서 사용되고 있고, 이름이 중복될 경우 헷갈릴 수도 있으니 아직 결과가 없음을 나타내는 NONE으로 수정했습니다.

혹시 제가 잘못 접근한 것이면 추가적인 조언 부탁드립니다!

Arachneee · 2026-04-09T16:02:07Z

src/main/java/janggi/domain/game/GameService.java

+
+    public Long save() {
+        return gameDao.save(GameDto.forSave());
+    }


아무것도 없이 save를 하는 건가요?? 아니면 처음 게임을 세팅하는 것일까요? init이나 start같은 메소드명이 더 어울리는 것 같아요.

GameDto.forSave()도 init이나 start가 더 어울리는 것 같아요.

초기 JanggiGame 객체를 생성하면서 저장하는 GameDto이기 때문에 init이라는 메서드명이 더 적합한 것 같아 수정했습니다!

Arachneee · 2026-04-09T16:03:01Z

src/main/java/janggi/domain/game/GameService.java

+
+import java.util.List;
+
+public class GameService {


JanggiGameService와 GameService는 어떤 다른 역할을 가지고 있나요?

JanggiGameService는 JanggiGame 객체를 생성합니다. 그리고 JanggiGame 객체와 그 하위에 있는 Turn, Piece 객체들을 영속화하는 로직들을 수행합니다.
때문에 JanggiGameService에서 모든 Service를 의존하도록 설계했습니다. 즉 모든 객체의 DB를 의존하는 TotalService 역할이라고 보시면 됩니다.

TurnService의 경우 단순히 DAO를 호출하는 것이 아닌 비즈니스 로직이 포함되어 있어 필요한 객체라고 생각이 들었습니다.
그러나 GameService, PieceService는 단순히 파라미터를 받아 DAO 객체에 요청하는 역할이기 때문에 GameService와 PieceService는 굳이 Service 계층을 만들 필요는 없다고 생각합니다.

그러나 JanggiGameService에서 Turn만 Service를 의존하고 Game과 Piece는 Dao를 의존한다면 계층이 통일되지 않고 혼잡해보여 Game과 Piece도 Service 계층을 만들어 통일시켰습니다.

혹시 추가적인 조언이 있다면 말씀해주시면 감사하겠습니다!

Arachneee · 2026-04-09T16:05:33Z

src/main/java/janggi/domain/piece/Jol.java

+            if (dxDelta.getDy() < 0) {
+                throw new IllegalArgumentException("이동할 수 없는 위치입니다.");
+            }
+            return;
+        }
+
+        if (dxDelta.getDy() > 0) {
+            throw new IllegalArgumentException("이동할 수 없는 위치입니다.");
+        }


Delta에게 위 방향인지 아래 방향인지 물어봐도 좋을 것 같아요.

Arachneee · 2026-04-09T16:10:38Z

src/main/java/janggi/domain/piece/PieceFactory.java

+    public static Piece create(PieceType pieceType, TeamType teamType) {
+        return pieceType.create(teamType);
+    }
+}


너무 간단한 로직같은데 Factory 객체가 꼭 필요한가요? 저는 Piece의 정적 팩토리 메소드로 구현해도 충분해보이는데 어떻게 생각하세요?

저도 팩토리 클래스를 생성하면서 찝찝한 감이 있었는데, 정적 팩토리 메서드를 생각하지 못 했네요..
Piece 생성 책임을 Piece에 두는 것이 더 객체지향적이라는 생각이 듭니다!

Arachneee · 2026-04-09T16:13:49Z

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

 public abstract class Team {

    private final Map<Position, Piece> pieces;
+    private final GameResult gameResult;


GameResult를 Team이 가지고 관리해야할까요? 게임 결과는 두 팀이 게임을 해야만 나오는 값이라고 생각하는데 팀이 온전히 자신의 결과를 가지고 있을 수 있을까요?

두 팀 사이의 게임 결과에 대한 정합성은 누가 관리해야할까요?

Arachneee · 2026-04-09T16:17:12Z

src/main/java/janggi/domain/Delta.java

+    public static Delta scaleDown(Delta delta) {
+        return new Delta(Integer.signum(delta.dx), Integer.signum(delta.dy));
+    }


이건 제가 저번에 놓쳤군요. static 메소드가 아니라 non-static 메소드로 만들면 좋을 것 같아요.

Delta unitDelta = delta.scaleDown()

Arachneee · 2026-04-09T16:18:57Z

src/main/java/janggi/domain/MovePath.java

+    public boolean matches(Delta delta) {
+        return totalDelta().equals(delta);
    }

-    public boolean matchesDirection(int dx, int dy) {
+    public boolean matchesDirection(Delta dxDelta) {


Arachneee · 2026-04-09T16:37:33Z

src/main/java/janggi/dto/TurnDto.java

+import janggi.domain.turn.Turn;
+import janggi.domain.team.TeamType;
+
+public record TurnDto(Long id, Long gameId, TeamType currentTurnTeam) {


저장을 위한 객체와 도메인 객체를 분리한 이유가 궁금합니다.

Chocoding1 added 20 commits April 7, 2026 12:53

refactor: Delta 필드 노출 최소화

3b2f1f9

chore: 메서드 순서 변경

7288f34

feat: 궁성 기능 구현

efffe3e

refactor: 기물 점수 추가

626f0ff

feat: 왕 잡힐 시 게임 종료 기능 구현

f74f4ac

feat: 점수 계산 로직 구현

0889418

feat: 점수 계산 로직 구현

00bbd43

refactor: 게임 종료 문구 출력 기능 추가

5d6e01f

docs: 프로그램 흐름 수정

fdb23c0

feat: DB 연동 및 DAO, DTO 생성

0bbcaea

feat: 장기 게임을 이어서 진행할 경우 장기 게임 불러오는 기능 구현

0ab9bfd

refactor: GameDto 팩토리 메서드 추가

cdf56c1

refactor: side -> team 폴더명 수정

f42297c

refactor: 기물 이동 메서드명 수정

9919545

chore: 기물 이동 메서드명 수정

b7c0458

feat: 턴 및 기물 정보 저장 기능 구현

f020ff9

refactor: 폴더 구조 변경

df3ee44

docs: ERD 추가

5a3fa09

refactor: SQL 파일 수정

e53316d

Merge branch 'step2' of https://github.com/Chocoding1/java-janggi int…

3a9ac59

…o step2

Arachneee requested changes Apr 9, 2026

View reviewed changes

Chocoding1 added 5 commits April 10, 2026 20:00

refactor: GameDto 팩토리 메서드명 수정

974e972

refactor: 위, 아래 움직임 여부 계산 로직 Delta로 캡슐화

9068764

refactor: 기물 저장 중 오류 발생 시 rollback 기능 추가

d9bb0b4

refactor: PieceType에서 Piece 생성 메서드를 함수형 인터페이스로 변경

5f093e9

refactor: PieceFactory 삭제 & Piece에 정적 팩토리 메서드 생성

795efef


		import static janggi.domain.game.GameStatus.IN_PROGRESS;

		public class GameDao {

Conversation

Chocoding1 commented Apr 9, 2026

체크 리스트

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

객체의 필드로 id를 둔 것

Uh oh!

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

Chocoding1 Apr 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

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

Chocoding1 Apr 10, 2026 •

edited

Loading