<Java> 国际象棋棋子移动问题

Question

我正在用 Java 制作国际象棋游戏，而棋子给我带来了麻烦。老实说，我不知道为什么，因为他们的动作是最简单的。这是 Piece 声明和 Pawn（扩展 Piece）声明。运动类定义了一个简单的 x;y 对象，我用它来标记板上可能的运动目标。我有一个 GameHandler 类，以后会根据游戏规则清除所有不可能的运动。但是 Pawn 的移动数组似乎很干净；所有其他作品的动作都像奇迹一样工作！提前致谢！

单品类

public abstract class Piece{
    protected int x, y;
    protected boolean isWhite;
    protected ArrayList<Movement> moves;
    public Piece(int x, int y, boolean isWhite) {
        this.x=x;
        this.y=y;
        this.isWhite = isWhite;
        moves = new ArrayList<Movement>();
    }
    public abstract ArrayList<Movement> getMoves();
    //obvious methods
    public int getX()
    public int getY()
    public boolean isWhite()
    //end obvious methods
    public void setCoordinates(int x, int y){
        this.x=x;
        this.y=y;
    }   
}

典当类

public class Pawn extends Piece{
    public Pawn(int x, int y, boolean isWhite){
        super(x,y,isWhite);
    }
    public ArrayList<Movement> getMoves() {
        moves.clear();
        if(isWhite){
            if(y>0) moves.add(new Movement(x, y-1));
            if(y==7) moves.add(new Movement(x, y-2));
        }else{
            if(y<7) moves.add(new Movement(x, y+1));
            if(y==0) moves.add(new Movement(x, y+2));
        }
        return moves;
    }
}

编辑：添加 King 类作为参考

public class King extends Piece{
    public King(int x, int y, boolean isWhite){
        super(x,y,isWhite);
    }
    public ArrayList<Movement> getMoves() {
        moves.clear();
        if(y-1>=0 && x-1>=0) moves.add(new Movement (x-1, y-1));
        if(y-1>=0 && x+1<8) moves.add(new Movement (x+1, y-1));
        if(y+1<8 && x+1<8) moves.add(new Movement (x+1, y+1));
        if(y+1<8 && x-1>=0) moves.add(new Movement (x-1, y+1));
        if(y-1>=0) moves.add(new Movement (x, y-1));
        if(y+1<8) moves.add(new Movement (x, y+1));
        if(x+1<8) moves.add(new Movement (x+1, y));
        if(x-1>=0) moves.add(new Movement (x-1, y));
        return moves;
    }
}

Answer 1

我认为这行有一个错误：

if(isWhite){
        if(y>0) moves.add(new Movement(x, y-1));
        ...

应该是：

if(!isWhite) {
   if(y> ...

（这样黑棋，而不是白棋棋子的等级在向前移动时会降低）。

如前所述，棋子移动是最难编码的——它可以从起始位置向前移动 2 个方格（但前提是前面没有其他棋子），它沿对角线捕获，它可以提升到其中一个其他 4 种类型。另外，考虑一下这个位置：

这里（假设白棋要走）白兵根本没有合法的走法，因为它被黑主教钉住了。但是，如果黑方的最后一步是 c7-c5，那么白棋可以捕获黑棋并移动到 c6（"En passant" rule）。

您可以通过将x 和y 分别重命名为file 和rank 来提高代码的可读性。我也会考虑引入一个

enum PieceColour {
   WHITE, BLACK
}

并使用PieceColour pieceColour 和if (pieceColour == PieceColour.BLACK) 代替boolean isWhite 和if(!isWhite)。

Answer 2

好的，一个想法。

看看你的 if 语句。

试试

if(this.isWhite)

或

if(isWhite())

对于棋盘，我认为您应该检查位置是否等于 1 或 6。而不是 0 或 7。