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

juntae6942 · 2026-03-31T13:27:18Z

체크 리스트

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

아래의 환경변수를 사용하여 h2 DB로 연동하였습니다.

db.url=jdbc:h2:./janggi;AUTO_SERVER=TRUE;INIT=RUNSCRIPT FROM 'classpath:init.sql'
db.username=sa
db.password=

현재 새로운 장기판으로 게임을 할 것인지 아니면 기존의 게임판을 찾아서 할 것인지 입력받는 로직으로 구현하였습니다.
다만 기존 장기판 리스트 조회는 구현하지 못하였습니다. 이점 참고해주시면 감사하겠습니다.

출력 예시

새로운 장기 게임을 시작하시겠습니까?(y,n)
n
기존 장기 게임방ID를 입력해주세요.
1
장기 게임을 시작합니다. 초나라부터 시작하고, 왼쪽 아래를 (0,0)으로 합니다.
[+, 마, 상, 사, +, 사, 상, 마, 차]
[+, +, +, +, 장, +, +, +, +]
[+, 포, +, +, +, +, +, 포, +]
[+, +, 졸, +, 졸, +, 졸, +, 졸]
[+, +, +, +, +, +, +, +, +]
[차, +, +, +, +, +, +, +, +]
[+, 졸, 졸, +, 졸, +, 졸, +, 졸]
[+, 포, +, +, +, +, +, 포, +]
[+, +, +, +, 장, +, +, +, +]
[+, 마, 상, 사, +, 사, 상, 마, 차]
한 : 71.5, 초 : 59.0
현재 초나라의 차례입니다.
움직일 기물의 출발지 좌표 입력해 주세요.(쉼표 기준으로 분리, ex. 0,0) :

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

1차 피드백을 기반으로 코드를 수정하였습니다. 제안 주셨던 내용을 고민해서 추상 골격 클래스를 확실하게 사용하는 방향을 모색해서 적용해보았습니다.

사이클 2에서는 Domain에 Controller와 Service를 추가하였는데, 각각의 레이어가 올바른 역할과 책임을 하고 있는지 궁금합니다.
Repository를 구현하면서 DAO를 사용하여 JDBC 기반으로 구현하였는데, 올바른 방식인지 궁금합니다.
DB에 저장하는 시점이 게임이 종료되었을 때, 매번 기물을 움직였을 때 중에서 요구사항에 이전에 진행하던 게임을 이어서 진행 할 수 있어야 한다고 하여 정합성을 위해서 매번 기물을 움직였을 때 저장하는 것으로 구현하였는데, 게임을 중지하는 입력을 추가로 받아서 그 경우만 저장하는 것과 현재 구현한 방법 중에서 각각의 트레이드 오프(정합성과 DB 연결 최소화 중)가 있을 것 같은데 아루라면 어떤 시점에 저장하는 방식으로 할 것인지 궁금합니다.

donghoony

벨로 안녕하세요~ 저번 사이클도 그렇고 늦은 시간임에도 빠르게 리뷰를 반영해주셨네요. 트랜잭션이라는 개념을 가져와서 구현해주셔서 깜짝 놀랐습니다. 그와 동시에 트랜잭션의 책임은? 인프라 레벨의 추상화는? 와 같은 객체/계층의 책임 질문을 더 물어왔어요. 😈

남겨드린 리뷰를 잘 확인해보시고 반영해주세요. 다음 요청 때 다시 만나요~!

donghoony · 2026-04-03T14:42:29Z

src/main/java/janggi/infra/JdbcBoardRepository.java

