Skip to content
Open
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
116 changes: 116 additions & 0 deletions src/main/java/chess/dao/ChessGameDao.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,116 @@
package chess.dao;

import chess.board.Board;
import chess.utils.DatabaseUtil;
import chess.piece.Team;
import java.sql.Connection;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;

public class ChessGameDao {
public void createNewRoom() {
String query = "INSERT INTO game (boardState, turn, status) VALUES (?, ?, ?)";
Board board = new Board();
Comment on lines +12 to +14
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이렇게 데이터베이스와 직접 소통하는 기능에 대한 테스트는 왜 없나요?
어떻게 테스트할 수 있을까요?


try (Connection conn = DatabaseUtil.getConnection(); PreparedStatement pstmt = conn.prepareStatement(query)) {
pstmt.setString(1, board.convertBoardStateToString());
pstmt.setBoolean(2, true);
pstmt.setString(3, "ready");
pstmt.executeUpdate();
} catch (SQLException e) {
e.printStackTrace();
}
}

public int selectRoom(int gameId) {
String query = "SELECT status FROM game WHERE gameId = ?";
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

현재 보드 상태를 하나의 문자열로 쭉 이어서 저장하는 방법이 너무 참신하네요ㅋㅋㅋ
그런데 하나의 컬럼이 체스판 전체 데이터를 저장하고 있는 건 데이터베이스의 제 1 정규화 규칙을 어긴 것 같기도 한데 어떻게 생각하나요?

이 방식에는 다른 방식과 비교했을 때 어떤 장단점이 있을까요?


try (Connection conn = DatabaseUtil.getConnection();
PreparedStatement pstmt = conn.prepareStatement(query)) {
pstmt.setInt(1, gameId); // gameId를 쿼리에 설정
try (ResultSet rs = pstmt.executeQuery()) {
if (rs.next()) {
return gameId;
}
}
} catch (SQLException e) {
e.printStackTrace();
}
throw new IllegalStateException("존재하지 않는 방 번호입니다.");
}

public void saveGame(int gameId, String boardState, boolean turn, String status) {
String query = "INSERT INTO game (gameId, boardState, turn, status) VALUES (?, ?, ?, ?) " + "ON DUPLICATE KEY UPDATE boardState = VALUES(boardState), turn = VALUES(turn), status = VALUES(status)";

Comment on lines +44 to +45
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ON DUPLICATE KEY UPDATE는 몇몇 DB에서만 한정적으로 사용 가능한 SQL 입니다. (ANSI 표준이 아닙니다)
이런 경우 DB의 종류가 바뀌면 서버가 정상작동하지 않을 수 있는데, 이를 어떻게 수정해줄 수 있을까요?

try (Connection conn = DatabaseUtil.getConnection(); PreparedStatement pstmt = conn.prepareStatement(query)) {
pstmt.setInt(1, gameId);
pstmt.setString(2, boardState);
pstmt.setBoolean(3, turn);
pstmt.setString(4, status);
pstmt.executeUpdate();
Comment on lines +46 to +51
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

preparedStatement 사용 👍👍
그냥 statement와 어떤 차이가 있나요?

} catch (SQLException e) {
e.printStackTrace();
}
}

public void updateCurrentBoard(int gameId, Board board) {
String query = "SELECT boardState, status FROM game WHERE gameId = ?";
try (Connection conn = DatabaseUtil.getConnection(); PreparedStatement pstmt = conn.prepareStatement(query)) {
pstmt.setInt(1, gameId);
try (ResultSet rs = pstmt.executeQuery()) {
if (rs.next()) {
String boardState = rs.getString("boardState");
board.loadBoardState(boardState);
}
}
} catch (SQLException e) {
e.printStackTrace();
}
}

public Team getCurrentTurn(int gameId, Team turn) {
String query = "SELECT turn FROM game WHERE gameID = ?";
try (Connection conn = DatabaseUtil.getConnection();
PreparedStatement pstmt = conn.prepareStatement(query)) {
pstmt.setInt(1, gameId);
try (ResultSet rs = pstmt.executeQuery()) {
if (rs.next()) {
return turn.getTurnByBinary(rs.getInt("turn"));
}
}
} catch (SQLException e) {
e.printStackTrace();
}
return turn;
}

public String getCurrentStatus(int gameId) {
String query = "SELECT status FROM game WHERE gameId = ?";

try (Connection conn = DatabaseUtil.getConnection();
PreparedStatement pstmt = conn.prepareStatement(query)) {
pstmt.setInt(1, gameId);
try (ResultSet rs = pstmt.executeQuery()) {
if (rs.next()) {
return rs.getString("status");
}
}
} catch (SQLException e) {
e.printStackTrace();
}
throw new IllegalStateException("존재하지 않는 방입니다.");
}

public void changeStatus(int gameId) {
String query = "UPDATE game SET status = ? WHERE gameId = ?";
try (Connection conn = DatabaseUtil.getConnection();
PreparedStatement pstmt = conn.prepareStatement(query)) {
pstmt.setString(1, "end");
pstmt.setInt(2, gameId);
pstmt.executeUpdate();
} catch (SQLException e) {
e.printStackTrace();
}
}
}
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
package chess;
package chess.game;

