From b32a4f241450282288b1191cc9e36b96673eb856 Mon Sep 17 00:00:00 2001 From: kske Date: Sat, 21 Dec 2019 21:42:36 +0100 Subject: [PATCH] Fixed some Javadoc and formatting --- src/dev/kske/chess/board/Board.java | 14 +++++++------- src/dev/kske/chess/ui/GamePane.java | 15 +++++++-------- 2 files changed, 14 insertions(+), 15 deletions(-) diff --git a/src/dev/kske/chess/board/Board.java b/src/dev/kske/chess/board/Board.java index 114f32e..28d6cca 100644 --- a/src/dev/kske/chess/board/Board.java +++ b/src/dev/kske/chess/board/Board.java @@ -30,11 +30,13 @@ public class Board { /** * Creates a copy of another {@link Board} instance.
- * The created object is a deep copy, but does not contain any move history - * apart from the current {@link MoveNode}. + * The created object is a deep copy, and can optionally contain the move + * history of the Board to copy. * - * @param other The {@link Board} instance to copy - * @param copyVariations TODO + * @param other The Board instance to copy + * @param copyVariations if set to {@code true}, the {@link Log} object of the + * other Board instance is copied with its entire move + * history */ public Board(Board other, boolean copyVariations) { for (int i = 0; i < 8; i++) @@ -99,9 +101,7 @@ public class Board { * * @param sanMove The move to execute in SAN (Standard Algebraic Notation) */ - public void move(String sanMove) { - move(Move.fromSAN(sanMove, this)); - } + public void move(String sanMove) { move(Move.fromSAN(sanMove, this)); } /** * Reverts the last move and removes it from the log. diff --git a/src/dev/kske/chess/ui/GamePane.java b/src/dev/kske/chess/ui/GamePane.java index 7f891d2..b8bf02d 100644 --- a/src/dev/kske/chess/ui/GamePane.java +++ b/src/dev/kske/chess/ui/GamePane.java @@ -31,7 +31,7 @@ import dev.kske.chess.game.NaturalPlayer; * Project: Chess
* File: GamePane.java
* Created: 23.08.2019
- * + * * @since Chess v0.4-alpha * @author Kai S. K. Engelbart */ @@ -44,9 +44,7 @@ public class GamePane extends JComponent { private Game game; private Color activeColor; private JPanel moveSelectionPanel; - private JButton btnNext; - private JButton btnFirst; - private JButton btnLast; + private JButton btnFirst, btnPrevious, btnNext, btnLast; public GamePane() { activeColor = Color.WHITE; @@ -96,9 +94,9 @@ public class GamePane extends JComponent { btnFirst.setEnabled(false); moveSelectionPanel.add(btnFirst); - JButton btnPreviousMove = new JButton("Previous"); - btnPreviousMove.setEnabled(false); - moveSelectionPanel.add(btnPreviousMove); + btnPrevious = new JButton("Previous"); + btnPrevious.setEnabled(false); + moveSelectionPanel.add(btnPrevious); btnNext = new JButton("Next"); btnNext.setEnabled(false); @@ -107,6 +105,7 @@ public class GamePane extends JComponent { btnLast = new JButton("Last"); btnLast.setEnabled(false); moveSelectionPanel.add(btnLast); + boardPane = new BoardPane(); GridBagConstraints gbc_boardPane = new GridBagConstraints(); gbc_boardPane.fill = GridBagConstraints.BOTH; @@ -189,7 +188,7 @@ public class GamePane extends JComponent { /** * Assigns a new {@link Game} instance to this game pane. If exactly one of the * players is natural, color swapping functionality is enabled. - * + * * @param game The {@link Game} to assign to this game pane. */ public void setGame(Game game) {