+    }
+
+    @Override
+    public JanggiGame loadGame(Long gameRoomId) {


gameRoomId를 primitive를 받으면 아래 검증이 불필요하지 않을까요? 위 update에서는 검증을 하지 않고 있어서, 이럴 때에는 오히려 받을 때 제한을 두는 게 인지부하도 줄어들 것 같습니다.

donghoony · 2026-04-03T14:43:55Z

src/main/java/janggi/infra/dao/GameRoomDao.java

dao나 repository를 믿고 쓸 수 있을까요? 😢

donghoony · 2026-04-03T14:46:49Z

src/main/java/janggi/infra/JdbcBoardRepository.java

+            connection.commit();
+            return result;
+        } catch (SQLException e) {
+            connection.rollback();


Dao의 예외가 롤백으로 진행될까요? 😈

이 함수는 SQLException을 던지나요?

이전 리뷰에서는 SQLException을 Dao에서 던지는 책임에 대해서 여쭤봤었습니다. 상황이 바뀐 지금은 SQLException을 외부로 던지면, 이를 받아 어떤 역할을 할 수 있어보이는 것처럼 보이네요. 이처럼 코드 맥락은 현재 코드가 어떻게 짜여졌는지에 따라 유연하게 변한답니다!

donghoony · 2026-04-03T14:47:30Z

src/main/java/janggi/presentation/dto/GameCommand.java

+
+import java.util.Arrays;
+
+public enum GameCommand {


👍🏻 표현 계층에서의 객체를 잘 생성해주셨다고 생각합니다. 적절한 책임도 가져가고 있네요

donghoony · 2026-04-03T14:49:01Z

src/main/java/janggi/infra/JdbcBoardRepository.java

+    private final DataConnectionManager manager;
+    private final GameRoomDao roomDao = new GameRoomDao();
+    private final PiecesDao piecesDao = new PiecesDao();


(궁금) manager는 외부에서 받아내는데, Dao들은 내부에서 new로 생성하는 이유가 있나요?

donghoony · 2026-04-03T15:21:55Z

src/main/java/janggi/presentation/JanggiGameController.java

+        Console.close();
+    }
+
+    private void playGame(Long roomId) {


Suggested change

private void playGame(Long roomId) {

private void playGame(long roomId) {

전체적으로 확인해주세요. 한 번 허용하게 되는 nullable은 돌이킬 수 없게 되곤 합니다.

donghoony · 2026-04-03T15:25:52Z

src/main/java/janggi/infra/dao/PiecesDao.java

+        moveStatement.executeUpdate();
+    }
+
+    public List<PieceData> findAllByRoomId(Long roomId, Connection connection) {


이 친구는 예외를 래핑하고 있는데 통일성 지켜주세요~!

donghoony · 2026-04-03T15:27:57Z

src/main/java/janggi/infra/dao/GameRoomDao.java

+
+public class GameRoomDao {
+
+    public Long save(GameRoomData data, Connection connection) throws SQLException {


트랜잭션이 도입되면서 변환의 책임을 밖으로 넘긴 이유가 생긴 것 같습니다. 👍🏻

donghoony · 2026-04-03T15:30:31Z

src/main/java/janggi/JanggiApplication.java

-        OutputView.printWinner(game.getWinner());
-        Console.close();
+    public static void main(String[] args) throws SQLException {
+        Server server = Server.createTcpServer("-tcp", "-tcpAllowOthers", "-tcpPort", "9092").start();


(선택) 시간이 많이 남으시거나 사이클의 연장선으로 생각하신다면 도전해보셔도 좋겠습니다 ㅎㅎ

donghoony · 2026-04-03T15:35:48Z

src/main/java/janggi/application/JanggiGameService.java

+        JanggiGame game = repository.loadGame(roomId);
+        game.play(from, to);
+        repository.update(roomId, from, to, game);
+    }


PR 질문에 대한 의견) play가 객체를 변형하고 있으니, 이를 반환하여 활용하면 DB 부하를 줄일 수 있지 않을까요? 동시성과 DB 부하 모두 챙길 수 있을 것 같아요.

juntae6942 added 30 commits March 28, 2026 15:42

refactor(OutputView): 출력 로직 분리

009de5d

chore(Point): 패키지 이동

53df5a9

refactor(Route): Point 리스트 역할부여

ce78b3b

refactor(Route): Points와 Route 책임 분리

1aa202e

test(Route): Points와 Route 책임 분리에 따른 수정

5a5da77

refactor(Route): Points와 Route 정의

aa9a62f

refactor(PositionInfo): col, row 통일

0f1da61

fix(Route): boolean 반환값 수정

fd63ca3

refactor(Board): 변수명 수정

4f41b55

refactor(Route): 매직 넘버 제거

506a318

refactor(Piece): 수학적 계산 로직을 방향을 사용한 로직으로 수정

2ee2313

refactor(Piece): col,row 통일

ee3d02f

test(Cha): 궁성 안에서 움직임 테스트 작성

1d3c1bf

feat(Cha): 궁성 안에서 움직임 구현

c128726

test(Pho): 궁성 안에서 움직임 테스트 작성

f9952c7

refactor(Pho): 궁성 안에서 움직임 구현

6f79f85

test(Board): 궁성 안에서 움직임 테스트 작성

db7292e

feat: 궁성 안에서 움직임 구현을 위한 방향 맵 구현

8fc3f25

test(jang&sa): 궁성 밖으로 못나감 테스트 작성

5c2096f

feat(jang&sa): 궁성 밖으로 못나감 검증 구현

e36b69f

test(jol): 궁성에서 한나라 졸은 위로 못가는 검증 테스트 작성

b200d50

feat(jol): 궁성에서 대각선 이동 기능 구현

4cddc93

refactor(jang&sa): 대각선 방향 제한 수정

ac47705

refactor(Cha&Pho): 대각선 방향 제한 코드 수정

27b3b6e

refactor(Point): 매직넘버 제거

3b1797b

refactor(Piece): 검증 분리

1dccc84

chore(Database): h2, jdbc 의존성 추가

2d06433

chore(Board): 패키징 이동

80ea3f1

chore(Board): 패키징 이동

331725b

style(Cha): 개행 제거

15aed38

juntae6942 added 29 commits April 2, 2026 23:51

refactor(MaDirection): 인스턴스 변수 줄이기

7b33741

refactor(GameCommand): y,n 명령어 역할 부여

8f147b5

style(JdbcBoardRepository): 컨벤션 수정

12bf254

refactor(View): View 객체로 변경

92f051a

refactor(Board): board 초기화 수정

73ba061

refactor(PieceData): 변수명 수정

8359400

refactor(View): view 분리에 따른 중복 Map 제거

36c2748

refactor(Service): 동시성 제어를 위해 Map으로 필드 변경

8d2ca59

refactor(Repository): nullable하지 않은 메서드 수정

c404444

refactor(Repository): 커넥션풀로 변경

0029a5a

refactor(Service): 게임방 ID 검증 추가

825229e

refactor(Repository): 트랜잭션 추가

5f46ac9

refactor(Repository): 트랜잭션 템플릿 적용

6cb316d

refactor(Controller): primitive type으로 수정 및 roomId 검증 추가

ce5ea69

refactor(Service): 파라미터 네이밍 수정 및 primitive type 변경

c631ad5

refactor(Piece): getTeam() 추가

24c9daa

refactor(Service): 동시성 제어

e0430f8

refactor(Repository): DB 커넥션 관리 분리 및 roomId 검증 이동

b04d73e

refactor(Repository): DB 커넥션 관리 분리 및 roomId 검증 이동

315983e

refactor(Controller): 필요없어진 메서드 제거

f4419bb

refactor(init): unique 제약조건 추가

41a7302

refactor(Board): 가산점 계산 책임 분리

dfd5eb3

refactor(GameRoom): 예외처리 추가

8f1c77a

refactor(Repository): 예외처리 추가

252829d

refactor(Castle): 궁성 기능 분리

8bdf44d

test(HanScore): 한나라 보너스 점수 계산 책임 위임에 따른 테스트 수정

40e5f49

refactor(Team): 방향 판단 team에 위임

db8cf19

refactor(BoardStatus): view 의존적 메서드 분리

d3297ea

refactor(Repository): status 수정에 따른 로직 수정

2343f65

donghoony requested changes Apr 3, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

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

[🚀 사이클2 - 미션 (기물 확장 + DB 적용)] 벨로 미션 제출합니다.#270
juntae6942 wants to merge 91 commits intowoowacourse:juntae6942from
juntae6942:step2

juntae6942 commented Mar 31, 2026 •

edited

Loading

Uh oh!

donghoony left a comment

Uh oh!

donghoony Apr 3, 2026

Uh oh!

donghoony Apr 3, 2026

Uh oh!

donghoony Apr 3, 2026

Uh oh!

donghoony Apr 3, 2026

Uh oh!

donghoony Apr 3, 2026

Uh oh!

donghoony Apr 3, 2026

Uh oh!

donghoony Apr 3, 2026

Uh oh!

donghoony Apr 3, 2026

Uh oh!

donghoony Apr 3, 2026

Uh oh!

donghoony Apr 3, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	private void playGame(Long roomId) {
	private void playGame(long roomId) {


		public class GameRoomDao {

		public Long save(GameRoomData data, Connection connection) throws SQLException {

Conversation

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

Uh oh!

체크 리스트

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

Uh oh!

donghoony 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

juntae6942 commented Mar 31, 2026 •

edited

Loading