有 Java 编程相关的问题?

你可以在下面搜索框中键入要查询的问题!

java声明计算机每次都会获胜,尽管在Android应用程序中这不是正确的结果

我几乎在安卓上完成了一个tic-tac-toe应用程序(玩家对电脑),但我注意到我的游戏出现了一个小故障。不管结果如何,电脑总是赢。所以,若玩家连续三场或者是平局,它仍然表示计算机获胜,我不太确定我在代码中加入了什么错误导致了这一点。我怎样才能获得比赛的正确结果

以下是完整代码:

public class MainActivityPlayer1 extends AppCompatActivity implements View.OnClickListener {

    private Button[][] buttons = new Button[3][3];

    private boolean playerOneMove = true;

    private int turnsCount;

    private int playerOnePoints;
    private int playerTwoPoints;

    private TextView textViewPlayerOne;
    private TextView textViewPlayerTwo;
    private TextView textViewPlayerOneTurn;
    private TextView textViewPlayerTwoTurn;

    int playerX = Color.parseColor("#e8e5e5");
    int playerO = Color.parseColor("#737374");


    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        setContentView(R.layout.activity_main_player2);

        textViewPlayerOne = findViewById(R.id.textView_player1);
        textViewPlayerTwo = findViewById(R.id.textView_player2);
        textViewPlayerOneTurn = findViewById(R.id.textView_player1Turn);
        textViewPlayerTwoTurn = findViewById(R.id.textView_player2Turn);


        for (int i = 0; i < 3; i++) {
            for (int j = 0; j < 3; j++) {
                String buttonID = "button_" + i + j;
                int resID = getResources().getIdentifier(buttonID, "id", getPackageName());
                buttons[i][j] = findViewById(resID);
                buttons[i][j].setOnClickListener(this);

                if (savedInstanceState != null) {
                    String btnState = savedInstanceState.getCharSequence(buttonID).toString();
                    if (btnState.equals("X")) {
                        buttons[i][j].setTextColor(playerX);
                    } else {
                        buttons[i][j].setTextColor(playerO);
                    }

                }

            }
        }

        Button buttonReset = findViewById(R.id.button_reset);
        buttonReset.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                resetGame();
            }
        });

        Button buttonExit = findViewById(R.id.button_exit);
        buttonExit.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                backToMainActivity();
            }
        });

    }

    @Override
    public void onClick(View v) {

        if (!((Button) v).getText().toString().equals("")) {
            return;
        }

        if (playerOneMove) {
            ((Button) v).setText("X");
            ((Button) v).setTextColor(playerX);
            computerMove();
        }

        turnsCount++;

        if (checkGameIsWon()) {
            if (playerOneMove) {
                player1Wins();
            } else {
                player2Wins();
            }
        } else if (turnsCount == 9) {
            draw();
        } else {
            playerOneMove = !playerOneMove;
            switchPlayerTurn();
        }
    }

    private boolean computerMove() {
        playerOneMove = false;
        String[][] field = new String[3][3];

        for (int i = 0; i < 3; i++) {
            for (int j = 0; j < 3; j++) {
                field[i][j] = buttons[i][j].getText().toString();
                if (field[i][j].equals("")) {
                    buttons[i][j].setText("O");
                    buttons[i][j].setTextColor(playerO);
                    turnsCount++;
                    switchPlayerTurn();
                    return true;
                }
            }
        }
        return false;
    }



    private boolean checkGameIsWon() {
        String[][] field = new String[3][3];

        for (int i = 0; i < 3; i++) {
            for (int j = 0; j < 3; j++) {
                field[i][j] = buttons[i][j].getText().toString();
            }
        }

        for (int i = 0; i < 3; i++) {
            if (field[i][0].equals(field[i][1])
                    && field[i][0].equals(field[i][2])
                    && !field[i][0].equals("")) {
                return true;
            }
        }

        for (int i = 0; i < 3; i++) {
            if (field[0][i].equals(field[1][i])
                    && field[0][i].equals(field[2][i])
                    && !field[0][i].equals("")) {
                return true;
            }
        }

        if (field[0][0].equals(field[1][1])
                && field[0][0].equals(field[2][2])
                && !field[0][0].equals("")) {
            return true;
        }

        if (field[0][2].equals(field[1][1])
                && field[0][2].equals(field[2][0])
                && !field[0][2].equals("")) {
            return true;
        }

        return false;
    }

    private void player1Wins() {
        playerOnePoints++;
        Toast.makeText(this, "Player 1 wins!", Toast.LENGTH_SHORT).show();
        updatePointsText();
        resetBoard();
    }

    private void player2Wins() {
        playerTwoPoints++;
        Toast.makeText(this, "Computer wins!", Toast.LENGTH_SHORT).show();
        updatePointsText();
        resetBoard();
        computerMove();
        playerOneMove = true;
        switchPlayerTurn();
    }

    private void draw() {
        Toast.makeText(this, "Draw!", Toast.LENGTH_SHORT).show();
        resetBoard();
        if (!playerOneMove){
            computerMove();
            turnsCount++;
        }
    }

    @SuppressLint("SetTextI18n")
    private void updatePointsText() {
        textViewPlayerOne.setText("Player 1:  " + playerOnePoints + " ");
        textViewPlayerTwo.setText("Computer: " + playerTwoPoints + " ");
    }

    private void resetBoard() {
        for (int i = 0; i < 3; i++) {
            for (int j = 0; j < 3; j++) {
                buttons[i][j].setText("");
            }
        }

        turnsCount = 0;
        switchPlayerTurn();
    }

    private void resetGame() {
        playerOnePoints = 0;
        playerTwoPoints = 0;
        turnsCount = 0;
        playerOneMove = true;
        updatePointsText();
        resetBoard();
    }

    @Override
    protected void onSaveInstanceState(Bundle outState) {
        super.onSaveInstanceState(outState);

        outState.putInt("turnsCount", turnsCount);
        outState.putInt("playerOnePoints", playerOnePoints);
        outState.putInt("playerTwoPoints", playerTwoPoints);
        outState.putBoolean("playerOneMove", playerOneMove);
        switchPlayerTurn();

        for (int i = 0; i < 3; i++) {
            for (int j = 0; j < 3; j++) {
                String buttonID = "button_" + i + j;
                Button btn = buttons[i][j];
                outState.putCharSequence(buttonID, btn.getText());
            }
        }

    }

    @Override
    protected void onRestoreInstanceState(Bundle savedInstanceState) { ;
        super.onRestoreInstanceState(savedInstanceState);

        turnsCount = savedInstanceState.getInt("turnsCount");
        playerOnePoints = savedInstanceState.getInt("playerOnePoints");
        playerTwoPoints = savedInstanceState.getInt("playerTwoPoints");
        playerOneMove = savedInstanceState.getBoolean("playerOneMove");
        switchPlayerTurn();

        for (int i = 0; i < 3; i++) {
            for (int j = 0; j < 3; j++) {
                String buttonID = "button_" + i + j;
                Button btn = buttons[i][j];
                savedInstanceState.putCharSequence(buttonID, btn.getText());

            }
        }

    }

    private void switchPlayerTurn(){
        if (playerOneMove){
            textViewPlayerOneTurn.setVisibility(View.VISIBLE);
            textViewPlayerTwoTurn.setVisibility(View.INVISIBLE);
        }
        else{
            textViewPlayerOneTurn.setVisibility(View.INVISIBLE);
            textViewPlayerTwoTurn.setVisibility(View.VISIBLE);
        }

    }

    private void backToMainActivity(){
        Intent intentMainActivity = new Intent(this, MainActivity.class);
        startActivity(intentMainActivity);
    }

}

