0

我写了一个 C++ 井字游戏。它可以工作,但是我认为如果我向用户展示电路板在每转一圈后的外观会更好。我让董事会出现,但它显示了默认董事会。我尝试输入代码以显示带有 x 或 o 的板,但我只是得到一个错误,说 squareChoice 未在此范围内声明。在 while 之后,我的两个 if 状态也出现错误(在 if 之前声明预期的 unqualiftied-id)。请帮忙,我将不胜感激。这是我的代码:

/*
Sophia Ali
   Template for TicTacToe.cpp (CS-509 Assignment 5)
can't get program to store x and o in the board
*/


#include<iostream>
#include <cstdlib>
using namespace std;

/*
   Game status enumeration
*/
enum Status { WIN, DRAW, CONTINUE, QUIT };


/*
   Function prototypes
*/
// showBoard: Show current state of board
void showBoard( const char board[], int boardSize );
// checkGameState: Returns WIN or CONTINUE
Status checkGameState( const char board[] );
int getHumanSquare( const char board[] );
int getComputerSquare( const char board[] );
// checkBadSquare: Checks to see if a chosen square is already taken; returns
//                 true if already taken; used by getHumanSquare and
//                 getComputerSquare functions above.
bool checkBadSquare( const char board[], int squareNum );
int getrandint( int min, int max );




int main()
{
    char board[] = "123456789";   // 10 element char board
    const int boardSize = 10;
    Status gameState = CONTINUE;
    int gametype, squareChoice, turnNum = 0;
    char currentSymbol;           // 'o' or 'x'



    cout << "\n This is a Tic Tac Toe program. Choose the type of game: "
         << "\n (1) human o vs. human x    (2) human o vs. dumb computer x"
         << "\n\n -> ";
    cin  >> gametype;


    /* Show the current state of Tic Tac Toe board. */
    cout << gameState;
    cout << "\n\n";




    /*
       Main game loop
    */
    while ( gameState == 2 )
    {
        /* Increment turnNum by 1. */
        turnNum++;


        /* If turnNum equal to 10
              Set gameState to DRAW.
              Break out of while loop. */
        if ( turnNum == 10 )
        {
            gameState = DRAW;
            break;
        }
        /* If we are on an odd-numbered turn
              Print "It's o's turn."
              Set currentSymbol to 'o'.
              Call getHumanSquare function to get squareChoice.*/
        if ( turnNum%2 != 0)
        {
            cout << "It's o's turn.";
            currentSymbol = 'o';
            cout << "\n\n";

            getHumanSquare(board);
            showBoard( board, boardSize );
        }

        /* Else (we are on an even-numbered turn)
            Print "It's x's turn."
            Set currentSymbol to 'x'. */
        else
        {
            cout << "It's x's turn.";
            currentSymbol = 'x';
            cout << "\n\n";
            showBoard( board, boardSize );

        }


        /*   If the gametype is 1 (human vs. human)
              Call getHumanSquare function to get squareChoice.*/
        if ( gametype == 1 )
        {
            getHumanSquare(board);


        }


        /* Else (gametype is 2 (human vs. computer))
           Call getComputerSquare function to get squareChoice. */
        else

            getComputerSquare(board);



        /* If squareChoice is -1 (human player quit)
              Set gameState to QUIT.*/
        if ( squareChoice == -1 )
        {
            gameState = QUIT;
        }

        /* Else
           Insert currentSymbol into board at (squareChoice - 1).
           Show the current state of the Tic Tac Toe board.
           Call checkGameState function to determine the gameState. */
        else
        {
            if (squareChoice == 1 && board[1] == '1')
            board[1] = currentSymbol;
            else if (squareChoice == 2 && board[2] == '2')
            board[2] = currentSymbol;
            else if (squareChoice == 3 && board[3] == '3')
            board[3] = currentSymbol;
            else if (squareChoice == 4 && board[4] == '4')
            board[4] = currentSymbol;
            else if (squarechoice == 5 && board[5] == '5')
            board[5] = currentSymbol;
            else if (squarechoice == 6 && board[6] == '6')
            board[6] = currentSymbol;
            else if (squarechoice == 7 && board[7] == '7')
            board[7] = currentSymbol;
            else if (squarechoice == 8 && board[8] == '8')
            board[8] = currentSymbol;
            else if (squarechoice == 9 && board[9] == '9')
            board[9] = currentSymbol;
            else
            {
                cout <<"Invalid move";
                turnNum--;
            }
        }

            checkGameState(board);

        }

}// end while


    /* If gameState is WIN
          print "Player " currentSymbol " is the winner." */
    if ( gameState == WIN)
        cout << "Player " << currentSymbol << " is the winner.";

    /* If gameState is DRAW
          print "It's a draw." */
    if ( gameState == DRAW )
        cout << "It's a draw.";

    return 0;
}

/////////////////////////////////////////////////////////////////////

void showBoard( const char board [], int size )
{
    cout << endl;

    for ( int i = 0; i < size ; i++ )
    {
        cout << board[ i ] << " ";
        if ( ( i + 1 ) % 3 == 0 )
            cout << endl;
    }

    cout << endl;
}

/////////////////////////////////////////////////////////////////////

