Question

我正在使用类编写这个tic tac toe程序。我写了这个名为makeAMove()的函数。此功能应该验证玩家的移动，但它会一直显示它是一个无效的动作，并且它要求玩家重新进入。可能有一些逻辑错误，但我只是没有看到它们。有人可以帮忙吗？它是一个使用名为Cell board[ROW][COL]的对象和另一个名为Player players[2]的对象的类。标记为X或O，而iVal是电路板的初始值（电路板上的每个单元格都有1到9之间的数字）。这里包括winGame()和switchPlayer()供您参考。虽然我认为我的winGame()功能没问题，但我知道我可以使用总和做到这一点。换句话说，每个单元格具有内部标记，如-1或1，如果行，列等的总和为-3或3，则对应于玩家的内部标记将输出获胜者。如果您对如何执行此操作有任何见解，请随时分享，否则，我的winGam e就可以了。

int TicTacToe::switchPlayer(){
        if ( currentPlayer == 0)
                currentPlayer = 1;
        else if ( currentPlayer == 1 )
                currentPlayer = 0;

        return (currentPlayer);
}
//
//method to make a move
//
void TicTacToe::makeAMove(){
        char move;
        int turns = 1;
        do{
                cout << "Player " << (getCurrentPlayer() + 1) << " make a move." << endl;
                cin >> move;
                for ( int i = 0; i < ROW; i++ ){
                        for ( int j = 0; j < COL; j++ ){
                                //This is where my problem begins
                                if ( board[i][j].getiVal() == move ){
                                        board[i][j].setiVal( players[currentPlayer].getMarker());
                                        DrawBoard();
                                        switchPlayer();
                                        turns++;
                                }

                                else if ( board[i][j].getiVal() == 'X' || board[i][j].getiVal() == 'O'){
                                        cout << "Invalid move, please reenter. " << endl;
                                        cin >> move;
                                }//the else if statement always outputs:x
                        }
                }
        } while ( turns <= 9 || !winGame() );
        cout << "Congratulations " << ( getCurrentPlayer() + 1 ) << "you won the game!" << endl;
}
bool TicTacToe::winGame(){
        bool validate = false;
        int k = 0;
        for ( int i = 0; i < ROW; i++ ){
                //check column wins
                if ( board[0][i].getMarker() == board[1][i].getMarker() && board[1][i].getMarker() == board[2][i].getMarker()){
                        players[currentPlayer].setNumWin(k++);
                        validate = true;
                        break;
                }
                //check row wins
                else if ( board[i][0].getMarker() == board[i][1].getMarker() && board[i][1].getMarker() == board[i][2].getMarker()){
                        players[currentPlayer].setNumWin(k++);
                        validate = true;
                        break;
                }
        }

        if( board[0][0].getMarker() == board[1][1].getMarker() && board[1][1].getMarker() == board[2][2].getMarker()){
                players[currentPlayer].setNumWin(k++);
                validate = true;
        }
        else if ( board[0][2].getMarker() == board[1][1].getMarker() && board[1][1].getMarker() == board[2][0].getMarker()){
                players[currentPlayer].setNumWin(k++);
                validate = true;
        }

        return (validate);
}

这是程序的示例运行

    +--+--+--+
    |1 |2 |3 |
    +--+--+--+
    |4 |5 |6 |
    +--+--+--+
    |7 |8 |9 |
    +--+--+--+
    Player 1 make a move.
    1

    +--+--+--+
    |O |2 |3 |
    +--+--+--+
    |4 |5 |6 |
    +--+--+--+
    |7 |8 |9 |
    +--+--+--+
    Player 2 make a move.
    5
    Invalid move, please reenter.
    6

    +--+--+--+
    |O |2 |3 |
    +--+--+--+
    |4 |5 |X |
    +--+--+--+
    |7 |8 |9 |
    +--+--+--+
    Player 1 make a move.

Answer 1

此处的逻辑错误可以简化为以下内容：

for ( int i = 0; i < ROW; i++ ){
   for ( int j = 0; j < COL; j++ ){
         if ( board[i][j].getiVal() == move ){

               // Not important what happens here
         }
   else if ( board[i][j].getiVal() == 'X' || board[i][j].getiVal() == 'O'){
               cout << "Invalid move, please reenter. " << endl;

               // Not important what happens here
   }

这里可以使用＆＃34; Rubber Duck＆＃34;来识别逻辑错误。调试方法。

＆＃34;橡皮鸭＆＃34;调试的方法包括拿一个浮在浴缸里的橡皮鸭，把它放在显示器屏幕旁边，然后试图向橡皮鸭解释你的代码在做什么，然后你继续解释，直到找出问题自己。

在这里，您将首先向您的橡皮鸭解释您的代码将扫描井字游戏矩阵，一次一行，一次一列，从左上角开始，并在右下角结束。这就是嵌套for循环在这里做的事情。

橡皮鸭会同意，你会继续解释这段代码。

现在，您将告诉您的橡皮鸭，您的代码将继续扫描井字游戏板，直到它看到当前的move或X或O它正在扫描的位置。如果它看到X或O，您的代码会抱怨移动无效。

因此，如果第一个单元格（左上角的单元格）包含一个＆＃39; X＆＃39;或者是＆＃39; O＆＃39;此代码将在内部循环的第一次迭代中看到它，并报告错误消息，无论其他tic-tac-toe矩阵中包含什么。

此时，您已使用Rubber Duck方法在此处成功识别出逻辑错误。

Tic Tac Toe麻烦（一些逻辑错误）

1 个答案: