java - Java免费现货

如果没有使用周围的坐标，则我有一块需要被其他灰色块包围。

我尝试执行以下操作：

public List<Piece> boardPiece(){
    List<Piece> boardPieces = new ArrayList<>();
    for (Piece pieces : listToPiece) {
        if (pieces.getCoordinate() != null){
                boardPieces.add(pieces);
        }
    }
    return boardPieces;
 }

public List<Coordinate> getSurroundings() {
     List<Coordinate> surroundings = new ArrayList<>();
     for (Piece boardpieces : boardPiece()) {
         for (Coordinate coordinate : makeDirections()) {
            surroundings.add(new Coordinate(boardpieces.getCoordinate().getRow() + coordinate.getRow(), boardpieces.getCoordinate().getColumn() + coordinate.getColumn()));
         }
     }
     return surroundings;
}

public List<Coordinate> makeDirections(){
      directions.add(new Coordinate(1,-1));
      directions.add(new Coordinate(-1,1));
      directions.add(new Coordinate(-1,0));
      directions.add(new Coordinate(1,0));
      directions.add(new Coordinate(0,-1));
      directions.add(new Coordinate(0,1));
      return directions;
    }

一块木板是一块在木板上并具有坐标的碎片。没有坐标的木板不应被其他灰色棋子包围。在我的抽屉里，我写了以下代码：

public void drawEmptyPieces() {
    for (Coordinate emptyPiece : emptyPieces) {
        EmptyPiece emptyPiece1 = new EmptyPiece(new Coordinate(emptyPiece.getRow(), emptyPiece.getColumn()));
        emptyPiece1.setFill(Color.LIGHTGRAY);
        emptyPiece1.setStroke(Color.BLACK);
        gameField.getChildren().add(emptyPiece1);
    }
}

我试图避免在木板上画出灰色碎片：

public void drawEmptyPieces() {
    for (Coordinate emptyPiece : emptyPieces) {
        for (Piece pieceObjects : pieces){
            if (pieceObjects.getCoordinate().getRow() != emptyPiece.getRow() && pieceObjects.getCoordinate().getColumn() != emptyPiece.getRow()){
                EmptyPiece emptyPiece1 = new EmptyPiece(emptyPiece);
                emptyPiece1.setFill(Color.LIGHTGRAY);
                emptyPiece1.setStroke(Color.BLACK);
                gameField.getChildren().add(emptyPiece1);
            }
        }
    }
}

但是此代码不起作用。

最佳答案

我不知道您提供了足够的信息来回答您的问题。但是这里有一些想法。



makeDirections()是损坏的（缺少directions局部变量的声明），或者是损坏的，因为它每次调用时都会不断向Coordinate字段添加6个新的directions对象。无论哪种情况，它所做的工作都超出了需要。请考虑：

private final static List<Coordinate> directions;

static {
    List<Coordinate> dirs = new ArrayList<>();
    dirs.add(new Coordinate(1,-1));
    dirs.add(new Coordinate(-1,1));
    dirs.add(new Coordinate(-1,0));
    dirs.add(new Coordinate(1,0));
    dirs.add(new Coordinate(0,-1));
    dirs.add(new Coordinate(0,1));
    directions = Collections.immutableList(dirs);
}

public static List<Coordinate> makeDirections() {  // or getDirections()
    return directions;
}

getSurroundings()看起来可以（会吗？）多次向周围添加相同的Coordinate。如果一块位于[5,5]，另一块位于[7,5]，则将[5,5]+[1,0]和[7,5]+[-1,0]添加到环境中。 [6,5]甚至可能有一块，也会在周围增加[5,5]和[7,5]。

相反...

确保Coordinate实现equals和hashCode，以便具有相同行和列的两个Coordinate对象相等，并且两个哈希值均相同。然后，在getSurroundings()中使用：

Set<Coordinate> surroundings = new HashSet<>();

因此，surroundings仅包含每个坐标一次。返回之前，您可以从周围环境中删除所有“占用”的坐标。

for (Piece piece : pieces) {
    surroundings.remove(piece.getCoordinate());
}

您可以返回集合而不是列表，或者如果要将其保留为列表：

return new ArrayList<>(surroundings);

如果没有包含重复项和座标的“周围环境”，您可能会发现绘制电路板要容易得多。



除非您的游戏板范围无限大，否则您将在板的边缘之外创建Coordinate对象。考虑将方法getNeighbours()添加到Coordinate类：

class Coordinate {
    public Collection<Coordinate> getNeighbours() {
        List<Coordinate> neighbours = new ArrayList<>(6);
        for (Coordinate dir: getDirections()) {
            Coordinate n = new Coordinate(getRow() + dir.getRow(), getColumn() + dir.getColumn());
            if (n.isValid()) {
                neighbours.add(n);
            }
        }
        return neighbours;
    }
}

然后getSurroundings变得更简单了：

for (Piece boardpieces : boardPiece()) {
    surroundings.addAll(boardpieces.getCoordinate().getNeighbours());
}

没有任何“环境”可能出现。



“方向”不是Coordinate。它们用于不同的事物。 Coordinate是板上的一个位置，您的“方向”是一个相对偏移量。董事会可能有约束，要求行和列的值必须在（例如）0到20之间；您的“方向”对象的行和列值为负；这将违反这些限制。这意味着如果给出错误的值，则不能在throw new IllegalArgumentException("Invalid coordinate")的构造函数中添加适当的检查和Coordinate。

我将为Direction对象创建一个新类。实际上，我会使用enum Direction { ... }。然后Direction.values()将返回所有方向的集合-您不需要自己创建该集合。这是比其他更改大得多的更改，并且可能带来的后果超出了您提供的一小部分代码。很好的改变，但可能需要做很多工作。



希望能有所帮助。

关于java - Java免费现货，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/36361859/