From ed07eecccbef90c019dafffc2f7248c415fb18b4 Mon Sep 17 00:00:00 2001 From: loveb <51426041+lov3b@users.noreply.github.com> Date: Tue, 15 Mar 2022 15:21:38 +0100 Subject: [PATCH] rook funkar inte --- src/schack/Board.java | 3 +- src/schack/Horse.java | 2 - src/schack/Pawn.java | 2 + src/schack/Rook.java | 132 ++++++++++++++++++++++++++++++++++-------- 4 files changed, 112 insertions(+), 27 deletions(-) diff --git a/src/schack/Board.java b/src/schack/Board.java index 371b9a9..59e6ca1 100644 --- a/src/schack/Board.java +++ b/src/schack/Board.java @@ -18,7 +18,7 @@ public class Board extends JPanel implements MouseListener { private Piece[][] pieces = new Piece[8][8]; private LinkedHashSet validMovesToDraw = new LinkedHashSet<>(); private Point selectedPiece = new Point(); - private Color moveableColor = new Color(200, 200, 200); + private Color moveableColor = new Color(255, 191, 0); public Board() throws IOException { @@ -193,7 +193,6 @@ public class Board extends JPanel implements MouseListener { Piece p = pieces[mouseCoordinateX][mouseCoordinateY]; LinkedHashSet validMoves = p.validMoves(pieces); System.out.println("valid moves " + validMoves); - moveableColor = new Color((int) (255 * Math.random()), (int) (255 * Math.random()), (int) (255 * Math.random())); validMovesToDraw.addAll(validMoves); System.out.println("valid moves to draw " + validMovesToDraw); diff --git a/src/schack/Horse.java b/src/schack/Horse.java index 09513ff..eb84468 100644 --- a/src/schack/Horse.java +++ b/src/schack/Horse.java @@ -16,6 +16,4 @@ public class Horse extends Piece { return new LinkedHashSet<>(); } - - } diff --git a/src/schack/Pawn.java b/src/schack/Pawn.java index 5e1aa81..0c62b26 100644 --- a/src/schack/Pawn.java +++ b/src/schack/Pawn.java @@ -17,6 +17,8 @@ public class Pawn extends Piece { @Override public LinkedHashSet validMoves(Piece[][] pieces) { + LinkedHashSet movable = new LinkedHashSet<>(); + return new LinkedHashSet<>(); } diff --git a/src/schack/Rook.java b/src/schack/Rook.java index 14cb0ee..e7cc02c 100644 --- a/src/schack/Rook.java +++ b/src/schack/Rook.java @@ -2,6 +2,7 @@ package schack; import java.awt.Point; import java.io.IOException; +import java.util.HashSet; import java.util.LinkedHashSet; public class Rook extends Piece { @@ -12,41 +13,126 @@ public class Rook extends Piece { } @Override - public LinkedHashSet validMoves(Piece[][] pieces) { + public LinkedHashSet validMoves(Piece[][] pieces) { LinkedHashSet movable = new LinkedHashSet<>(); //Behöver skriva att om rookX = this.position.x så ska vi istället loopa igenom //int rookY = 0-this.position.y; rookY < 8-this.position.Y; rookY++ //men jag är trög och har spenderat alldles förmycket tid på att vara trög :^) - for (int rookX = 0-this.position.x; rookX < 8-this.position.x; rookX++) { - - if (this.position.y == 0 && rookX == 0) { - continue; - } - Point pos = new Point(this.position.x + rookX, this.position.y); - // Instead of checking index and null, try-catch - try { - Piece p = pieces[pos.x][pos.y]; - System.out.println(p); - // If this piece is the same team as ours, skip - if (p.isWhite == this.isWhite) { - continue; - } - movable.add(pos); + // Vänster + for (int rookX = this.position.x; rookX >= 0; rookX--) { + Point pos = new Point(this.position.x - rookX, this.position.y); - } catch (NullPointerException npe) { - // This is an empty spot - movable.add(pos); - } catch (Exception e) { - // This means that the player is at the edge + // Instead of checking index and null, try-catch + try { + Piece p = pieces[pos.x][pos.y]; + System.out.println(p); + // If this piece is the same team as ours, skip + if (p.isWhite == this.isWhite) { + break; } - + + movable.add(pos); + break; + + } catch (NullPointerException npe) { + // This is an empty spot + movable.add(pos); + } catch (Exception e) { + // This means that the player is at the edge + } } + + // Höger + for (int rookX = 1; rookX <= this.position.x + rookX; rookX++) { + Point pos = new Point(this.position.x + rookX, this.position.y); + + // Instead of checking index and null, try-catch + try { + Piece p = pieces[pos.x][pos.y]; + System.out.println(p); + // If this piece is the same team as ours, skip + if (p.isWhite == this.isWhite) { + break; + } + + movable.add(pos); + break; + + } catch (NullPointerException npe) { + // This is an empty spot + movable.add(pos); + } catch (Exception e) { + // This means that the player is at the edge + } + + } + +// for (int rookY = 0 - this.position.y; rookY < 8 - this.position.y; rookY++) { +// if (this.position.y == 0 && rookY == 0) { +// continue; +// } +// Point pos = new Point(this.position.x, this.position.y + rookY); +// +// // Instead of checking index and null, try-catch +// try { +// Piece p = pieces[pos.x][pos.y]; +// System.out.println(p); +// // If this piece is the same team as ours, skip +// if (p.isWhite == this.isWhite) { +// continue; +// } +// movable.add(pos); +// +// } catch (NullPointerException npe) { +// // This is an empty spot +// movable.add(pos); +// } catch (Exception e) { +// // This means that the player is at the edge +// } +// } +// +// // En lista för att kolla alla vi har lagt till innan en pjäs av samma färg. +// HashSet toBeRemoved = new HashSet(); +// +// for (int rookX = 0 - this.position.x; rookX < 8 - this.position.x; rookX++) { +// +// if (this.position.y == 0 && rookX == 0) { +// continue; +// } +// Point pos = new Point(this.position.x + rookX, this.position.y); +// +// // Instead of checking index and null, try-catch +// try { +// Piece p = pieces[pos.x][pos.y]; +// System.out.println(p); +// +// // Funkar bara åt vänster +// if (pieces[pos.x + 1][pos.y] != null && pieces[pos.x + 1][pos.y] != this) { +// toBeRemoved.add(pos); +// } +// +// // If this piece is the same team as ours, skip +// if (p.isWhite == this.isWhite) { +// movable.removeAll(toBeRemoved); +// toBeRemoved.clear(); +// continue; +// } +//// toBeRemoved.add(pos); +// movable.add(pos); +// +// } catch (NullPointerException npe) { +// // This is an empty spot +// movable.add(pos); +// } catch (Exception e) { +// // This means that the player is at the edge +// } +// +// } System.out.println("Len of movable: " + movable.size()); return movable; } - }