Skip to content

Commit

Permalink
refactor: 다수의 검증 메소드 하나로 추출
Browse files Browse the repository at this point in the history
  • Loading branch information
seongbeenkim committed Sep 3, 2021
1 parent d6edf85 commit c85d73f
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions src/main/java/chess/domain/board/Board.java
Original file line number Diff line number Diff line change
Expand Up @@ -23,13 +23,17 @@ public void move(final MoveOptions moveOptions, final boolean isWhiteTurn) {
Position source = moveOptions.getSource();
Position target = moveOptions.getTarget();

validate(player, enemy, source, target);

enemy.removePieceOn(target);
movePiece(player, source, target);
}

private void validate(final Player player, final Player enemy, final Position source, final Position target) {
validateSourceOwner(enemy, source);
validateSamePosition(source, target);
validateTarget(player, target);
validateKingMovable(player, enemy, source, target);

enemy.removePieceOn(target);
movePiece(player, source, target);
}

private Player currentPlayer(final boolean isWhiteTurn) {
Expand Down

0 comments on commit c85d73f

Please sign in to comment.