我怎样才能简化这个

时间:2021-04-17 13:52:44

标签: java java-stream refactoring simplify

有什么办法可以简化这段代码吗?我正好有 1 个白色棋子,想要得到它的位置

代码:

final Tile[] white = {null};
board.forEach(tile -> {
  Piece temp = tile.getPiece();
  if (temp != null) {
    if (temp.getType().equals("white")) { white[0] = tile; }
  }
});

System.out.println(white[0].getX());
System.out.println(white[0].getY());

瓷砖类:

public class Tile {

private final StringProperty color = new SimpleStringProperty(this, "color");
private final IntegerProperty x = new SimpleIntegerProperty(this, "x");
private final IntegerProperty y = new SimpleIntegerProperty(this, "y");
private final BooleanProperty hasPiece = new SimpleBooleanProperty(this, "hasPiece");
private final BooleanProperty isMarked = new SimpleBooleanProperty(this, "isMarked");
private final ObjectProperty<Piece> piece = new SimpleObjectProperty<>(this, "piece");

件类:

public class Piece {
private final StringProperty type = new SimpleStringProperty(this, "type");
private final StringProperty imagePath = new SimpleStringProperty(this, "imagePath");
private final ObjectProperty<List<Coords>> possible_moves = new SimpleObjectProperty<>(this, "possible_moves");

1 个答案:

答案 0 :(得分:0)

假设 boardTile 对象的集合,第一个“白色”部分可能是这样的:

// Collection<Tile> board
Tile white = board.stream()
    .filter(tile -> tile.getPiece() != null && "white".equals(tile.getPiece().getType()))
    .findFirst() // Optional<Tile>
    .orElse(null);

或者可以使用 Optional::map 来查找白色部分:

Tile white = board.stream()
    .filter(tile -> Optional.ofNullable(tile.getPiece())
                        .map(piece -> "white".equals(piece.getType()))
                        .orElse(Boolean.FALSE))
    .findFirst() // Optional<Tile>
    .orElse(null);