Пришлось изменить код , чтобы можно было его сделать лаконичнее:
public static void iterate(Set<Position> positions, int px, int py){
for (int i = 1; i <= 7; i++) {
int mx = Math.max(0, Math.min(x + px*i, 7));
int my = Math.max(0, Math.min(y + py*i, 7));
// в реальной жизни вторая проверка не нужна,
// т.к. в этом месте стоит текущая фигура и первая проверка не пройдет
if (f[mx][my] == null && !(mx==x && my==y))
positions.add(new Position(mx, my));
else break;
}
}
public static Set<Position> aviableMove2() {
Set<Position> positionList = new HashSet<>();
iterate(positionList, 1,0);
iterate(positionList, 0,1);
iterate(positionList, -1,0);
iterate(positionList, 0,-1);
return positionList;
}
Простыня которой тестировал:
import java.util.HashSet;
import java.util.Objects;
import java.util.Set;
public class CheckMate {
static Figure f[][] = new Figure[8][8];
static int x, y;
public static void iterate(Set<Position> positions, int px, int py){
for (int i = 1; i <= 7; i++) {
int mx = Math.max(0, Math.min(x + px*i, 7));
int my = Math.max(0, Math.min(y + py*i, 7));
if (f[mx][my] == null && !(mx==x && my==y))
positions.add(new Position(mx, my));
else break;
}
}
public static Set<Position> aviableMove() {
Set<Position> positionList = new HashSet<>();
for (int i = 1; i <= 7; i++) {
if ((x + i) <= 7 && f[x + i][y] == null) {
positionList.add(new Position((x + i), y));
} else break;
}
for (int i = 1; i <= 7; i++) {
if ((x - i) >= 0 && f[x - i][y] == null) {
positionList.add(new Position((x - i), y));
} else break;
}
for (int i = 1; i <= 7; i++) {
if ((y + i) <= 7 && f[x][y + i] == null) {
positionList.add(new Position(x, (y + i)));
} else break;
}
for (int i = 1; i <= 7; i++) {
if ((y - i) >= 0 && f[x][y - i] == null) {
positionList.add(new Position(x, (y - i)));
} else break;
}
return positionList;
}
public static void main(String[] args) {
for (int i=0; i<8; i++)
for (int j=0; j<8; j++)
test(i, j);
}
static void test(int tx, int ty) {
x = tx;
y = ty;
Set<Position> moves1 = aviableMove();
Set<Position> moves2 = aviableMove2();
boolean eq = moves1.containsAll(moves2) && moves2.containsAll(moves1);
System.out.println("test x=" + tx + " y=" + ty + " " + eq);
if (eq)return;
System.out.println(moves1);
System.out.println(moves2);
System.out.println("---");
}
public static Set<Position> aviableMove2() {
Set<Position> positionList = new HashSet<>();
iterate(positionList, 1,0);
iterate(positionList, 0,1);
iterate(positionList, -1,0);
iterate(positionList, 0,-1);
return positionList;
}
static private class Position {
private final int x;
private final int y;
public Position(int x, int y) {
this.x = x;
this.y = y;
}
@Override
public String toString() {
return "[" + x + "," + y + "]";
}
@Override
public boolean equals(Object o) {
if (this == o) return true;
if (o == null || getClass() != o.getClass()) return false;
Position position = (Position) o;
return x == position.x &&
y == position.y;
}
@Override
public int hashCode() {
return Objects.hash(x, y);
}
}
static private class Figure {}
}