[🚀 사이클2 - 미션 (기물 확장 + DB 적용)] 루드비코 미션 제출합니다. by HyoYoonNam · Pull Request #325 · woowacourse/java-janggi

HyoYoonNam · 2026-04-08T02:32:41Z

안녕하세요, 주노!
이번에도 미리 감사드립니다~~!

체크 리스트

미션의 필수 요구사항을 모두 구현했나요?
- Repository 계층에서 Connection과 PreparedStatement는 하나의 try-with-resources 블록으로 자원을 반납할 수 있었지만 해당 블록 내에서 생성해야 되는 ResultSet을 닫을 방법이 추가로 필요했고, 이를 위해 내부에서 try-with-resources를 중첩 사용했습니다. (아래에 해당 코드)

java-janggi/src/main/java/repository/JanggiGameRepository.java

Lines 56 to 79 in 0f7650a

    
           public long save(JanggiGame janggiGame) { 
        
               String sql = "INSERT INTO game (current_turn) VALUES (?)"; 
        
               try ( 
        
                       Connection conn = dataSource.getConnection(); 
        
                       PreparedStatement pstmt = conn.prepareStatement(sql, Statement.RETURN_GENERATED_KEYS) 
        
               ) { 
        
                   pstmt.setString(1, janggiGame.currentTurn().name()); 
        
                   pstmt.executeUpdate(); 
        
                   // TODO: ResultSet 자원도 닫아야 됨을 인지해서 일단 중첩 try-with-resources로 대응했지만, 중첩없이 해결 가능한 지 고민 필요. 
        
                   try (ResultSet rs = pstmt.getGeneratedKeys()) { 
        
                       if (rs.next()) { 
        
                           long gameId = rs.getLong(1); 
        
                           insertPieces(conn, gameId, janggiGame); 
        
                           return gameId; 
        
                       } 
        
                   } 
        
                   throw new SQLException("게임 생성에 실패했습니다."); 
        
               } catch (SQLException e) { 
        
                   throw new IllegalStateException(e); 
        
               } 
        
           }

Gradle test를 실행했을 때, 모든 테스트가 정상적으로 통과했나요?
애플리케이션이 정상적으로 실행되나요?

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

cycle1의 merge 시점에 남겨주신 빈 기물은 매번 생성되어야 할까?에 대해 다음처럼 반영했습니다!

java-janggi/src/main/java/domain/piece/AlivePieces.java

Lines 35 to 37 in da4b0e8

    
           public Piece placedAt(Intersection intersection) { 
        
               return alivePieces.getOrDefault(intersection, Piece.EMPTY); 
        
           }

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