import chess.board.Board;
import chess.dao.ChessGameDao;
import chess.piece.Piece;
import chess.ui.InputView;
import chess.ui.OutputView;
Expand All @@ -10,29 +11,56 @@
public class ChessGameManager {
private static final String START_COMMAND = "start";
private static final String STATUS_COMMAND = "status";
private static final String END_COMMAND = "end";
private static final String CREATE_NEW_ROOM_COMMAND = "create";
private Team turn = Team.WHITE;
private boolean isGameOver = false;
private ChessGameDao chessGameDao;
private int gameId;

public ChessGameManager() {
this.chessGameDao = new ChessGameDao();
}

public void startNewGame() {
OutputView.printStartMessage();
if (InputView.inputCommand().equals(START_COMMAND)) {
String command = InputView.inputCommand();
if (command.equals(START_COMMAND)) {
Board board = new Board();
selectRoom();
loadGame(board);
OutputView.printBoard(board);
proceedGame(board);
}
if (command.equals(CREATE_NEW_ROOM_COMMAND)) {
createNewRoom();
startNewGame();
}
}

public void createNewRoom() {
OutputView.printCreateRoomMessage();
chessGameDao.createNewRoom();
}

public void proceedGame(Board board) {
if (isGameOver) {
changeStatus(gameId);
return;
}
String command = InputView.inputCommand();
if (command.equals(END_COMMAND)) {
saveGame(gameId, board);
return;
}
if (!command.equals(STATUS_COMMAND)) {
moveProcess(board, command);
saveGame(gameId, board);
proceedGame(board);
}
if (command.equals(STATUS_COMMAND)) {
OutputView.printEndGameByStatusMessage(board);
changeStatus(gameId);
}
}

Expand All @@ -57,6 +85,30 @@ public void moveProcess(Board board, String command) {
turn = turn.changeTurn();
}

public void selectRoom() {
OutputView.printSelectRoomMessage();
int roomNumber = Integer.parseInt(InputView.inputCommand());
if (chessGameDao.getCurrentStatus(roomNumber).equals(END_COMMAND)) {
throw new IllegalStateException("종료된 게임입니다.");
}
OutputView.printEnterRoomMessage(chessGameDao.selectRoom(roomNumber));
gameId = roomNumber;
}

public void saveGame(int gameId, Board board) {
String status = "playing";
chessGameDao.saveGame(gameId, board.convertBoardStateToString(), getTurnState(), status);
}

public void loadGame(Board board) {
chessGameDao.updateCurrentBoard(gameId, board);
turn = chessGameDao.getCurrentTurn(gameId, turn);
}

public void changeStatus(int gameId) {
chessGameDao.changeStatus(gameId);
}

public Position searchPosition(String location) {
int file = location.charAt(0) - 'a' + 1;
int rank = location.charAt(1) - '0';
Expand All @@ -69,4 +121,8 @@ public void verifyTurn(Piece piece) {
throw new IllegalStateException("상대방 기물을 이동시킬 수 없습니다.");
}
}

private boolean getTurnState() {
return turn.isSameTeam(Team.WHITE);
}
}
5 changes: 5 additions & 0 deletions src/main/java/chess/piece/Team.java
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,9 @@ public Team changeTurn() {
}
return BLACK;
}

public Team getTurnByBinary(int n) {
if (n == 0) return Team.BLACK;
return Team.WHITE;
}
Comment on lines +19 to +22
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. google style guide에서는 한줄의 문법이어도 중괄호를 사용하기를 권합니다.
  2. n값이 -7, 6 처럼 이상한 값일때 검증이 필요하지 않을까요?

}