(Code Review) Оператор Java if с логическим и (&&) оператором

Question

Я начал программировать несколько недель назад в java/android. Я хочу написать небольшую игру tic tac toe в качестве приложения для Android, но у меня возникают проблемы с моим методом, который будет проверять победителя. Она заключается в следующем:

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){} 
} 

Здесь я массив называется принято, что имеет 9 чисел, каждое из этих чисел является либо один, то есть игрок один владеет этот блок, или два, означающий игрока два РМО этого блока , Текущий, я пытаюсь попробовать все возможные сценарии, в которых игрок будет победителем, но затмение говорит мне, что The operator && is undefined for the argument type(s) int, int. Кажется, что ошибка показывает только первую логику и работу каждой строки оператора if. Например, первая ошибка увеличивается до taken[0] && taken[3], а затем исчезает до следующей строки.

Answer 1

В качестве альтернативы, вы можете чек приняты [п] значения, чтобы увидеть, если они имеют 1 или 2, если вы думаете, что бы сделать код более ясным:

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

Имейте в виду, что & & оператор ожидает boolean operands ... поэтому он не будет работать с вашим массивом int так, как вы ожидаете.

Answer 2

Поменяйте свой & & на ==, вы пытаетесь выяснить, все ли они одинаковые значения. Я предполагаю, что будет показывать победителя, и обязательно используйте круглые скобки для его сортировки, поэтому одно условие выигрыша будет

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

Однако, это только скажет вам, что какой-то игрок выиграл, а не какой игрок. Я думаю, вы можете проверить, какой игрок сделал последний ход, как только будет определено, что кто-то выиграл и объявит этого игрока победителем.

Answer 3

В Java

if (a && b) or if (a || b) 

работает только тогда, когда а и Ь BOOLEANS/логические выражения.

Answer 4

ОК здесь есть две основные проблемы.

Во-первых, taken[0] должен возвращать либо true, либо false, чтобы иметь возможность оставаться как есть внутри оператора if. Вы упомянули, что это целое число, поэтому для возврата true или false вам нужно выполнить сравнение, например taken[0] == 1.

Во-вторых, вам нужно использовать круглые скобки и выполнять некоторые группировки. Java не уважает пробелы. Вместо if(taken[0] && taken[3] && taken[6] || ... вам нужно будет сделать if((taken[0] && taken[3] && taken[6]) || .... То есть вам нужно поместить круглые скобки вокруг каждого набора группировок.

Причина, по которой Eclipse является ошибкой, заключается в том, что оператор && предназначен только для сравнения булевых. То есть true && true. Вы даете ему целые числа, и это не нравится.

Один из способов, которым вы, возможно, захотите решить это, - написать функцию, которая определяет, достигнуто ли решение. Так что, может быть что-то вроде:

private boolean isSolutionPresent(int[] taken, int index1, int index2, int index3) { 
    return (taken[index1] == 1) && (taken[index2] == 1) && (taken[index3] == 1); 
} 

Затем вы можете конвертировать ваши, если заявление на что-то вроде:

if (isSolutionPresent(taken, 0, 3, 6) || 
    isSolutionPresent(taken, 0, 1, 2) || 
    isSolutionPresent(taken, 2, 5, 8) || //... etc, removing the final ==1 

Поскольку метод isSolutionPresent возвращает логическое значение (то есть, истина/ложь), вы можете применить || оператора к нему.

Answer 5

1) Вы не можете использовать && или || операторов на int переменных, так как предназначены для boolean значений.

2) также использовать скобки для условий группы, как

if (
     (taken[0]==1 && taken[3]==1 && taken[6]==1)|| 
     (taken[0]==1 && taken[1]==1 && taken[2]==1)|| 
     ... 

Answer 6

Здесь я массив называется принятым, который содержит 9 целых чисел, каждое из этих чисел является либо один, то есть игрок один владеет этот блок, или два, что означает, что двум игрокам принадлежит этот блок.

В этом случае, вы можете заменить логические операторы с операторами поразрядными:

int winner = 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]; 

Тогда переменная winner будет содержать 1, если игрок 1 вона, 2, если игрок 2 вона, 0, если ни один из них выиграли, или 3 из них выиграли (что, вероятно, невозможно в вашей игре).