Question

I have started programming a few weeks ago in java/android. I want to write a small tic tac toe game as an android app but I’m having trouble with my method that will check for the winner. It is as follows:

 public void checkForWinner() { 

        if( taken[0] && taken[3] && taken[6]  ||
            taken[0] && taken[1] && taken[2] ||
            taken[2] && taken[5] && taken[8] ||
            taken[6] && taken[7] && taken[8] ||
            taken[0] && taken[4] && taken[8] ||
            taken[2] && taken[4] && taken[6] ||
            taken[1] && taken[4] && taken[7] ||
            taken[3] && taken[4] && taken[5] == 1 ){}
}

Here I have an array called taken that holds 9 integers, each of those integers being either a one, meaning player one owns that block, or a two, meaning player two ows that block. Current, I am trying trying all possible scenarios in which player one would be the winner but eclipse is telling me that The operator && is undefined for the argument type(s) int, int. The error only seems to be showing for the first logical and operation of each line of the if statement. For example the first error goes up to taken[0] && taken[3] and then disappears until the next line.

Answer 1

Alternatively, you can check taken[n] values to see if they hold 1 or 2, if you think that’d make your code clearer:

(taken[0]==1 && taken[3]==1 && taken[6]==1)

Keep in mind that the && operator expects boolean operands…so it won’t work with your int array the way you’re expecting it to.