共 (1) 个答案

  1. # 1 楼答案

    这是因为每当玩家一号通过调用computerMove()打开时,你都会设置playerOneMove = false;(在if (playerOneMove) {),并且在该方法中,你会立即设置playeOneMove = false;

    如果不是这样的话,也请发布switchPlayerTurn()方法

    此外,我认为你应该在if(playerOneMove)之前提高你的获胜条件测试

    示例(我更改了您的条件,移动了测试获胜条件的代码,创建了其他方法……:

       @Override
        public void onClick(View v) {
            if (!((Button) v).getText().toString().equals("")) {
                return;
            }
                    ((Button) v).setText("X");
                    ((Button) v).setTextColor(playerX);
                    playerOneMove = false;
                    switchPlayerTurn();
                    computerMove();
                    playerOneMove = true;
                    switchPlayerTurn();
    
            turnsCount++;
        }
    
        private boolean computerMove() {
    
            String[][] field = new String[3][3];
    
            for (int i = 0; i < 3; i++) {
                for (int j = 0; j < 3; j++) {
                    field[i][j] = buttons[i][j].getText().toString();
                    if (field[i][j].equals("")) {
                        buttons[i][j].setText("O");
                        buttons[i][j].setTextColor(playerO);
                        turnsCount++;
                        return true;
                    }
                }
            }
            return false;
        }
    
    
    
        private boolean checkGameIsWon() {
            String[][] field = new String[3][3];
            Boolean isDone = false.
            for (int i = 0; i < 3; i++) {
                for (int j = 0; j < 3; j++) {
                    field[i][j] = buttons[i][j].getText().toString();
                }
            }
    
            for (int i = 0; i < 3; i++) {
                if (field[i][0].equals(field[i][1])
                        && field[i][0].equals(field[i][2])
                        && !field[i][0].equals("")) {
                    isDone = true;
                }
            }
    
            for (int i = 0; i < 3; i++) {
                if (field[0][i].equals(field[1][i])
                        && field[0][i].equals(field[2][i])
                        && !field[0][i].equals("")) {
                    isDone = true;
                }
            }
    
            if (field[0][0].equals(field[1][1])
                    && field[0][0].equals(field[2][2])
                    && !field[0][0].equals("")) {
                isDone = true;
            }
    
            if (field[0][2].equals(field[1][1])
                    && field[0][2].equals(field[2][0])
                    && !field[0][2].equals("")) {
                isDone = true;
            }
               if(isDone){
                  if (playerOneMove) {
                      player1Wins();
                      return true;
                  } else {
                      player2Wins();
                      return true;
                  }
              }
              if (turnsCount == 9) {
                  draw();
                  return true;
              }
          }
        return false;
        }
    
        private void player1Wins() {
            playerOnePoints++;
            Toast.makeText(this, "Player 1 wins!", Toast.LENGTH_SHORT).show();
            updatePointsText();
            resetBoard();
        }
    
        private void player2Wins() {
            playerTwoPoints++;
            Toast.makeText(this, "Computer wins!", Toast.LENGTH_SHORT).show();
            updatePointsText();
            resetBoard();
            computerMove();
            playerOneMove = true;
            switchPlayerTurn();
        }