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

simhokyung · 2026-04-10T03:32:03Z

체크 리스트

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

H2 DB 실행 방법

1. H2 콘솔 접속

브라우저에서 아래 주소로 접속합니다.
http://localhost:8082

2. DB 연결 정보 입력

JDBC URL: jdbc:h2:~/janggi_test;AUTO_SERVER=TRUE
User: sa
Password: (빈 값)

3. 데이터 확인

콘솔 접속 후 아래 쿼리를 실행하여 데이터 저장 여부를 확인할 수 있습니다.

SELECT * FROM game;
SELECT * FROM piece;

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

안녕하세요 아서! 이번에 콘솔에 DB를 연결하는것과 JDBC에 익숙하지 않아서 학습하면서 진행하느라 제출이 많이 늦어졌네요..ㅠ
이번 리뷰도 잘 부탁드립니다!

DB 연결 구조 방식이 적절한지

이번 단계에서는 콘솔 애플리케이션에 DB를 연결하고 JDBC로 저장/복원 기능을 구현하는 과정이 처음이어서, AI의 도움을 많이 받으면서 구현을 하였습니다. 그 과정에서 구조적으로 부족한 부분이 있었는지 피드백 받고 싶습니다.

현재 구조는 아래와 같이 구성되어 있습니다.

config는 DB 연결과 테이블 초기화 책임을 가집니다.
repository는 저장/조회의 책임을 가집니다.
SavedGame, SavedPiece는 DB에 저장하기 위한 저장용 객체로 두었습니다.
GameSnapshot, GameRestorer는 도메인 객체와 저장용 객체를 서로 변환하는 역할로 분리했습니다.
controller는 새 게임 시작, 저장된 게임 복원, 게임 진행, 자동저장 같은 전체 흐름을 담당하도록 하였습니다.

도메인이 DB 구조를 직접 알지 않게 하려고 저장용 모델과 변환 객체를 분리했습니다.
위와 같은 구조가 적절했는지, 혹은 부족하거나 잘못된 부분이 있었는지 궁금합니다.

그 외에 새로 추가된 궁성규칙과 관련된 비즈니스 로직에서도 부족한 부분이 있으면 피드백 부탁드립니다!

- 궁성에서 화살표 방향에 맞는 경우에만 대각선 이동 가능

- 게임의 진행/종료 상태를 관리하는 GameStatus 구현

- PieceType에 기물별 점수 추가 - Board에서 점수계산 로직 구현

- SavedGame의 턴, 종료여부, 승자를 game 테이블에 저장

Hyunta

안녕하세요 맥스,
장기 2단계 미션 구현하시느라 고생하셨습니다.
관련해서 몇가지 커멘트 남겨드렸는데 확인 부탁드립니다.

Hyunta · 2026-04-10T21:05:32Z

src/main/java/repository/JdbcGameRepository.java

+    @Override
+    public void save(SavedGame savedGame) {
+        Connection connection = null;
+        try {
+            connection = DatabaseConfig.getConnection();
+            connection.setAutoCommit(false);
+
+            deleteAll(connection);
+            long gameId = insertGame(connection, savedGame);
+            insertPieces(connection, gameId, savedGame);
+
+            connection.commit();
+        } catch (SQLException exception) {
+            rollback(connection);
+            throw new IllegalStateException("[ERROR] 게임 저장에 실패했습니다.", exception);
+        } finally {
+            close(connection);
+        }
+    }


try-with-resources를 학습하고 한번 적용해보세요.

Hyunta · 2026-04-10T21:07:54Z

src/main/java/repository/JdbcGameRepository.java

+    @Override
+    public void clear() {
+        try (Connection connection = DatabaseConfig.getConnection()) {
+            deleteAll(connection);
+        } catch (SQLException exception) {
+            throw new IllegalStateException("[ERROR] 저장 데이터 삭제에 실패했습니다.", exception);
+        }
+    }
+
+    private void deleteAll(Connection connection) throws SQLException {
+        try (PreparedStatement pieceStatement = connection.prepareStatement(DELETE_PIECE);
+             PreparedStatement gameStatement = connection.prepareStatement(DELETE_GAME);
+        ) {
+            pieceStatement.executeUpdate();
+            gameStatement.executeUpdate();
+        }
+    }


