From 69f66e754252540b581adeeb3d5e7a9d54ad720a Mon Sep 17 00:00:00 2001 From: loveb Date: Thu, 19 May 2022 15:26:16 +0200 Subject: [PATCH] =?UTF-8?q?Sm=C3=A5=20=C3=A4ndringar?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/schack/Board.java | 14 +++++++------- src/schack/King.java | 17 ++++++++--------- src/schack/Pawn.java | 2 -- src/schack/Rook.java | 1 - 4 files changed, 15 insertions(+), 19 deletions(-) diff --git a/src/schack/Board.java b/src/schack/Board.java index 14a0a03..5cf3ac2 100644 --- a/src/schack/Board.java +++ b/src/schack/Board.java @@ -104,7 +104,7 @@ public class Board extends JPanel implements MouseListener { // Ifall vi har tryckt på en pjäs och sedan ska gå dit if (validMovesToDraw.contains(clickedCoordinate)) { - final Piece selectedPiece = pieces[previouslyClickedPoint.x][previouslyClickedPoint.y]; + Piece selectedPiece = pieces[previouslyClickedPoint.x][previouslyClickedPoint.y]; if (selectedPiece == null) { validMovesToDraw.clear(); return; @@ -141,7 +141,7 @@ public class Board extends JPanel implements MouseListener { // Om vi inte redan har valt en pjäs klickar vi på en pjäs if (!validMovesToDraw.contains(clickedCoordinate)) { - final Piece selectedPiece = pieces[mouseCoordinateX][mouseCoordinateY]; + Piece selectedPiece = pieces[clickedCoordinate.x][clickedCoordinate.y]; if (selectedPiece != null && selectedPiece.isWhite() == whitesTurn) { validMovesToDraw.addAll(selectedPiece.validMoves(pieces, true)); @@ -161,9 +161,9 @@ public class Board extends JPanel implements MouseListener { * @return SCHACK, SCHACKMATT, PATT, NORMAL */ private SchackState getSchackState() { - final ArrayList allValidMoves = getMoves(whitesTurn); - final ArrayList opposingAttacks = getAttacks(!whitesTurn); - final boolean weCanMove = !allValidMoves.isEmpty(); + ArrayList allValidMoves = getMoves(whitesTurn); + ArrayList opposingAttacks = getAttacks(!whitesTurn); + boolean weCanMove = !allValidMoves.isEmpty(); boolean inSchack = false; for (Point attack : opposingAttacks) { @@ -189,7 +189,7 @@ public class Board extends JPanel implements MouseListener { } private ArrayList getMoves(boolean whiteMovesAreWanted) { - final ArrayList allValidMoves = new ArrayList<>(); + ArrayList allValidMoves = new ArrayList<>(); for (Piece[] pieceArr : pieces) { for (Piece piece : pieceArr) { if (piece == null || whiteMovesAreWanted != piece.isWhite()) { @@ -202,7 +202,7 @@ public class Board extends JPanel implements MouseListener { } public ArrayList getAttacks(boolean whiteAttacksAreWanted) { - final ArrayList attacks = new ArrayList(); + ArrayList attacks = new ArrayList(); for (Piece[] pieceArr : pieces) { for (Piece piece : pieceArr) { if (piece == null || whiteAttacksAreWanted != piece.isWhite()) { diff --git a/src/schack/King.java b/src/schack/King.java index 4ec650c..7cdad5d 100644 --- a/src/schack/King.java +++ b/src/schack/King.java @@ -24,22 +24,22 @@ public final class King extends Piece { } boolean[] somethingBetweenOrSchackOnTheWay = new boolean[2]; // Vänster, höger - int left_modifier = -1, right_modifier = 1; - for (int modifier : new int[]{left_modifier, right_modifier}) { + int leftModifier = -1, rightModifier = 1; + for (int modifier : new int[]{leftModifier, rightModifier}) { for (int loopX = this.position.x + modifier; loopX > 0 && loopX < 7; loopX += modifier) { if (pieces[loopX][this.position.y] != null || isInSchack(pieces, new Point(loopX, this.position.y))) { - somethingBetweenOrSchackOnTheWay[(modifier == left_modifier) ? 0 : 1] = true; + somethingBetweenOrSchackOnTheWay[(modifier == leftModifier) ? 0 : 1] = true; break; } } } - left_modifier = 0; - right_modifier = 1; - for (int direction : new int[]{left_modifier, right_modifier}) { + leftModifier = 0; + rightModifier = 1; + for (int direction : new int[]{leftModifier, rightModifier}) { if (!somethingBetweenOrSchackOnTheWay[direction]) { - Piece possibleRook = pieces[direction == left_modifier ? 0 : 7][this.position.y]; + Piece possibleRook = pieces[direction == leftModifier ? 0 : 7][this.position.y]; if (possibleRook != null && !possibleRook.isMoved()) { - possibleCastling.add(new Point(direction == left_modifier ? 2 : 6, this.position.y)); + possibleCastling.add(new Point(direction == leftModifier ? 2 : 6, this.position.y)); } } } @@ -75,7 +75,6 @@ public final class King extends Piece { } else { super.move(pieces, toMove); } - } @Override diff --git a/src/schack/Pawn.java b/src/schack/Pawn.java index 56b2e7b..c95d823 100644 --- a/src/schack/Pawn.java +++ b/src/schack/Pawn.java @@ -3,8 +3,6 @@ package schack; import java.awt.Point; import java.io.IOException; import java.util.ArrayList; -import javax.swing.JMenuItem; -import javax.swing.JPopupMenu; public class Pawn extends Piece { diff --git a/src/schack/Rook.java b/src/schack/Rook.java index f348810..d4b5f5d 100644 --- a/src/schack/Rook.java +++ b/src/schack/Rook.java @@ -17,7 +17,6 @@ public class Rook extends LongWalkers { pieces, allowedToRecurse ); - } }