Was hoping someone could help me make changes to this. I don't really want to redo the whole program. Here's what it's asking.

Write a program that allows two players to play a game of tic tac toe. Use a two-dimensional char array with three rows and three columns as the game board. Be sure to pass arrays as parameters to functions. The program should be running on loops. Each element of the array should be initialized with an asterisk (*)

#include <iostream>
using namespace std;

char square[10] = {'o','1','2','3','4','5','6','7','8','9'};

int checkwin();
void board();

int main()
{
    int player = 1,i,choice;

    char mark;
    do
    {
        board();
        player=(player%2)?1:2;

        cout << "Player " << player << ", enter a number:  ";
        cin >> choice;

        mark=(player == 1) ? 'X' : 'O';

        if (choice == 1 && square[1] == '1')

            square[1] = mark;
        else if (choice == 2 && square[2] == '2')

            square[2] = mark;
        else if (choice == 3 && square[3] == '3')

            square[3] = mark;
        else if (choice == 4 && square[4] == '4')

            square[4] = mark;
        else if (choice == 5 && square[5] == '5')

            square[5] = mark;
        else if (choice == 6 && square[6] == '6')

            square[6] = mark;
        else if (choice == 7 && square[7] == '7')

            square[7] = mark;
        else if (choice == 8 && square[8] == '8')

            square[8] = mark;
        else if (choice == 9 && square[9] == '9')

            square[9] = mark;
        else
        {
            cout<<"Invalid move ";

            player--;
            cin.ignore();
            cin.get();
        }
        i=checkwin();

        player++;
    }while(i==-1);
    board();
    if(i==1)

        cout<<"==>\aPlayer "<<--player<<" win ";
    else
        cout<<"==>\aGame draw";

    cin.ignore();
    cin.get();
    return 0;
}

//RETURN GAME STATUS

int checkwin()
{
    if (square[1] == square[2] && square[2] == square[3])

        return 1;
    else if (square[4] == square[5] && square[5] == square[6])

        return 1;
    else if (square[7] == square[8] && square[8] == square[9])

        return 1;
    else if (square[1] == square[4] && square[4] == square[7])

        return 1;
    else if (square[2] == square[5] && square[5] == square[8])

        return 1;
    else if (square[3] == square[6] && square[6] == square[9])

        return 1;
    else if (square[1] == square[5] && square[5] == square[9])

        return 1;
    else if (square[3] == square[5] && square[5] == square[7])

        return 1;
    else if (square[1] != '1' && square[2] != '2' && square[3] != '3' 
                    && square[4] != '4' && square[5] != '5' && square[6] != '6' 
                  && square[7] != '7' && square[8] != '8' && square[9] != '9')

        return 0;
    else
        return -1;
}


//FUNCTION TO DRAW BOARD

void board()
{
    system("cls");
    cout << "\n\n\tTic Tac Toe\n\n";

    cout << "Player 1 (X)  -  Player 2 (O)" << endl << endl;
    cout << endl;

    cout << "     |     |     " << endl;
    cout << "  " << square[1] << "  |  " << square[2] << "  |  " << square[3] << endl;

    cout << "_____|_____|_____" << endl;
    cout << "     |     |     " << endl;

    cout << "  " << square[4] << "  |  " << square[5] << "  |  " << square[6] << endl;

    cout << "_____|_____|_____" << endl;
    cout << "     |     |     " << endl;

    cout << "  " << square[7] << "  |  " << square[8] << "  |  " << square[9] << endl;

    cout << "     |     |     " << endl << endl;
}

Was hoping someone could help me make changes to this. I don't really want to redo the whole program. Here's what it's asking.

Write a program that allows two players to play a game of tic tac toe. Use a two-dimensional char array with three rows and three columns as the game board. Be sure to pass arrays as parameters to functions. The program should be running on loops. Each element of the array should be initialized with an asterisk (*)

You haven't told us what's wrong with the program. Does it not compile? Does it not run? Does it crash? If so, under what circumstances? Does it run without crashing, but there is something else wrong? Etcetera. Don't make us guess.

That said, right off the bat, you are not following the specification:

Use a two-dimensional char array with three rows and three columns as the game board.

You have a ONE-DIMENSIONAL array with TEN elements, not a 3x3 array. You are also not passing that array as a parameter, as required.

Be sure to pass arrays as parameters to functions.

Your array is a global variable. My guess is that they want you to make it a non-global variable and pass it. Google "passing 2 dimensional array C++" for tutorials on how to do this.

This article has been dead for over six months. Start a new discussion instead.