나중에 장기를 여러명이서 사용하면 deleteAll이 치명적일 수도 있겠네요.
Game에 Id 값을 추가해서, 원하는 값만 삭제하도록 구현해보면 어떨까요?

Hyunta · 2026-04-10T21:10:12Z

src/main/java/model/move/GeneralMoveRule.java

+    private boolean isInsidePalace(Move move, Country country) {
+        Palace palace = Palace.from(country);
+        return palace.contains(move.from()) && palace.contains(move.to());
+    }
+
+    private boolean isMovable(Move move, Country country){
+        if(isStraightOneStep(move)){
+            return true;
+        }
+        Palace palace = Palace.from(country);
+        return palace.isDiagonalMove(move);
+    }
+
+    private boolean isStraightOneStep(Move move){
+        if(!move.isStraight()){
+            return false;
+        }
+        return move.distance() == 1;
+    }


GeneralMoveRule과 GuardMoveRule 메서드가 완전히 동일하네요.
중복을 제거하려면 어떻게 접근할 수 있을까요?

Hyunta · 2026-04-10T21:19:00Z

src/main/java/repository/JdbcGameRepository.java

+        try (Connection connection = DatabaseConfig.getConnection();
+             PreparedStatement gameStatement = connection.prepareStatement(SELECT_GAME);
+             ResultSet gameResultSet = gameStatement.executeQuery();
+
+        ) {
+            if (!gameResultSet.next()) {
+                return Optional.empty();
+            }
+
+            return findSavedGame(gameResultSet, connection);
+
+        } catch (SQLException exception) {
+            throw new IllegalStateException("[ERROR] 저장된 게임 조회에 실패했습니다.", exception);
+        }
+    }


try-with-resources를 사용해주셨네요, 다른 부분도 같이 통일해주세요

Hyunta · 2026-04-10T21:21:55Z

src/main/java/config/DatabaseInitializer.java

+                CONSTRAINT fk_piece_game
+                    FOREIGN KEY (game_id) REFERENCES game(id)
+                    ON DELETE CASCADE
+            )


CASCADE를 설정해주셨네요

Hyunta · 2026-04-10T21:25:36Z

src/main/java/model/move/ChariotMoveRule.java

+    private boolean isPalaceDiagonal(Move move){
+        Palace choPalace = Palace.from(Country.CHO);
+        if(choPalace.isDiagonalMove(move)){
+            return true;
+        }
+        Palace hanPalace = Palace.from(Country.HAN);
+        return hanPalace.isDiagonalMove(move);
+    }


Cannon, Chariot, Soldier 모두 같은 메서드를 구현하고 있네요.
중복을 없애볼 수 있을까요?

Hyunta · 2026-04-10T21:26:11Z

src/main/java/model/board/Palace.java

+    private List<PalaceRoute> routes(){
+        return List.of(
+                new PalaceRoute(Move.of(topLeft(), center())),
+                new PalaceRoute(Move.of(center(), bottomRight())),
+                new PalaceRoute(Move.of(topRight(), center())),
+                new PalaceRoute(Move.of(center(), bottomLeft())),
+                new PalaceRoute(Move.of(topLeft(), bottomRight())),
+                new PalaceRoute(Move.of(topRight(), bottomLeft()))
+        );
+    }


해당 메서드가 호출될 떄마다 새로운 객체가 생성되고 있습니다.
상수를 활용해보면 어떤가요?

Hyunta · 2026-04-10T21:26:56Z

src/main/java/model/board/Palace.java

+    private Position center(){
+        if(country==Country.CHO){
+            return Position.of(9, 5);
+        }
+        return Position.of(2, 5);


상단에 상수가 정의되어있는데 해당 값을 쓰는게 읽는 사람한테 문맥 전달이 잘 될 것 같습니다.

Hyunta · 2026-04-10T21:27:49Z

src/main/java/model/game/JanggiGame.java

+    public static JanggiGame restore(Board board, Country turn, boolean finished, Country winner) {
+        JanggiGame game = new JanggiGame(board);
+        game.turn = turn;
+        game.status = GameStatus.restore(finished, winner);
+        return game;


객체를 생성한 뒤에 덮어쓰는 방식으로 되어 있는데 생성할 때 주입해주면 안되나요?

Hyunta · 2026-04-10T21:28:57Z

src/main/java/model/board/Country.java

    private final String title;
    private final Direction forward;
    private final String color;
+    private final double bonusScore;


double 과 Double은 같은 값일까요? bonusscore() 메서드에서 반환할 때는 Bonus를 반환하고 있네요.

simhokyung added 20 commits April 5, 2026 15:41

feat: 궁성을 나타내는 Palace 도메인 구현

699ea03

feat: 장,사 궁성 대각선 이동 구현

4e8b5f5

refactor: 장,사 궁성 대각선 이동 구현 보완

9d04b0c

- 궁성에서 화살표 방향에 맞는 경우에만 대각선 이동 가능

feat: 궁성 범위 내에서 차의 이동 구현

45a199b

feat: 궁성 범위 내에서 포의 이동 구현

c41ea0f

feat: 궁성 범위 내에서 졸(병)의 이동 구현

f23f9c9

refactor: 차,포,졸이 상대편 궁성에 있을때도 이동규칙이 적용되도록 개선

f69abea

feat: 장이 잡히면 게임을 종료하는 규칙 구현

76a2c3a

- 게임의 진행/종료 상태를 관리하는 GameStatus 구현

feat: 남아 있는 기물에 대한 점수를 계산하는 기능 구현

c557521

- PieceType에 기물별 점수 추가 - Board에서 점수계산 로직 구현

feat: 게임 종료시 승리 나라와 남은 기물들에 대한 점수 출력

5918d7e

feat: h2 데이터베이스 설정 및 테이블 생성

35b8f00

feat: 저장 단위 생성 및 흐름 확인을 위한 InMemoryGameRepository 구현

c577ef8

feat: 게임 메타 정보 저장

7d3edc6

- SavedGame의 턴, 종료여부, 승자를 game 테이블에 저장

feat: piece테이블 저장 로직 구현

55f7d62

feat: db에서 savedGame을 찾아 반환하는 find 메서드 구현

a9a30fd

feat: board를 복원하는 GameRestorer 구현

4f39dc9

feat: JanggiGame을 복원하는 기능 구현

f1b0931

feat: db 반영하여 컨트롤러 수정

f309d22

feat: 저장된 게임 시작시 안내 문구 추가

c1b6f12

docs: README 갱신

a6f3872

Hyunta requested changes Apr 10, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[🚀 사이클2 - 미션 (기물 확장 + DB 적용)] 맥스 미션 제출합니다.#349

[🚀 사이클2 - 미션 (기물 확장 + DB 적용)] 맥스 미션 제출합니다.#349
simhokyung wants to merge 20 commits intowoowacourse:simhokyungfrom
simhokyung:step2

simhokyung commented Apr 10, 2026

Uh oh!

Hyunta left a comment

Uh oh!

Hyunta Apr 10, 2026

Uh oh!

Hyunta Apr 10, 2026

Uh oh!

Hyunta Apr 10, 2026

Uh oh!

Hyunta Apr 10, 2026

Uh oh!

Hyunta Apr 10, 2026

Uh oh!

Hyunta Apr 10, 2026

Uh oh!

Hyunta Apr 10, 2026

Uh oh!

Hyunta Apr 10, 2026

Uh oh!

Hyunta Apr 10, 2026

Uh oh!

Hyunta Apr 10, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

simhokyung commented Apr 10, 2026

체크 리스트

H2 DB 실행 방법

1. H2 콘솔 접속

2. DB 연결 정보 입력

3. 데이터 확인

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

DB 연결 구조 방식이 적절한지

Uh oh!

Hyunta 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