‘X’ changes to ‘O’ and vice versa

Question

If I click on the O’s it changes to X and vice versa.
I’m trying to prevent cheating in this game and I added a code (in JavaScript file below comment: prevents cheating) but it’s not working properly. Could someone please assist me?

https://code.sololearn.com/W31nOKfdzzj7/?ref=app

Answer 1

if (shape.innerHTML!="" || gameEnded) return;

Answer 2

if (event.target.innerHTML !="" && !gameEnded) return;

Answer 3

Triz , To improve your game

//Determining Winner
WhoWon(lastPlayer);

WhoWon = (lp) => {
if ((arr[0]===lp && arr[1]===lp && arr[2]===lp) ||
(arr[3]===lp && arr[4]===lp && arr[5]===lp) ||
(arr[6]===lp && arr[7]===lp && arr[8]===lp) ||

(arr[0]===lp && arr[3]===lp && arr[6]===lp) ||
(arr[1]===lp && arr[4]===lp && arr[7]===lp) ||
(arr[2]===lp && arr[5]===lp && arr[8]===lp) ||

(arr[0]===lp && arr[4]===lp && arr[8]===lp) ||
(arr[2]===lp && arr[4]===lp && arr[6]===lp) )
{
announcer.classList.add("you-won");
announcer.innerHTML= `Congratulations! ${lp} is the winner!`;
gameEnded=true;
}
}

https://www.sololearn.com/compiler-playground/WSogXQxGni0B

Answer 4

Good job

Answer 5

Triz
Great suggestions from SoloProg.
One suggestion is to fix the css so it would be resized when viewed on the phone. Most Sololearners use the app and the board is too big for small screens.

Answer 6

Triz , good game

body {
user-select: none; /* add it */
}

Answer 7

It works now. Thank you! Arturop SoloProg

Answer 8

SoloProg I’ll check it out. Thanks again!

Answer 9

SoloProg when i add the JavaScript to VS code, the board doesn’t load

Answer 10

Triz ,

I think you forgot to add "script" tag in html code.

Answer 11

SoloProg I have the script tag.

Answer 12

I cannot help you because i cannot try it and cant run the code

Answer 13

SoloProg i fixed it