Проблемы с условиями победы TicTacToe

Что ж, я взял код, который вы предоставили, и после значительных усилий смог создать полностью функционирующую игру «Крестики-нолики». Вы были в основном на правильном пути в том, что вы делали, вам просто нужно было сначала начать с дизайна.

В моем классе NoughtsCrosses у меня есть следующее:

• class Action implements ActionListener
  • This has a JButton attribute that I pass in through a constructor
  • In the actionPerformed
    • set the text
    • отключить кнопку
    • увеличить счетчик
    • check if someone wins
      • If there is a winner or draw game ends set the "Play again?" text
      • иначе вызовите функцию changeTurn
• class Reset implements ActionListenter
  • This has a JButton attribute that I pass in through a constructor
  • In the actionPerformed
    • I call the resetGame function
• функция changeTurn
• функция сброса игры
• функция проверкиForWinners

как подсказка, это моя реализация класса Action и пример конструктора, о котором я упоминал

class Action implements ActionListener{ 
 private JButton button;
 public Action(JButton button){ 
  this.button = button; 
 } 
 public void actionPerformed(ActionEvent e) { 
   button.setText(letter); 
   button.setEnabled(false); 
   counter++; 
   boolean gameOver = checkForWinners(); 
   if(!gameOver) 
    changeTurn(); 
   else{ 
    newgame.setText("Play again?"); 
    newgame.addActionListener(resetButton); 
   } 
 } 
}

такой вызов, как new Action(square[i]), - это то, что вам нужно, чтобы это работало. Примечание: resetButton относится к классу Reset, о котором я упоминал выше, так же, как и класс Action, он имеет ту же конструкцию, в которую я передал newgame.

Похоже, что текст одного из квадратов равен нулю. Важно помнить одну вещь: пустая строка — это не то же самое, что null. В Java, если вы специально не присвоили значение строке, оно будет нулевым. Чтобы исправить это, вы захотите явно установить для каждого квадрата текст "" (пустая строка) при настройке игрового поля.

Похоже, ваша проверка условия победы находится не в вашем коде actionPerformed, а на уровне класса, поэтому, возможно, она вызывается до того, как окно будет заполнено вашими кнопками.

Попробуйте разместить галочку внутри actionPerformed следующим образом: http://pastebin.com/xRViSUzy

Какова область (просто говоря, фигурные скобки) внутри проблемной строки?

Было немного сложно сказать, основываясь на вашем отступе, но мне кажется, что ваше «если» не было внутри метода (например, конструктора). Я предполагаю, что вы намеревались выполнить эту строку и те, что вокруг них, после строк в теле вашего конструктора, где инициализируются квадраты. Вместо этого эти строки выполняются заранее, и поэтому вызов «нового» еще не выполнялся.

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

Надеюсь, это поможет.

Если вы собираетесь реализовать такое решение, то упростите себе работу. Судя по крошечному фрагменту кода, который я вижу выше, похоже, что вы действительно слишком усложняете работу, которую должны выполнять.

char cell0 = //get that char, be it X or O
char cell1 = //
...
char cell8 = //

Теперь вы можете сравнивать клетки одну за другой, чтобы определить победу. Ваша настольная игра устроена следующим образом:

0 1 2
3 4 5
6 7 8

Так что можно просто идти по порядку:

Horizontal Solutions:
(cell0 == cell1 && cell0 == cell2)
(cell3 == cell4 && cell3 == cell5)
(cell6 == cell7 && cell6 == cell8)

Vertical Solutions
(cell0 == cell3 && cell0 == cell6)
//And so on

Cross Solutions:
(cell0 == cell4 && cell0 == cell8)
(cell2 == cell4 && cell2 == cell6)

Это проверит ваше условие победы.

Проблема в том, что у вас в коде избыток фигурных скобок, так что операторы в вопросе фактически появляются в блоке инициализатора экземпляра класса NoughtsCrosses, но ни один из компонентов JButton еще не был инициализирован, поскольку инициализаторы экземпляра вызываются до конструкторы, в которых экземпляры JButton существуют (но никогда не вызываются). Когда вы пытаетесь вызвать getText для первого элемента массива square, выдается NullPointerException.

Чтобы исправить, удалите дополнительные фигурные скобки, чтобы код был заключен в предыдущий ActionListener

class Action implements ActionListener {
    public void actionPerformed(ActionEvent e) {
       // existing code here
/// }    remove
//}      remove
// {     remove

    // win conditions. if true, set win==true; else set win
    // here is where the compilation error is, next line
    if (square[0].getText() == square[1].getText() ...) { 
      win = true;
    } //etc
} <-- add this