Lines 7 to 9 in da4b0e8

    
           public final class Piece { 
        
               public static final Piece EMPTY = new Piece(PieceType.EMPTY, Side.NONE);

기존에는 Piece의 movablePaths() 메서드 내에서 PieceType의 movablePaths()와 movableDestinations를 절차적으로 호출했다. 이를 PieceType이 자율적으로 행동할 수 있도록 movableDestinations만 외부로 공개한다. 또한 Piece의 movablePaths()가 List<Intersection>을 반환하는 것과 달리, 메서드명이 'Paths'여서 이를 'Destinations'로 변경한다.

기존에는 Piece.EMPTY 상수를 public으로 제공하긴 했지만, new Piece(PieceType.EMPTY, Side.NONE)처럼 생성자로 강제 생성하면 새로운 빈 기물 인스턴스가 리턴되는 것을 막을 수 없었다. Piece의 생성자를 private으로 감추고, 정적 팩터리 메서드 of()만 외부로 열어 만약 빈 기물을 생성하려고 하면 내부에서 상수로 관리하는 EMPTY를 리턴하도록 한다.

기존에는 'Point'를 사용했다. 하지만 새로운 요구사항에서의 '점수'가 장기 도메인에서 'Point'로 사용되기 때문에 혼란을 방지하고자 'Position'으로 변경한다.

기존 문제 1: 왕이 잡혔는데도 점수 기반으로 승패를 판정 해결 1: 왕이 잡혔다면, 무조건 잡히지 않은 쪽 승리. 기존 문제 2: 왕이 잡혔을 때만 게임이 종료 해결 2: 왕이 잡히지 않았더라도 양측 진영 점수가 모두 30점 미만이면 종료 기타 개선 1: TestFixture 도입하여 초기 상황 셋팅을 단순화 기타 개선 2: 승패 판정의 근거(왕 잡힘 or 점수 높음)를 함께 출력 기타 개선 3: GameResult record 도입

해당 메서드는 같은 interface의 default method인 palaceDiagonalPaths에서만 호출되기 때문에 불필요한 외부 노출을 없애고자 private으로 변경한다.

Choi-JJunho

루드비코 step2도 고생하셨습니다~
질문주신내용과 코드 관련해서 몇가지 코멘트 남겼으니 확인 부탁드릴게요

Choi-JJunho · 2026-04-08T13:51:53Z

sql/ddl.sql

CREATE TABLE IF NOT EXISTS piece ( game_id INTEGER NOT NULL, position_row INTEGER, position_file INTEGER, side VARCHAR(10) NOT NULL, piece_type VARCHAR(10) NOT NULL, FOREIGN KEY (game_id) REFERENCES game(game_id), PRIMARY KEY (game_id, position_row, position_file) );

varchar 10으로 설정해준 이유가 있을까요?

Choi-JJunho · 2026-04-08T14:05:27Z

src/main/java/controller/JanggiController.java

좋은 시도였다고 생각합니다~! 다만 인스턴스 변수로 Map을 두면서 표현하려다보니 과도한 추상화처럼 다가오기도 하네요

YANGI : You Ain’t Gonna Need It 라는 말이 있습니다. 필요한 작업만을 하라는 의미인데요. 프로그램을 작성하다보면 지금처럼 현재는 사용하지 않지만, 확장성을 고려해 미리 작업하는 경우가 등장하기도 합니다. 주어진 요구사항에 집중하고, 이런 행위를 지양하라는 의미로 사용되는 말인데 이 의미를 반영해서 간단하게 표현해볼까요?

조건이 2~3개밖에 안되는 이런 경우는 간단한 조건 분기가 오히려 읽기 쉬울 수 있다고 생각합니다

Choi-JJunho · 2026-04-08T14:07:01Z

src/main/java/domain/game/Side.java

+    public abstract double bonusPoint();
+


현재 Side가 가진 책임이 많아지면서 이런 고민이 드는것 같네요
Side의 역할과 책임을 한번 다시 생각해보면서 적절하게 구성되어있는지 검토해보면 좋겠네요

bonusPoint 같은 메서드는 정말로 Side가 가져야하는 행위일까요?

Choi-JJunho · 2026-04-08T14:10:11Z

src/main/java/domain/move/strategy/StraightLineMovement.java

-public final class StraightLineMovement extends Movement {
+public final class StraightLineMovement extends Movement implements PalaceMovement {

    private static final MoveAmount MOVE_AMOUNT = new MoveAmount(1);

    @Override
    protected List<Path> candidatePaths(Intersection from, Side side) {
-        return allDirectionPaths(from, side);
+        List<Path> paths = new ArrayList<>();
+
+        List<Path> orthogonalPaths = allDirectionPaths(from, side);
+        List<Path> palaceDiagonalPaths = PalaceMovement.super.palaceDiagonalPathsForStraight(from);
+        paths.addAll(orthogonalPaths);
+        paths.addAll(palaceDiagonalPaths);
+
+        return List.copyOf(paths);


제 의견을 먼저 말씀드리기 전에 루드비코의 생각을 먼저 듣고싶네요! (질문에 질문으로 답하게되었네요 😅 )
루드비코가 생각하는 인터페이스의 본분은 무엇인지 궁금합니다! default 메서드는 왜 존재할까요?

Choi-JJunho · 2026-04-08T14:11:40Z

src/main/java/domain/move/strategy/PalaceMovement.java

+        if (from.row() == 1 || from.row() == 3) {
+            return new Intersection(2, 5);
+        }
+
+        return new Intersection(9, 5);
+    }


의미가 모호한 값들 (매직넘버)들이 자주 등장하는 것 같네요 👀 상수로 추출할 방법이 없을까요?

(PalaceMovement와 Intersection에서 사용되던 매직넘버 관련)

궁성과 관련한 판단 및 매직넘버를 Palace 메서드 및 상수로 이동했습니다!

java-janggi/src/main/java/domain/board/Palace.java

Lines 6 to 84 in 9328d22

public final class Palace {

private static final int MIN_FILE = 4;

private static final int CENTER_FILE = 5;

private static final int MAX_FILE = 6;

private static final int HAN_MIN_ROW = 1;

private static final int HAN_CENTER_ROW = 2;

private static final int HAN_MAX_ROW = 3;

private static final int CHO_MIN_ROW = 8;

private static final int CHO_CENTER_ROW = 9;

private static final int CHO_MAX_ROW = 10;

private Palace() {

}

public static boolean contains(Intersection pos, Side side) {

final int row = pos.row();

final int file = pos.file();

boolean inPalaceFile = file >= MIN_FILE && file <= MAX_FILE;

if (side == Side.HAN) { // 만약 Side enum의 이름이 다르다면 맞게 수정해주세요

return inPalaceFile && row >= HAN_MIN_ROW && row <= HAN_MAX_ROW;

}

return inPalaceFile && row >= CHO_MIN_ROW && row <= CHO_MAX_ROW;

}

public static boolean isCenter(Intersection pos) {

final int row = pos.row();

final int file = pos.file();

return file == CENTER_FILE &&

(row == HAN_CENTER_ROW || row == CHO_CENTER_ROW);

}

public static boolean isCorner(Intersection pos) {

final int row = pos.row();

final int file = pos.file();

boolean isHanCorner = Math.abs(row - HAN_CENTER_ROW) == 1;

boolean isChoCorner = Math.abs(row - CHO_CENTER_ROW) == 1;

boolean isFileCorner = Math.abs(file - CENTER_FILE) == 1;

return (isHanCorner || isChoCorner) && isFileCorner;

}

public static Intersection getCenterOf(Intersection cornerPos) {

if (cornerPos.row() <= HAN_MAX_ROW) {

return new Intersection(HAN_CENTER_ROW, CENTER_FILE);

}

return new Intersection(CHO_CENTER_ROW, CENTER_FILE);

}

public static List<Intersection> getCornersOf(Intersection center) {

if (!isCenter(center)) {

throw new IllegalStateException("중앙 좌표가 아닙니다.");

}

final int r = center.row();

final int f = center.file();

return List.of(

new Intersection(r - 1, f - 1), new Intersection(r - 1, f + 1),

new Intersection(r + 1, f - 1), new Intersection(r + 1, f + 1)

);

}

public static Intersection getOppositeCornerOf(Intersection corner) {

if (!isCorner(corner)) {

throw new IllegalArgumentException("코너 좌표가 아닙니다.");

}

Intersection center = getCenterOf(corner);

int deltaRow = center.row() - corner.row();

int deltaFile = center.file() - corner.file();

return new Intersection(center.row() + deltaRow, center.file() + deltaFile);

}

}

Choi-JJunho · 2026-04-08T14:28:26Z

src/main/java/domain/game/Side.java

+    public static Side from(String sideName) {
+        return Arrays.stream(values())
+                .filter(side -> side.name().equals(sideName))
+                .findAny()
+                .orElse(NONE);
+    }


사용자의 입력이나, DB에 잘못된 값이 저장되어있으면 예외없이 NONE이 발생하게되는데요 이 부분은 의도하신 부분일까요?

의도하지 않았고, 제가 제대로 확인하지 않아 발생한 상황이었습니다.

해당 리뷰를 반영하여 찾지 못한 경우 orElseThrow()를 던지도록 개선했는데요.

이 경우 결국 Enum이 자체적으로 제공하는 valueOf()와 로직이 동일하다고 판단하여 Side.from() 자체를 제거하고, 이를 호출하던 곳에서는 Side.valueOf()를 호출하도록 변경했습니다.

public static <T extends Enum<T>> T valueOf(Class<T> enumClass, String name) { T result = enumClass.enumConstantDirectory().get(name); if (result != null) return result; if (name == null) throw new NullPointerException("Name is null"); throw new IllegalArgumentException( "No enum constant " + enumClass.getCanonicalName() + "." + name); }

Choi-JJunho · 2026-04-08T14:31:14Z

src/main/java/dto/MoveCommand.java

+    public static MoveCommand from(String rawMoveCommand) {
+        if (rawMoveCommand.equalsIgnoreCase("exit")) {
+            // TODO 이것도 null 넣어서 뱉는 거 호출부에서는 방어적으로 사용하는데, 애초에 null 뱉는 방식이 문제긴 한 듯
+            return new MoveCommand(Type.EXIT, null);
+        }


TODO가 남아있네요 👀 이부분은 어떻게 해결할 수 있을까요?

null이나 값 대신 Optional을 뱉도록 하여 이를 사용하는 컨트롤러에서 검증 후 사용하도록 했습니다.

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

Lines 156 to 159 in eb25725

private GameWrapper processTurn(MoveCommand moveCommand, GameWrapper gameWrapper) {

JanggiGame janggiGame = gameWrapper.game();

Intersection startPosition = moveCommand.selectedToMove()

.orElseThrow(() -> new IllegalArgumentException("이동할 좌표(x,y) 또는 종료(exit)를 입력해주세요."));

Choi-JJunho · 2026-04-08T14:32:48Z

src/main/java/domain/board/Intersection.java

+    public boolean isPalaceCenter() {
+        return (row == 2 || row == 9)
+                && file == 5;
+    }
+
+    public boolean isPalaceCorner() {
+        return (row == 1 || row == 3 || row == 8 || row == 10)
+                && (file == 4 || file == 6);
+    }


하드코딩된 값들이 많네요 💫
이런것들은 어떻게 관리하기 편하게 만들 수 있을까요?

(PalaceMovement와 Intersection에서 사용되던 매직넘버 관련)

궁성과 관련한 판단 및 매직넘버를 Palace 메서드 및 상수로 이동했습니다!

java-janggi/src/main/java/domain/board/Palace.java

Lines 6 to 84 in 9328d22

public final class Palace {

private static final int MIN_FILE = 4;

private static final int CENTER_FILE = 5;

private static final int MAX_FILE = 6;

private static final int HAN_MIN_ROW = 1;

private static final int HAN_CENTER_ROW = 2;

private static final int HAN_MAX_ROW = 3;

private static final int CHO_MIN_ROW = 8;

private static final int CHO_CENTER_ROW = 9;

private static final int CHO_MAX_ROW = 10;

private Palace() {

}

public static boolean contains(Intersection pos, Side side) {

final int row = pos.row();

final int file = pos.file();

boolean inPalaceFile = file >= MIN_FILE && file <= MAX_FILE;

if (side == Side.HAN) { // 만약 Side enum의 이름이 다르다면 맞게 수정해주세요

return inPalaceFile && row >= HAN_MIN_ROW && row <= HAN_MAX_ROW;

}

return inPalaceFile && row >= CHO_MIN_ROW && row <= CHO_MAX_ROW;

}

public static boolean isCenter(Intersection pos) {

final int row = pos.row();

final int file = pos.file();

return file == CENTER_FILE &&

(row == HAN_CENTER_ROW || row == CHO_CENTER_ROW);

}

public static boolean isCorner(Intersection pos) {

final int row = pos.row();

final int file = pos.file();

boolean isHanCorner = Math.abs(row - HAN_CENTER_ROW) == 1;

boolean isChoCorner = Math.abs(row - CHO_CENTER_ROW) == 1;

boolean isFileCorner = Math.abs(file - CENTER_FILE) == 1;

return (isHanCorner || isChoCorner) && isFileCorner;

}

public static Intersection getCenterOf(Intersection cornerPos) {

if (cornerPos.row() <= HAN_MAX_ROW) {

return new Intersection(HAN_CENTER_ROW, CENTER_FILE);

}

return new Intersection(CHO_CENTER_ROW, CENTER_FILE);

}

public static List<Intersection> getCornersOf(Intersection center) {

if (!isCenter(center)) {

throw new IllegalStateException("중앙 좌표가 아닙니다.");

}

final int r = center.row();

final int f = center.file();

return List.of(

new Intersection(r - 1, f - 1), new Intersection(r - 1, f + 1),

new Intersection(r + 1, f - 1), new Intersection(r + 1, f + 1)

);

}

public static Intersection getOppositeCornerOf(Intersection corner) {

if (!isCorner(corner)) {

throw new IllegalArgumentException("코너 좌표가 아닙니다.");

}

Intersection center = getCenterOf(corner);

int deltaRow = center.row() - corner.row();

int deltaFile = center.file() - corner.file();

return new Intersection(center.row() + deltaRow, center.file() + deltaFile);

}

}

Choi-JJunho · 2026-04-08T14:37:28Z

src/main/java/repository/JanggiGameRepository.java

+    public long save(JanggiGame janggiGame) {
+        String sql = "INSERT INTO game (current_turn) VALUES (?)";
+
+        try (
+                Connection conn = dataSource.getConnection();
+                PreparedStatement pstmt = conn.prepareStatement(sql, Statement.RETURN_GENERATED_KEYS)
+        ) {
+            pstmt.setString(1, janggiGame.currentTurn().name());
+            pstmt.executeUpdate();
+            // TODO: ResultSet 자원도 닫아야 됨을 인지해서 일단 중첩 try-with-resources로 대응했지만, 중첩없이 해결 가능한 지 고민 필요.
+            try (ResultSet rs = pstmt.getGeneratedKeys()) {
+                if (rs.next()) {
+                    long gameId = rs.getLong(1);
+                    insertPieces(conn, janggiGame, gameId);
+
+                    return gameId;
+                }
+            }
+
+            throw new SQLException("게임 생성에 실패했습니다.");
+        } catch (SQLException e) {
+            throw new IllegalStateException(e);
+        }
+    }


game INSERT와 piece INSERT가 트랜잭션으로 묶여있지 않네요. insert가 완료되고 예외가 발생하면 어떤 일이 발생할까요?

게임은 저장되지만 그에 대한 기물 정보는 제대로 반영되지 않는 심각한 문제가 발생하게 됩니다.

트랜잭션 관리를 리포지토리가 아닌 TransactionTemplate가 담당하도록 변경하는 과정에서 해당 문제를 해결했습니다!

java-janggi/src/main/java/support/TransactionTemplate.java

Lines 20 to 40 in 9328d22

public <T> T execute(TransactionCallback<T> action) {

Connection conn = null;

try {

conn = dataSource.getConnection();

conn.setAutoCommit(false);

T doingResult = action.doInTransaction(conn);

conn.commit();

return doingResult;

} catch (SQLException e) {

rollback(conn);

throw new DataAccessException(e);

} catch (RuntimeException e) {

rollback(conn);

throw e;

} finally {

close(conn);

}

}

java-janggi/src/main/java/service/JanggiService.java

Lines 23 to 29 in 9328d22

public GameWrapper createGame(Board board) {

return transactionTemplate.execute(conn -> {

JanggiGame janggiGame = JanggiGame.create(board);

long generatedKey = repository.save(conn, janggiGame);

return new GameWrapper(generatedKey, janggiGame);

});

}

Choi-JJunho · 2026-04-08T14:38:07Z

src/main/java/dto/GameSummary.java

+
+import java.util.Date;
+
+public record GameSummary(long id, Date startedAt, String currentTurn) {


Date와 LocalDateTime 의 차이를 한번 확인해보고 이야기해주세요!

Date의 경우 JDK 1.0부터 포함된 것으로 '가변적'이라는 큰 단점을 가지고 있음을 확인했습니다.
또한 java.util 패키지 하위에 위치해있는데, 오라클에서 "Contains the collections framework, legacy collection classes, event model, date and time facilities" 라고 설명하는, 시간을 다루는 '레거시'한 방법임을 확인했습니다.

이후 JDK 1.8에서 날짜와 시간을 전용으로 다루는 java.time 패키지가 등장했고, LocalDateTime이 여기에 속하게 됩니다.
가장 큰 변화로는 '불변'이라는 것이고, 이외에 날짜/시간 계산을 위한 편의 메서드 추가 등이 되었음을 확인했습니다.

The main API for dates, times, instants, and durations.
...
All the classes are immutable and thread-safe.

결과적으로, 다음처럼 LocalDateTime을 사용하도록 변경했습니다!

java-janggi/src/main/java/dto/GameSummary.java

Lines 5 to 6 in 43dcc80

public record GameSummary(long id, LocalDateTime startedAt, String currentTurn) {

}

java-janggi/src/main/java/repository/JanggiGameRepository.java

Lines 127 to 142 in 43dcc80

public List<GameSummary> findAll(Connection conn) {

String sql = ""

+ "SELECT game_id, created_at, current_turn "

+ "FROM game ";

List<GameSummary> gameSummaries = new ArrayList<>();

try (PreparedStatement pstmt = conn.prepareStatement(sql)) {

try (ResultSet rs = pstmt.executeQuery()) {

while (rs.next()) {

gameSummaries.add(new GameSummary(

rs.getLong("game_id"),

rs.getObject("created_at", LocalDateTime.class),

rs.getString("current_turn")

));

}

}

기존에는 트랜잭션 관리를 서비스 계층에서 수행했다. 관리를 위한 중복 코드를 템플릿화한다. 또한 덕분에 서비스 계층에서는 DB 종속적인 예외를 다루지 않아도 된다.

기존의 from()은 찾지 못하면 NONE을 리턴했다. 하지만 이를 필요로 하는 곳이 없었고, 오히려 조용한 실패로 인해 문제 원인을 파악하기 어렵게 했다. 이를 해결하고자 orElse(NONE)을 orElseThrow(...)으로 변경했더니, 그 구조가 Enum이 자체적으로 제공하는 valueOf()와 동일함을 확인했다. 따라서 중복적으로 구현하지 않고, JDK 상에서 이미 구현된 Enum.valueOf()를 사용한다.

진영별 보너스 점수, 즉 '점수'란 게임을 진행하는 상황에서 필요한 개념이다. 따라서 진영 그 자체로 보너스 점수를 가지는 것이 아니라, 게임에서 책임을 가짐이 적절하다.

기존에는 DB 관련 예외만 핸들링하여 비즈니스 로직에서 발생한 예외에는 롤백이 수행되지 않는 문제가 있다. 이를 해결하기 위해 DB 예외 뿐만 아니라 RuntimeException도 추가로 다룬다.

HyoYoonNam

안녕하세요, 주노!

이번에도 잘 부탁드리고, 새롭게 시작되는 한 주도 화이팅입니다!!

HyoYoonNam · 2026-04-12T11:35:37Z

src/main/java/controller/JanggiController.java

컨트롤러 내부에서 if 분기를 추상화하여 가지던 Map을 제거하고, 명시적으로 if 문을 사용하도록 변경했습니다!

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

Lines 92 to 106 in c86894b

private void dispatchLoadCommand(LoadCommand loadCommand, List<Long> selectableNumbers) {

long gameNumberToLoad = loadCommand.gameNumberToLoad();

if (!selectableNumbers.contains(gameNumberToLoad)) {

throw new IllegalArgumentException(gameNumberToLoad + "는 유효하지 않는 번호입니다.");

}

if (loadCommand.isNewGame()) {

newGameFlow();

return;

}

if (loadCommand.isLoadGame()) {

continueGame(gameNumberToLoad);

return;

}

HyoYoonNam · 2026-04-12T11:55:42Z

src/main/java/dto/MoveCommand.java

+    public static MoveCommand from(String rawMoveCommand) {
+        if (rawMoveCommand.equalsIgnoreCase("exit")) {
+            // TODO 이것도 null 넣어서 뱉는 거 호출부에서는 방어적으로 사용하는데, 애초에 null 뱉는 방식이 문제긴 한 듯
+            return new MoveCommand(Type.EXIT, null);
+        }


null이나 값 대신 Optional을 뱉도록 하여 이를 사용하는 컨트롤러에서 검증 후 사용하도록 했습니다.

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

Lines 156 to 159 in eb25725

private GameWrapper processTurn(MoveCommand moveCommand, GameWrapper gameWrapper) {

JanggiGame janggiGame = gameWrapper.game();

Intersection startPosition = moveCommand.selectedToMove()

.orElseThrow(() -> new IllegalArgumentException("이동할 좌표(x,y) 또는 종료(exit)를 입력해주세요."));

HyoYoonNam · 2026-04-12T12:38:11Z

src/main/java/domain/piece/Piece.java

+    public int toPoint() {
+        return type.point();
+    }
+
+    public PieceType getType() {
+        return type;
+    }
+
+    public Side getSide() {
+        return side;
+    }
+


과거(우테코 입과 전)에 말씀하신 toDbRow()같은 매핑 메서드를 도메인 안에 두는 방식을 배웠었는데, 크루들과 이야기하다 보니 '편하긴 하지만 도메인이 DB나 뷰에 종속되는 문제가 생긴다'로 결론내려 사용하지 않았습니다.

PieceMapper 중간 다리를 놓는다는 방향은 생각하지 못했고, 덕분에 적용해봤습니다!

적용 과정에서 '매퍼를 사용하더라도 결국 도메인의 getter는 열려야 한다'는 점을 확인했고, 매퍼의 의의는 'getter를 호출하는 곳을 매퍼 한 곳으로 응집'시키는 것이라고 이해했습니다.

추가로, PieceMapper를 사용할 곳이 repository로 한정된다고 판단하여 클래스 접근 제어자를 package-private로 설정했습니다(하위 메서드들의 경우 클래스 수준의 접근 제어자를 따라감으로 알고 있지만, 범위를 한정한다는 의미를 강조하기 위해 메서드들의 접근 제어자들도 public이 아닌 package-private로 설정했습니다).

java-janggi/src/main/java/repository/PieceMapper.java

Lines 8 to 29 in 1410cab

final class PieceMapper {

private PieceMapper() {

}

static PieceEntity toEntity(long gameId, Intersection position, Piece piece) {

return new PieceEntity(

gameId,

position.row(),

position.file(),

piece.getType().name(),

piece.getSide().name()

);

}

static Piece toPiece(String typeName, String sideName) {

return Piece.of(

PieceType.from(typeName),

Side.valueOf(sideName)

);

}

}

java-janggi/src/main/java/repository/JanggiGameRepository.java

Lines 159 to 167 in 1410cab

for (Entry<Intersection, Piece> entry : map.entrySet()) {

PieceEntity entity = PieceMapper.toEntity(gameId, entry.getKey(), entry.getValue());

pstmt.setInt(2, entity.row());

pstmt.setInt(3, entity.file());

pstmt.setString(4, entity.type());

pstmt.setString(5, entity.side());

pstmt.addBatch();

}

HyoYoonNam · 2026-04-12T12:46:11Z

src/test/java/repository/MemoryDBConnectionUtil.java

+
+public final class MemoryDBConnectionUtil {
+
+    private static final String JDBC_SQLITE_URL = "jdbc:sqlite::memory:?cache=shared";


'리뷰어가 clone 후 별 다른 설정없이 바로 Application을 실행할 수 있게 함'이 제가 중요하게 생각한 부분입니다.

이때 MySQL은 로컬 설치를 요구하거나, Docker를 사용하더라도 도커의 설치가 요구되기 때문에 선택에서 배제했습니다.
(제가 찾은 바로는 그랬는데, 방법이 있음에도 조사가 미숙해서 내려진 결론이라면 죄송하다는 말씀드립니다!!)

이후 H2와 SQLite가 후보로 남았는데, H2는 과거에 간단하게나마 사용한 경험이 있어 이번 기회에 새로운 시도를 하고자 SQLite를 선택했습니다.

HyoYoonNam · 2026-04-12T12:55:59Z

src/main/java/dto/GameSummary.java

+
+import java.util.Date;
+
+public record GameSummary(long id, Date startedAt, String currentTurn) {


Date의 경우 JDK 1.0부터 포함된 것으로 '가변적'이라는 큰 단점을 가지고 있음을 확인했습니다.
또한 java.util 패키지 하위에 위치해있는데, 오라클에서 "Contains the collections framework, legacy collection classes, event model, date and time facilities" 라고 설명하는, 시간을 다루는 '레거시'한 방법임을 확인했습니다.

이후 JDK 1.8에서 날짜와 시간을 전용으로 다루는 java.time 패키지가 등장했고, LocalDateTime이 여기에 속하게 됩니다.
가장 큰 변화로는 '불변'이라는 것이고, 이외에 날짜/시간 계산을 위한 편의 메서드 추가 등이 되었음을 확인했습니다.

The main API for dates, times, instants, and durations.
...
All the classes are immutable and thread-safe.

결과적으로, 다음처럼 LocalDateTime을 사용하도록 변경했습니다!

java-janggi/src/main/java/dto/GameSummary.java

Lines 5 to 6 in 43dcc80

public record GameSummary(long id, LocalDateTime startedAt, String currentTurn) {

}

java-janggi/src/main/java/repository/JanggiGameRepository.java

Lines 127 to 142 in 43dcc80

public List<GameSummary> findAll(Connection conn) {

String sql = ""

+ "SELECT game_id, created_at, current_turn "

+ "FROM game ";

List<GameSummary> gameSummaries = new ArrayList<>();

try (PreparedStatement pstmt = conn.prepareStatement(sql)) {

try (ResultSet rs = pstmt.executeQuery()) {

while (rs.next()) {

gameSummaries.add(new GameSummary(

rs.getLong("game_id"),

rs.getObject("created_at", LocalDateTime.class),

rs.getString("current_turn")

));

}

}

HyoYoonNam · 2026-04-12T14:33:13Z

src/main/java/domain/board/Intersection.java

+    public boolean isPalaceCenter() {
+        return (row == 2 || row == 9)
+                && file == 5;
+    }
+
+    public boolean isPalaceCorner() {
+        return (row == 1 || row == 3 || row == 8 || row == 10)
+                && (file == 4 || file == 6);
+    }


(PalaceMovement와 Intersection에서 사용되던 매직넘버 관련)

궁성과 관련한 판단 및 매직넘버를 Palace 메서드 및 상수로 이동했습니다!

java-janggi/src/main/java/domain/board/Palace.java

Lines 6 to 84 in 9328d22

public final class Palace {

private static final int MIN_FILE = 4;

private static final int CENTER_FILE = 5;

private static final int MAX_FILE = 6;

private static final int HAN_MIN_ROW = 1;

private static final int HAN_CENTER_ROW = 2;

private static final int HAN_MAX_ROW = 3;

private static final int CHO_MIN_ROW = 8;

private static final int CHO_CENTER_ROW = 9;

private static final int CHO_MAX_ROW = 10;

private Palace() {

}

public static boolean contains(Intersection pos, Side side) {

final int row = pos.row();

final int file = pos.file();

boolean inPalaceFile = file >= MIN_FILE && file <= MAX_FILE;

if (side == Side.HAN) { // 만약 Side enum의 이름이 다르다면 맞게 수정해주세요

return inPalaceFile && row >= HAN_MIN_ROW && row <= HAN_MAX_ROW;

}

return inPalaceFile && row >= CHO_MIN_ROW && row <= CHO_MAX_ROW;

}

public static boolean isCenter(Intersection pos) {

final int row = pos.row();

final int file = pos.file();

return file == CENTER_FILE &&

(row == HAN_CENTER_ROW || row == CHO_CENTER_ROW);

}

public static boolean isCorner(Intersection pos) {

final int row = pos.row();

final int file = pos.file();

boolean isHanCorner = Math.abs(row - HAN_CENTER_ROW) == 1;

boolean isChoCorner = Math.abs(row - CHO_CENTER_ROW) == 1;

boolean isFileCorner = Math.abs(file - CENTER_FILE) == 1;

return (isHanCorner || isChoCorner) && isFileCorner;

}

public static Intersection getCenterOf(Intersection cornerPos) {

if (cornerPos.row() <= HAN_MAX_ROW) {

return new Intersection(HAN_CENTER_ROW, CENTER_FILE);

}

return new Intersection(CHO_CENTER_ROW, CENTER_FILE);

}

public static List<Intersection> getCornersOf(Intersection center) {

if (!isCenter(center)) {

throw new IllegalStateException("중앙 좌표가 아닙니다.");

}

final int r = center.row();

final int f = center.file();

return List.of(

new Intersection(r - 1, f - 1), new Intersection(r - 1, f + 1),

new Intersection(r + 1, f - 1), new Intersection(r + 1, f + 1)

);

}

public static Intersection getOppositeCornerOf(Intersection corner) {

if (!isCorner(corner)) {

throw new IllegalArgumentException("코너 좌표가 아닙니다.");

}

Intersection center = getCenterOf(corner);

int deltaRow = center.row() - corner.row();

int deltaFile = center.file() - corner.file();

return new Intersection(center.row() + deltaRow, center.file() + deltaFile);

}

}

HyoYoonNam · 2026-04-12T14:35:19Z

src/main/java/support/DataAccessException.java

기존에는 데이터 접근 계층과 관련된 예외를 모두 IllegalStateException으로 감싸서 던졌는데, 리팩터링 과정에서 구분이 가능한 커스텀 예외로 바꿔야 할 필요가 있다고 판단했습니다.

HyoYoonNam · 2026-04-12T14:45:10Z

src/main/java/support/TransactionTemplate.java

+    public <T> T execute(TransactionCallback<T> action) {
+        Connection conn = null;
+        try {
+            conn = dataSource.getConnection();
+            conn.setAutoCommit(false);
+
+            T doingResult = action.doInTransaction(conn);
+
+            conn.commit();
+
+            return doingResult;
+        } catch (SQLException e) {
+            rollback(conn);
+            throw new DataAccessException(e);
+        } catch (RuntimeException e) {
+            rollback(conn);
+            throw e;
+        } finally {
+            close(conn);
+        }
+    }
+
+    private void rollback(Connection conn) {
+        if (conn == null) {
+            return;
+        }
+
+        try {
+            conn.rollback();
+        } catch (SQLException e) {
+            LOGGER.log(Level.SEVERE, LOG_MESSAGE_FORMAT.formatted("롤백"), e);
+        }
+    }
+
+    private void close(Connection conn) {
+        if (conn == null) {
+            return;
+        }
+
+        try {
+            conn.close();
+        } catch (SQLException e) {
+            LOGGER.log(Level.SEVERE, LOG_MESSAGE_FORMAT.formatted("Connection 반납"), e);
+        }
+    }


결론: 롤백과 자원 반납 과정에서 발생하는 예외는 상위 계층으로 전달되지 않아도 된다고 판단하여 로그만 찍도록 했습니다.

근거 및 상황 설명:

(line 26) action.doInTransaction()에서 CRUD를 실행했을 때 예외가 발생

(line 32) 1번의 예외를 catch하고, rollback()을 호출

(line 48) 롤백 과정에서 또 예외가 발생

(line 32) 3번의 예외가 다시 던져지지만, 이를 catch해줄 곳이 더 이상 없음

따라서 line 33은 실행되지 않기 때문에 1번의 예외가 상위 계층으로 던져지지 않고, 롤백으로 인한 3번 예외만 던져짐.

이때 '상위 계층으로 전달되어야 할 중요한 예외'는 근본적인 1번 예외인데, 이를 수습하던 과정의 롤백 예외가 1번 대신 던져지는 문제가 있음을 확인했습니다.

롤백에 대한 예외가 던져져도 근본적인 CRUD 예외까지 같이 담아서 던지려면 다음처럼 중복적인 catch block이 필요한데, 너무 과한 대응이라고 생각하여 롤백 예외는 단순히 로그만 찍음으로써 CRUD 예외가 덮어지지 않고 상위 계층으로 던져질 수 있도록 했습니다.

public <T> T execute(TransactionCallback<T> action) { Connection conn = null; try { conn = dataSource.getConnection(); conn.setAutoCommit(false); T doingResult = action.doInTransaction(conn); conn.commit(); return doingResult; } catch (SQLException e) { // 롤백에서 나온 예외를 다시 try-catch로 잡아서 처리 try { rollback(conn); } catch (DataAccessException exception) { throw new DataAccessException(e); } throw new DataAccessException(e); } catch (RuntimeException e) { rollback(conn); throw e; } finally { close(conn); } }

HyoYoonNam · 2026-04-12T14:46:20Z

src/test/java/laboratory/time/CompareJavaPackageUtilAndTime.java

Date와 LocalDateTime 차이를 알아보면서 사용한 실험 코드입니다.

HyoYoonNam · 2026-04-12T14:47:04Z

src/test/java/laboratory/exception/LostExceptionProblemTest.java

https://github.com/woowacourse/java-janggi/pull/325/changes#r3069609600

에서 다룬 '예외 가려짐' 문제를 테스트한 실험 코드입니다.

HyoYoonNam added 30 commits April 4, 2026 17:06

refactor: 빈 기물은 매번 생성하지 않는다

da4b0e8

docs: 구현할 기능 목록 추가

4cc6cbc

refactor: '지점(장소)'을 가리키는 변수명 용어 변경

dbc111f

기존에는 'Point'를 사용했다. 하지만 새로운 요구사항에서의 '점수'가 장기 도메인에서 'Point'로 사용되기 때문에 혼란을 방지하고자 'Position'으로 변경한다.

docs: 진영별 보너스 점수 요구사항 추가

9720727

feat: 진영별 기물 점수 계산 구현

4dba09b

docs: 실행 예시에 승자 및 점수 출력 추가

9f53d46

feat: 승자 출력 시 점수도 함께 출력

e9d93af

docs: 추가나 완료 체크를 누락한 기능 요구사항 반영

10e08b1

refactor: Board의 getter를 제거

382bad7

feat: 궁과 사는 궁성 밖으로 이동할 수 없다

adb0058

feat: 궁과 사의 궁성 내 대각선 이동 구현

86a5ca5

feat: 졸/병의 궁성 내 대각선 이동 구현

5f80c18

refactor: 테스트 코드에서의 출력 제거

8976b5e

refactor: PalaceMovement.toPalaceCenter private 변경

09ffe4d

해당 메서드는 같은 interface의 default method인 palaceDiagonalPaths에서만 호출되기 때문에 불필요한 외부 노출을 없애고자 private으로 변경한다.

refactor: 미사용 import 제거

9a4f79c

feat: 차와 포의 궁성 내 대각선 이동 구현

c243bde

docs: 누락한 체크리스트 반영

eefd8cc

docs: 용어사전 업데이트

15261ba

docs: DB 요구사항 명세서 추가

05f78a3

build: add sqlite-jdbcdriver dependency

46cd87b

docs: 게임 종료 요구사항 추가

634ae5b

docs: 게임 종료 요구사항 추가

2d7f803

feat: 사용자 입력에 의한 종료 구현

80ec9a5

docs: 데이터베이스 요구사항 추가

e1599ce

temp: DB 저장 및 로드 구현. 컨트롤러 조립 필요.

d41253f

feat: DB를 활용한 게임 저장 및 불러오기 구현

ccd43f0

build: add mockito dependency

feaf682

Choi-JJunho requested changes Apr 8, 2026

View reviewed changes

HyoYoonNam added 28 commits April 9, 2026 16:47

refactor: 게임 생성 날짜를 LocalDateTime으로 관리

959c78b

refactor: Transaction 관리 책임 Service로 변경

2ac518a

refactor: TransactionTemplate으로 관리 중복 해결

14107c5

기존에는 트랜잭션 관리를 서비스 계층에서 수행했다. 관리를 위한 중복 코드를 템플릿화한다. 또한 덕분에 서비스 계층에서는 DB 종속적인 예외를 다루지 않아도 된다.

refactor: RuntimeException 대신 커스텀 예외로 변환

fc08aa8

refactor: 패키지 구조 변경

eab92a3

refactor: 진영 보너스 점수 책임을 별도 객체로 분리

20c8105

refactor: 출력 개행 추가

3de9802

refactor: Repository에서 SQLException을 바꿔서 던진다

fc06a76

refactor: 불필요한 주석 제거

0ed3a7c

refactor: 테스트 독립성을 위한 clear 1회만 호출

0f06c98

refactor: MoveCommand가 null 대신 Optional 리턴

73d0bc4

fix: 다른 진영 기물 선택시 게임 종료문제 해결

2dac0e5

refactor: 컨트롤러의 과도한 분기 추상화를 제거

51993df

fix: 서비스 계층 테스트의 단위 독립성 보장

4df628e

refactor: 트랜잭션 테스트를 템플릿으로 이동

857ddff

refactor: 기물 이동에 대한 영속성 즉시 반영

c46d916

refactor: 궁성 관련 매직넘버를 별도 클래스와 상수로 분리

09acb09

refactor: repository 계층과 도메인을 mapper로 연결

c86894b

refactor: 격자점 파싱 매직넘버 상수화

be2a904

refactor: GameRule 보너스 포인트 상수화

b2ef91b

refactor: 미사용 import 제거

db2a006

refactor: 좌표 관련 매직넘버 상수화

eb25725

refactor: DB 초기화 책임을 별도 객체로 분리

1410cab

refactor: LoadCommand의 새 게임 값 상수화

43dcc80

refactor: IntegrationTestSupport 패키지 변경

9328d22

HyoYoonNam commented Apr 12, 2026

View reviewed changes

	public long save(JanggiGame janggiGame) {
	String sql = "INSERT INTO game (current_turn) VALUES (?)";

	try (
	Connection conn = dataSource.getConnection();
	PreparedStatement pstmt = conn.prepareStatement(sql, Statement.RETURN_GENERATED_KEYS)
	) {
	pstmt.setString(1, janggiGame.currentTurn().name());
	pstmt.executeUpdate();
	// TODO: ResultSet 자원도 닫아야 됨을 인지해서 일단 중첩 try-with-resources로 대응했지만, 중첩없이 해결 가능한 지 고민 필요.
	try (ResultSet rs = pstmt.getGeneratedKeys()) {
	if (rs.next()) {
	long gameId = rs.getLong(1);
	insertPieces(conn, gameId, janggiGame);

	return gameId;
	}
	}

	throw new SQLException("게임 생성에 실패했습니다.");
	} catch (SQLException e) {
	throw new IllegalStateException(e);
	}
	}

	public Piece placedAt(Intersection intersection) {
	return alivePieces.getOrDefault(intersection, Piece.EMPTY);
	}

	public final class Piece {

	public static final Piece EMPTY = new Piece(PieceType.EMPTY, Side.NONE);

	public final class Palace {

	private static final int MIN_FILE = 4;
	private static final int CENTER_FILE = 5;
	private static final int MAX_FILE = 6;

	private static final int HAN_MIN_ROW = 1;
	private static final int HAN_CENTER_ROW = 2;
	private static final int HAN_MAX_ROW = 3;

	private static final int CHO_MIN_ROW = 8;
	private static final int CHO_CENTER_ROW = 9;
	private static final int CHO_MAX_ROW = 10;

	private Palace() {
	}

	public static boolean contains(Intersection pos, Side side) {
	final int row = pos.row();
	final int file = pos.file();

	boolean inPalaceFile = file >= MIN_FILE && file <= MAX_FILE;

	if (side == Side.HAN) { // 만약 Side enum의 이름이 다르다면 맞게 수정해주세요
	return inPalaceFile && row >= HAN_MIN_ROW && row <= HAN_MAX_ROW;
	}
	return inPalaceFile && row >= CHO_MIN_ROW && row <= CHO_MAX_ROW;
	}

	public static boolean isCenter(Intersection pos) {
	final int row = pos.row();
	final int file = pos.file();

	return file == CENTER_FILE &&
	(row == HAN_CENTER_ROW \|\| row == CHO_CENTER_ROW);
	}

	public static boolean isCorner(Intersection pos) {
	final int row = pos.row();
	final int file = pos.file();

	boolean isHanCorner = Math.abs(row - HAN_CENTER_ROW) == 1;
	boolean isChoCorner = Math.abs(row - CHO_CENTER_ROW) == 1;
	boolean isFileCorner = Math.abs(file - CENTER_FILE) == 1;

	return (isHanCorner \|\| isChoCorner) && isFileCorner;
	}

	public static Intersection getCenterOf(Intersection cornerPos) {
	if (cornerPos.row() <= HAN_MAX_ROW) {
	return new Intersection(HAN_CENTER_ROW, CENTER_FILE);
	}
	return new Intersection(CHO_CENTER_ROW, CENTER_FILE);
	}

	public static List<Intersection> getCornersOf(Intersection center) {
	if (!isCenter(center)) {
	throw new IllegalStateException("중앙 좌표가 아닙니다.");
	}
	final int r = center.row();
	final int f = center.file();
	return List.of(
	new Intersection(r - 1, f - 1), new Intersection(r - 1, f + 1),
	new Intersection(r + 1, f - 1), new Intersection(r + 1, f + 1)
	);
	}

	public static Intersection getOppositeCornerOf(Intersection corner) {
	if (!isCorner(corner)) {
	throw new IllegalArgumentException("코너 좌표가 아닙니다.");
	}

	Intersection center = getCenterOf(corner);
	int deltaRow = center.row() - corner.row();
	int deltaFile = center.file() - corner.file();

	return new Intersection(center.row() + deltaRow, center.file() + deltaFile);
	}
	}

	private GameWrapper processTurn(MoveCommand moveCommand, GameWrapper gameWrapper) {
	JanggiGame janggiGame = gameWrapper.game();
	Intersection startPosition = moveCommand.selectedToMove()
	.orElseThrow(() -> new IllegalArgumentException("이동할 좌표(x,y) 또는 종료(exit)를 입력해주세요."));

	public <T> T execute(TransactionCallback<T> action) {
	Connection conn = null;
	try {
	conn = dataSource.getConnection();
	conn.setAutoCommit(false);

	T doingResult = action.doInTransaction(conn);

	conn.commit();

	return doingResult;
	} catch (SQLException e) {
	rollback(conn);
	throw new DataAccessException(e);
	} catch (RuntimeException e) {
	rollback(conn);
	throw e;
	} finally {
	close(conn);
	}
	}

	public GameWrapper createGame(Board board) {
	return transactionTemplate.execute(conn -> {
	JanggiGame janggiGame = JanggiGame.create(board);
	long generatedKey = repository.save(conn, janggiGame);
	return new GameWrapper(generatedKey, janggiGame);
	});
	}


		import java.util.Date;

		public record GameSummary(long id, Date startedAt, String currentTurn) {

	public record GameSummary(long id, LocalDateTime startedAt, String currentTurn) {
	}

	public List<GameSummary> findAll(Connection conn) {
	String sql = ""
	+ "SELECT game_id, created_at, current_turn "
	+ "FROM game ";

	List<GameSummary> gameSummaries = new ArrayList<>();
	try (PreparedStatement pstmt = conn.prepareStatement(sql)) {
	try (ResultSet rs = pstmt.executeQuery()) {
	while (rs.next()) {
	gameSummaries.add(new GameSummary(
	rs.getLong("game_id"),
	rs.getObject("created_at", LocalDateTime.class),
	rs.getString("current_turn")
	));
	}
	}

	private void dispatchLoadCommand(LoadCommand loadCommand, List<Long> selectableNumbers) {
	long gameNumberToLoad = loadCommand.gameNumberToLoad();
	if (!selectableNumbers.contains(gameNumberToLoad)) {
	throw new IllegalArgumentException(gameNumberToLoad + "는 유효하지 않는 번호입니다.");
	}

	if (loadCommand.isNewGame()) {
	newGameFlow();
	return;
	}

	if (loadCommand.isLoadGame()) {
	continueGame(gameNumberToLoad);
	return;
	}

	final class PieceMapper {

	private PieceMapper() {
	}

	static PieceEntity toEntity(long gameId, Intersection position, Piece piece) {
	return new PieceEntity(
	gameId,
	position.row(),
	position.file(),
	piece.getType().name(),
	piece.getSide().name()
	);
	}

	static Piece toPiece(String typeName, String sideName) {
	return Piece.of(
	PieceType.from(typeName),
	Side.valueOf(sideName)
	);
	}
	}

	for (Entry<Intersection, Piece> entry : map.entrySet()) {
	PieceEntity entity = PieceMapper.toEntity(gameId, entry.getKey(), entry.getValue());

	pstmt.setInt(2, entity.row());
	pstmt.setInt(3, entity.file());
	pstmt.setString(4, entity.type());
	pstmt.setString(5, entity.side());
	pstmt.addBatch();
	}


		public final class MemoryDBConnectionUtil {

		private static final String JDBC_SQLITE_URL = "jdbc:sqlite::memory:?cache=shared";

Conversation

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

Uh oh!

체크 리스트

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

Uh oh!

Choi-JJunho 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!

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!

HyoYoonNam 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

HyoYoonNam commented Apr 8, 2026 •

edited

Loading