odd and even mouse clicks

[Fillip F]

We've been tasked to make a tic-tac-toe game but it seems like were all getting stuck on alternating between X and O and instead each click outputs and X in a box.

My idea was to make an if statement saying each even number of clicks will output a X and each odd number of clicks will output an O.

I've tried different things but i'm stuck on this.

#include <GameDev2D.h>

//Function prototypes
void Init();
void Shutdown();
void Update(double delta);
void Draw();
int MouseClicked(float x, float y);

//variables
int state[9] = {};
int Click[9] = {};


//Entry point to the application
int WINAPI WinMain(HINSTANCE aCurrentInstance, HINSTANCE aPreviousInstance, LPSTR aCommandLine, int aCommandShow)
{
    //Run GameDev2D, pass in the Init, Shutdown, Update and Draw methods
    GameDev2D::Run(Init, Shutdown, Update, Draw);
    return 0;
}

void Init()
{
    GameDev2D::RegisterLeftMouseButtonPressedCallback(MouseClicked);
    GameDev2D::LoadTexture("x");
    GameDev2D::LoadTexture("o");

}

void Shutdown()
{
    GameDev2D::UnloadTexture("x");
    GameDev2D::UnloadTexture("o");

}

void Update(double aDelta)
{
   
}

void Draw()
{
    //Guide lines
    GameDev2D::DrawRectangle(256.0f, 0.0f, 10.0f, 768.0f, 0.0f, GameDev2D::Color::BlackColor(), true);
    GameDev2D::DrawRectangle(512.0f, 0.0f, 10.0f, 768.0f, 0.0f, GameDev2D::Color::BlackColor(), true);
    GameDev2D::DrawRectangle(0.0f, 256.0f, 768.0f, 10.0f, 0.0f, GameDev2D::Color::BlackColor(), true);
    GameDev2D::DrawRectangle(0.0f, 512.0f, 768.0f, 10.0f, 0.0f, GameDev2D::Color::BlackColor(), true);
	


	//Drawing in diffrent boxes
	if (state[0] == 1)
	{
		GameDev2D::DrawTexture("x", 0.0f, 0.0f, 0.0f); // row 1 / 1 
	}
	 if (state[0] == 2)
	{
		GameDev2D::DrawTexture("o", 0.0f, 0.0f, 0.0f);
	}

	if (state[1] == 1)
	{
		GameDev2D::DrawTexture("x", 256.0f, 0.0f, 0.0f); // row 1 / 2
	}
	if (state[1] == 2)
	{
		GameDev2D::DrawTexture("o", 256.0f, 0.0f, 0.0f);
	}

	if (state[2] == 1)
	{
		GameDev2D::DrawTexture("x", 512.0f, 0.0f, 0.0f); // row 1 / 3
	}
	if (state[2] == 2)
	{
		GameDev2D::DrawTexture("o", 512.0f, 0.0f, 0.0f);
	}

	if (state[3] == 1)
	{
		GameDev2D::DrawTexture("x", 0.0f, 256.0f, 0.0f); // row 2 / 1
	}
	if (state[3] == 2)
	{
		GameDev2D::DrawTexture("o", 0.0f, 256.0f, 0.0f);
	}

	if (state[4] == 1)
	{
		GameDev2D::DrawTexture("x", 256.0f, 256.0f, 0.0f); // row 2 / 2
	}
	if (state[4] == 2)
	{
		GameDev2D::DrawTexture("o", 256.0f, 256.0f, 0.0f);
	}

	if (state[5] == 1)
	{
		GameDev2D::DrawTexture("x", 512.0f, 256.0f, 0.0f); // row 2 / 3
	}
	if (state[5] == 2)
	{
		GameDev2D::DrawTexture("o", 512.0f, 256.0f, 0.0f);
	}

	if (state[6] == 1)
	{
		GameDev2D::DrawTexture("x", 0.0f, 512.0f, 0.0f); // row 3 / 1
	}
	if (state[6] == 2)
	{
		GameDev2D::DrawTexture("o", 0.0f, 512.0f, 0.0f);
	}

	if (state[7] == 1)
	{
		GameDev2D::DrawTexture("x", 256.0f, 512.0f, 0.0f); // row 3 / 2
	}
	if (state[7] == 2)
	{
		GameDev2D::DrawTexture("o", 256.0f, 512.0f, 0.0f);
	}

	if (state[8] == 1)
	{
		GameDev2D::DrawTexture("x", 512.0f, 512.0f, 0.0f); // row 3 / 3
	}
	if (state[8] == 2)
	{
		GameDev2D::DrawTexture("o", 512.0f, 512.0f, 0.0f);
	}

}

int MouseClicked(float x, float y) 
{
    if (x > 0 && x < 256 && y > 0 && y < 256)
    {
		state[0] = 1 || 2;
    }

    if (x > 256 && x < 512 && y > 0 && y < 256)
    {
        if (state[1] == 0)
        {
			state[1] = 1 || 2;
        }
    }

     if (x > 512 && x < 768 && y > 0 && y < 256)
    {
         if (state[2] == 0)
         {
			 state[2] = 1 || 2;
         }
    }

     if (x > 0 && x < 256 && y > 256 && y < 512)
     {
         if (state[3] == 0)
         {
			 state[3] = 1 || 2;
         }
     }

     if (x > 256 && x < 512 && y > 256 && y < 512)
     {
         if (state[4] == 0)
         {
			 state[4] = 1 || 2;
         }
     }

     if (x > 512 && x < 768 && y > 256 && y < 512)
     {
         if (state[5] == 0)
         {
			 state[5] = 1 || 2;
         }
     }

     if (x > 0 && x < 256 && y > 512 && y < 768)
     {
         if (state[6] == 0)
         {
			 state[6] = 1 || 2;
         }
     }

     if (x > 256 && x < 512 && y > 512 && y < 768)
     {
         if (state[7] == 0)
         {
			 state[7] = 1 || 2;

         }
     }

     if (x > 512 && x < 768 && y > 512 && y < 768)
     {
         if (state[8] == 0)
         {
			 state[8] = 1 || 2;
         }
     }
}

[Golden Lizard]

Those nested ifs are eyesore as hell. Here are my 2 cents, don't count odd and even clicks, just have a boolean value you can use as a switch. Also use a function to divide the board, rather than just spamming ifs.

bool isXTurn = true; //start with X

//inside your game loop
   
uint8_t chosenSquare = GetSelectedSquare();     //some function that gets player's input (i.e in which square he clicked)
UpdateBoard(chosenSquare, isXTurn ? 'X' : 'O'); //check if is X's turn, if not use O
isXTurn = !isXTurn;                             //alternate turns

[Fillip F]

The way we were taught to do it today was with all the if's, i agree its not very efficient and i wish we had gotten an alternative but this is all i know on how to do this.

No one was able to get the mouse input to alternate between X and O.

As far as your response goes i'm not sure where im implementing this. I'm pretty sure the Bool function im adding to the variable list but the rest is a blur since this is 1st year programming we haven't seen extensive coding yet like uint8_t.