Status checkGameState( const char board[] )
{
    // Board       Array
    //
    // 1 2 3       0 1 2
    // 4 5 6  -->  3 4 5
    // 7 8 9       6 7 8
    //
    // Diagonal winners
    if ( board[ 0 ] == board[ 4 ] && board[ 0 ] == board[ 8 ] )
        return WIN;
    else if ( board[ 2 ] == board[ 4 ] && board[ 4 ] == board[ 6 ] )
        return WIN;
    // Horizontal winners
    else if ( board[ 0 ] == board[ 1 ] && board[ 1 ] == board[ 2 ] )
        return WIN;
    else if ( board[ 3 ] == board[ 4 ] && board[ 4 ] == board[ 5 ] )
        return WIN;
    else if ( board[ 6 ] == board[ 7 ] && board[ 7 ] == board[ 8 ] )
        return WIN;
    // Vertical winners
    else if ( board[ 0 ] == board[ 3 ] && board[ 3 ] == board[ 6 ] )
        return WIN;
    else if ( board[ 1 ] == board[ 4 ] && board[ 4 ] == board[ 7 ] )
        return WIN;
    else if ( board[ 2 ] == board[ 5 ] && board[ 5 ] == board[ 8 ] )
        return WIN;
    else
        // No one has won yet
        return CONTINUE;
}

/////////////////////////////////////////////////////////////////////

int getHumanSquare( const char board[] )
{
    int squareNum;

    cout << "\n Input the number of an empty square: (-1 to quit) ";
    cin  >> squareNum;

    while ( checkBadSquare( board, squareNum ) == true )
    {
        cout << "\n Bad input. Choose another square: ";
        cin >> squareNum;
    }

    return squareNum;
}

/////////////////////////////////////////////////////////////////////

int getComputerSquare( const char board[] )
{
    int squareNum;

    squareNum = getrandint( 1, 9 );

    while ( checkBadSquare( board, squareNum ) == true )
    {
        squareNum = getrandint( 1, 9 );
    }

    return squareNum;
}

/////////////////////////////////////////////////////////////////////

bool checkBadSquare( const char board[], int squareNum )
{
    int realSquareNum = squareNum - 1; // count from 0

    if ( squareNum == -1 )
        return false;  // Let quit code pass as a valid square
    else if ( squareNum > 9 )
        return true;   // Square numbers out of range are invalid
    else if ( board[ realSquareNum ] == 'o' || board[ realSquareNum ] == 'x' )
        return true;   // Already taken squares are invalid
    else
        return false;  // Valid square number
}

/////////////////////////////////////////////////////////////////////

int getrandint( int min, int max )
{

    int scale, shift;
    scale = max - min + 1;
    shift = min;
    return rand() % scale + shift;
}
4

2 回答 2

1

squareChoice 的拼写在大多数情况下都是错误的if statement:应该是squareChoice而不是squarechoice

所以替换:

else
        {
            if (squareChoice == 1 && board[1] == '1')
            board[1] = currentSymbol;
            else if (squareChoice == 2 && board[2] == '2')
            board[2] = currentSymbol;
            else if (squareChoice == 3 && board[3] == '3')
            board[3] = currentSymbol;
            else if (squareChoice == 4 && board[4] == '4')
            board[4] = currentSymbol;
            else if (squarechoice == 5 && board[5] == '5')
            board[5] = currentSymbol;
            else if (squarechoice == 6 && board[6] == '6')
            board[6] = currentSymbol;
            else if (squarechoice == 7 && board[7] == '7')
            board[7] = currentSymbol;
            else if (squarechoice == 8 && board[8] == '8')
            board[8] = currentSymbol;
            else if (squarechoice == 9 && board[9] == '9')
            board[9] = currentSymbol;
            else
            {
                cout <<"Invalid move";
                turnNum--;
            }
        }

经过

 else
        {
            if (squareChoice == 1 && board[1] == '1')
                board[1] = currentSymbol;
            else if (squareChoice == 2 && board[2] == '2')
                board[2] = currentSymbol;
            else if (squareChoice == 3 && board[3] == '3')
                board[3] = currentSymbol;
            else if (squareChoice == 4 && board[4] == '4')
                board[4] = currentSymbol;
            else if (squareChoice == 5 && board[5] == '5')/*SPELLING MISTAKE*/
                board[5] = currentSymbol;
            else if (squareChoice == 6 && board[6] == '6')
                board[6] = currentSymbol;
            else if (squareChoice == 7 && board[7] == '7')
                board[7] = currentSymbol;
            else if (squareChoice == 8 && board[8] == '8')
                board[8] = currentSymbol;
            else if (squareChoice == 9 && board[9] == '9')
                board[9] = currentSymbol;
            else
            {
                cout <<"Invalid move";
                turnNum--;
            }
        }

一个额外的右大括号“ }”在主函数中。删除以下行以修复它。

}// end while.  THIS IS NOT NEEDED.REMOVE THIS LINE FROM YOUR CODE.

我已经修复了编译错误。演示。仍然存在一些逻辑和运行时错误。

UPD:根据@Ben Voigt 评论,运行时错误可能是由于板 [9] 超出范围。阅读您的以下评论后:// 10 element char board,我认为

char board[] = "123456789"; 

应替换为:

 char board[] = "0123456789"; 

即使在修复它之后,仍然存在一些逻辑和运行时错误。

于 2013-03-31T22:30:25.700 回答
0

大长if已经被指出有问题。您可以用更短的测试替换它:

if (squareChoice >= 1 && squareChoice <= 9 && board[squareChoice-1] == '0' + squareChoice) {
    board[squareChoice-1] = currentSymbol;
}
else {
    cout <<"Invalid move";
    turnNum--;
}
于 2013-03-31T22:39:18.787 回答