在Android Studio中无法调试错误(最可能是逻辑错误)

时间:2016-10-18 02:48:11

标签: java arrays android-studio-2.2

package com.example.sys.tictactoe2;

import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.view.View;
import android.widget.Button;
import android.widget.TextView;
import java.util.Random;

public class GameBoard extends AppCompatActivity {
int c[][];
int i,j=0;
Button b[][];
Button newgame;
AI ai;
Human human;
TextView textView;

//Initiates the application
@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_game_board);

    setBoard();
    randomMove();
}

// Random player moves first
public boolean randomMove(){
    Random firstMove = new Random();
    int player = firstMove.nextInt(2);

    if (player == 1) {
        textView.setText("You play first!");
        human.humanTurn();
        return true;

    } else {
        textView.setText("I play first!");
        ai.aiTurn();
        return false;
    }
}

//Initiates the Game Board
public void setBoard(){
ai = new AI();
    b = new Button[4][4];
    c = new int[4][4];

    newgame = (Button) findViewById(R.id.New);
    newgame.setOnClickListener(new NewClickListener());

    textView = (TextView) findViewById(R.id.message);

    b[1][1] = (Button) findViewById(R.id.button1);
    b[1][2] = (Button) findViewById(R.id.button2);
    b[1][3] = (Button) findViewById(R.id.button3);

    b[2][1] = (Button) findViewById(R.id.button4);
    b[2][2] = (Button) findViewById(R.id.button5);
    b[2][3] = (Button) findViewById(R.id.button6);

    b[3][1] = (Button) findViewById(R.id.button7);
    b[3][2] = (Button) findViewById(R.id.button8);
    b[3][3] = (Button) findViewById(R.id.button9);

    for (i=1; i<=3;i++){
        for (j=1;j<=3;j++)
            c[i][j] = 2;
        if (!b[i][j].isEnabled()) {
            b[i][j].setText("");
            b[i][j].setEnabled(true);
        }
    }
}

//If human plays first, class MyClickListener is initialized
public class Human {
    private void humanTurn() {
        for (i = 1; i <= 3; i++) {
            for (j = 1; j <= 3; j++) {
                b[i][j].setOnClickListener(new MyClickListener(i, j));
            }
        }
    }
}

public class MyClickListener implements View.OnClickListener{
    int x ;
    int y;

    private MyClickListener (int x, int y){
        this.x = x;
        this.y = y;
    }

    public void onClick(View View){
        if (b[x][y].isEnabled()){
            b[x][y].setEnabled(false);
            b[x][y].setText("O");
            c[x][y] = 0;
            textView.setText("");
            checkBoard();
            if(!checkBoard()){
               ai.aiTurn();
            }
      }
    }
}

public class AI {
    private void aiTurn(){
        if(c[1][1]==2 &&
        ((c[1][2]==0 && c[1][3]==0) ||
                (c[2][2]==0 && c[3][3]==0) ||
                (c[2][1])==0 && c[3][1]==0)){
            markSquare(1,1);
        }  else if (c[1][2]==2 &&
                ((c[2][2]==0 && c[3][2]==0) ||
                        (c[1][1]==0 && c[1][3]==0))) {
            markSquare(1,2);
        } else if(c[1][3]==2 &&
                ((c[1][1]==0 && c[1][2]==0) ||
                        (c[3][1]==0 && c[2][2]==0) ||
                        (c[2][3]==0 && c[3][3]==0))) {
            markSquare(1,3);
        } else if(c[2][1]==2 &&
                ((c[2][2]==0 && c[2][3]==0) ||
                        (c[1][1]==0 && c[3][1]==0))){
            markSquare(2,1);
        } else if(c[2][2]==2 &&
                ((c[1][1]==0 && c[3][3]==0) ||
                        (c[1][2]==0 && c[3][2]==0) ||
                        (c[3][1]==0 && c[1][3]==0) ||
                        (c[2][1]==0 && c[2][3]==0))) {
            markSquare(2,2);
        } else if(c[2][3]==2 &&
                ((c[2][1]==0 && c[2][2]==0) ||
                        (c[1][3]==0 && c[3][3]==0))) {
            markSquare(2,3);
        } else if(c[3][1]==2 &&
                ((c[1][1]==0 && c[2][1]==0) ||
                        (c[3][2]==0 && c[3][3]==0) ||
                        (c[2][2]==0 && c[1][3]==0))){
            markSquare(3,1);
        } else if(c[3][2]==2 &&
                ((c[1][2]==0 && c[2][2]==0) ||
                        (c[3][1]==0 && c[3][3]==0))) {
            markSquare(3,2);
        } else if( c[3][3]==2 &&
                ((c[1][1]==0 && c[2][2]==0) ||
                        (c[1][3]==0 && c[2][3]==0) ||
                        (c[3][1]==0 && c[3][2]==0))) {
            markSquare(3, 3);
        } else {
            Random rand = new Random();

            int a = rand.nextInt(4);
            int b = rand.nextInt(4);
            while (a==0 || b==0 || c[a][b]!=2) {
                a = rand.nextInt(3);
                b = rand.nextInt(3);
            }
            markSquare(a,b);
        }
        }
    private void markSquare(int x, int y){
        b[x][y].setEnabled(false);
        b[x][y].setText("X");
        c[x][y] = 1;
        checkBoard();
        if(!checkBoard()){
            human.humanTurn();
        }
    }

}

//Check for the winner
public boolean checkBoard(){
    boolean gameOver=false;
    if ((c[1][1]==0 && c[2][2]==0 && c[3][3]==0) ||
            (c[1][3]==0 && c[2][2]==0 && c[3][1]==0) ||
            (c[1][2]==0 && c[2][2]==0 && c[3][2]==0) ||
            (c[1][3]==0 && c[2][3]==0 && c[3][3]==0) ||
            (c[1][1]==0 && c[1][2]==0 && c[1][3]==0) ||
            (c[2][1]==0 && c[2][2]==0 && c[2][3]==0) ||
            (c[3][1]==0 && c[3][2]==0 && c[3][3]==0) ||
            (c[1][1]==0 && c[2][1]==0 && c[3][1]==0)){
        textView.setText("Game over. You win!");
        gameOver = true;

    } else if ((c[1][1]==1 && c[2][2]==1 && c[3][3]==1) ||
            (c[1][3]==1 && c[2][2]==1 && c[3][1]==1) ||
            (c[1][2]==1 && c[2][2]==1 && c[3][2]==1) ||
            (c[1][3]==1 && c[2][3]==1 && c[3][3]==1) ||
            (c[1][1]==1 && c[1][2]==1 && c[1][3]==1) ||
            (c[2][1]==1 && c[2][2]==1 && c[2][3]==1) ||
            (c[3][1]==1 && c[3][2]==1 && c[3][3]==1) ||
            (c[1][1]==1 && c[2][1]==1 && c[3][1]==1)){
        textView.setText("Game over. You lost!");
        gameOver = true;
    } else {
        boolean empty = false;
        for (i=1; i<=3; i++) {
            for (j = 1; j <= 3; j++) {
                if (c[i][j] == 2) {
                    empty = true;
                    break;
                }
            }
        }
        if(!empty){
            gameOver=true;
            textView.setText("Game over. It's a draw!");
        }
    }
    return gameOver;
}

//New Button resets the game
class NewClickListener implements View.OnClickListener{

    private NewClickListener(){
    }

    @Override
    public void onClick(View v) {
        setBoard();
        textView.setText("");
    }
}
}

这是我在Android Studio中的MainActivity文件(Refactor as GameBoard)。我正在尝试构建一个Tic Tac Toe游戏。当我尝试在我的Android设备上运行此应用程序时,它会崩溃并显示消息“不幸的是,应用程序已停止”。

根据我的搜索,我尝试删除设备上的缓存并重建android项目。控制台根本不显示任何错误。它必定是一些逻辑错误。请让我知道我在哪里做错了。另外,有人也可以建议我,如何找到逻辑错误?

1 个答案:

答案 0 :(得分:0)

我们确实需要更多信息。我将首先向我们展示布局文件。错误可能在那里。或者错误可能是以下两种方法之一:

  • setBoard();
  • randomMove();

当调用human.humanMove()时,randomMove()中可能会发生错误。您尚未为人类对象分配值,因此它们为空。

我建议:

public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_game_board);

    human = new Human();
    setBoard();
    randomMove();
}

虽然我认为setBoard()不会抛出致命错误,但我可以看到你已经习惯了基于非零的索引。你的数组是1个元素。例如你应该声明一个包含三个元素的数组:

b = new Button[3][3];

并访问所有这样的元素:

b[0][0] = ...; 
b[0][1] = ...; 
b[0][2] = ...; 
b[1][0] = ...; 

所以你可以看到索引2实际上是第三个元素。

我还建议你制作一个通用的Player类,让AI和Human从中继承 - 这将使你更容易确保两个玩家的行为遵循相同的规则,而不是每次引入新的规则时都要执行两次规则之一。