7

Ao checar se alguem ganhou, preciso usar o IF várias vezes. Gostaria de saber se tem como simplificar um pouco o código e até talvez substituir os IFs.

//left, top, right, bottom are int positions relative to canvas
//state is a enum{Blank,Red,Blue} which is used to determine if the square was clicked and who clicked it

private boolean checkHorizontalWin() {
    for (Square s1 : squareList) {
        if (s1.getState() != State.BLANK) {
            for (Square s2 : squareList) {
                if (s1 != s2) {
                    if (s1.getTop() == s2.getTop()) {
                        if (s1.getState() == s2.getState()) {
                            for (Square s3 : squareList) {
                                if (s1 != s3 && s2 != s3) {
                                    if (s1.getTop() == s3.getTop()) {
                                        if (s1.getState() == s3.getState()) {
                                            return true;
                                        } else break;
                                    }
                                }
                            }
                        } else break;
                    }
                }
            }
        }
    } return false;
}

private boolean checkVerticalWin() {
    for (Square s1 : squareList) {
        if (s1.getState() != State.BLANK) {
            for (Square s2 : squareList) {
                if (s1 != s2) {
                    if (s1.getLeft() == s2.getLeft()) {
                        if (s1.getState() == s2.getState()) {
                            for (Square s3 : squareList) {
                                if (s1 != s3 && s2 != s3) {
                                    if (s1.getLeft() == s3.getLeft()) {
                                        if (s1.getState() == s3.getState()) {
                                            return true;
                                        } else break;
                                    }
                                }
                            }
                        } else break;
                    }
                }
            }
        }
    } return false;
}

private boolean checkDiagonalWin() {
    for (Square s1 : squareList) {
        if (s1.getState() != State.BLANK) {
            for (Square s2 : squareList) {
                if (s1 != s2) {
                    if (s1.getBottom() != s2.getBottom()) {
                        if (s1.getRight() == s2.getLeft()) {
                            for (Square s3 : squareList) {
                                if (s1 != s3 && s2 != s3) {
                                    if (s1.getBottom() != s3.getBottom() && s2.getBottom() != s3.getBottom()) {
                                        if (s2.getRight() == s3.getLeft()){
                                            if (s1.getState() == s2.getState() && s1.getState() == s3.getState()) {
                                                return true;
                                            } else break;
                                        }
                                    }
                                }
                            }
                        }
                    }
                }
            }
        }
    } return false;
}

squareList é criada de uma maneira parecida com esta:

private List<Square> squareList = new ArrayList<>();
private final int GRID_SIZE = 3;
private final int GRID_AREA = GRID_SIZE * GRID_SIZE;

private void createSquares() {
    int x = 0;
    int y = 0;
    for (int i = 0; i < gridArea; i++) {
        int left = field.getWidth() / 3 * x;
        int top = field.getHeight() / 3 * y + field.getBorder();
        int right = field.getWidth() / 3 * (x + 1);
        int bottom = field.Height() / 3 * (y + 1) + field.getBorder();
        squareList.add(new Square(left, top, right, bottom, State.BLANK));

        x++;
        if (x == GRID_SIZE) {
            x = 0;
            y++;
        }
    }
}

Prefiro não esar Java 8 devida a falta de compatibilidade com APIs velhas

5
  • Como essa squareList é construída? Commented 6/08/2018 às 20:26
  • O que são os atributos Top, State, Left, Right e Bottom de Square?
    – Marcel
    Commented 6/08/2018 às 20:33
  • squareList é uma lista com os 9 quadrados do tabuleiro e os atributos são a posição deles relativa ao canvas Commented 6/08/2018 às 20:49
  • E como é que você cria a squareList? Em que ordem os quadrados são inseridos nessa lista? Commented 6/08/2018 às 20:54
  • primeira linha da esquerda pra direita, depois linha de baixo na mesma ordem e depois a ultima Commented 6/08/2018 às 20:58

2 Respostas 2

14

Faça algo assim:

// O squareList deve ser uma lista com 9 posições onde os índices estão assim:

// +---+---+---+
// | 0 | 1 | 2 |
// +---+---+---+
// | 3 | 4 | 5 |
// +---+---+---+
// | 6 | 7 | 8 |
// +---+---+---+

private State[] getStates() {
    State[] s = new State[9];
    int i = 0;
    for (Square sq : squareList) {
        s[i] = sq.getState();
        i++;
    }
    return s;
}

private boolean isWon(State[] s, int a, int b, int c) {
    return s[a] != State.BLANK && s[a] == s[b] && s[b] == s[c];
}

private boolean checkWin() {
    State[] s = getStates();
    return isWon(s, 0, 1, 2)  // Horizontal 1.
        || isWon(s, 3, 4, 5)  // Horizontal 2.
        || isWon(s, 6, 7, 8)  // Horizontal 3.
        || isWon(s, 0, 3, 6)  // Vertical 1.
        || isWon(s, 1, 4, 7)  // Vertical 2.
        || isWon(s, 2, 5, 8)  // Vertical 3.
        || isWon(s, 0, 4, 8)  // Diagonal 1.
        || isWon(s, 2, 4, 6); // Diagonal 2.
}

Se preferir usar Java ≥ 8, você pode construir o método getStates() assim:

private State[] getStates() {
    return squareList.stream()
            .map(Square::getState)
            .collect(Collectors.toArray(State[]::new));
}

Se a ordem dos elementos no squareList não for a que está delineada acima, recomendo fortemente que altere o mecanismo pelo qual a squareList é construída para que seja.

0

Creio que assim seja mais limpo:

private boolean checkHorizontalWin() {
        if(isEqual(squareList.get(0), squareList.get(1), squareList.get(2))) return true;
        if(isEqual(squareList.get(3), squareList.get(4), squareList.get(5))) return true;
        if(isEqual(squareList.get(6), squareList.get(7), squareList.get(8))) return true;
        return false;
    }

    private boolean checkVerticalWin() {
        if(isEqual(squareList.get(0), squareList.get(3), squareList.get(6))) return true;
        if(isEqual(squareList.get(1), squareList.get(4), squareList.get(7))) return true;
        if(isEqual(squareList.get(2), squareList.get(5), squareList.get(8))) return true;
        return false;
    }

    private boolean checkDiagonalWin() {
        if(isEqual(squareList.get(0), squareList.get(4), squareList.get(8))) return true;
        if(isEqual(squareList.get(2), squareList.get(4), squareList.get(6))) return true;
        return false;
    }

    private boolean isEqual(Square first, Square second, Square third) {
        if(first.getState() == State.BLANK || second.getState() == State.BLANK || third.getState() == State.BLANK)
            return false;

        long equalStatus = Arrays.asList(first, second, third)
                .stream().map(square -> square.getState()).collect(Collectors.toList())
                .stream().distinct().count();

        return equalStatus == 1;
    }

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .