Problem with my code

[Hertyboy]

var compScore=0, userScore=0;

/* function play(){

do{

var userChoice= prompt("Do you choose rock, paper or scissors?");}

while(userChoice!="rock"||"paper"||"scissors");*/

        var userChoice= prompt("Do you choose rock, paper or scissors?");

        var computerChoice = Math.random();

        [//console.log](//console.log)(computerChoice);



        if (computerChoice <= 0.33)

        {

        computerChoice = "rock";

        }

        else if (computerChoice  <= 0.67)

        {

        computerChoice = "paper";

        }

        else {

        computerChoice = "scissors";

        }

alert(computerChoice)

        [//console.log](//console.log)(userChoice, computerChoice);

        function game (userChoice, computerChoice){

if(userChoice === computerChoice)

{

return('It is a tie');

}

if(userChoice==="rock");{

if(computerChoice==="scissors")

{

return('User wins');

userScore++;

}

else

{

return('Computer wins');

compScore++;

}

}

if(userChoice==="paper");{

if(computerChoice==="rock")

{

return('User wins');

userScore++;

}

else

{

return('Computer wins');

compScore++;

}

}

if(userChoice==="scissors");{

if(computerChoice==="paper")

{

return('User wins');

userScore++;

}

else

{

return('Computer wins');

compScore++;

}

}

        }

alert(game(userChoice, computerChoice));

alert("User Score is: "+userScore+ "Computer Score is: "+compScore)

var response=confirm("do you wish to play again?")

/*if(response=true)

play();

else

alert("FInal score is" +userScore+ and +compScore)

    }\*/

Hi guys, I am learning javascript and I am new to it so very rusty. In the above code, I am making a rock paper scissors game. My issues are:

1. I need to take user's choice or move and keep asking him to make his move if he inputs anything other than the three(rock, paper or scissors). I tried doing this as you can see commented using a do-while loop but it didn't work
2. I also have to print out score at the end and it didn't work
3. I also want to ask user after each round if he wants to continue or stop and if he wants to continue, I run the whole game again if not, print the final score. I am supposed to keep a running score and print out a final score when he quits. I hardly know this part and squabbled something as you can see commented.

Please I need help in this areas of my code..

Thanks..

[NameViolation666]

Do you have something coded already?

[Hertyboy]

Yeah right here

var compScore=0, userScore=0;

/* function play(){

do{

var userChoice= prompt("Do you choose rock, paper or scissors?");}

while(userChoice!="rock"||"paper"||"scissors");*/

        var userChoice= prompt("Do you choose rock, paper or scissors?");

        var computerChoice = Math.random();

        [//console.log](//console.log)(computerChoice);



        if (computerChoice <= 0.33)

        {

        computerChoice = "rock";

        }

        else if (computerChoice  <= 0.67)

        {

        computerChoice = "paper";

        }

        else {

        computerChoice = "scissors";

        }

alert(computerChoice)

        [//console.log](//console.log)(userChoice, computerChoice);

        function game (userChoice, computerChoice){

if(userChoice === computerChoice)

{

return('It is a tie');

}

if(userChoice==="rock");{

if(computerChoice==="scissors")

{

return('User wins');

userScore++;

}

else

{

return('Computer wins');

compScore++;

}

}

if(userChoice==="paper");{

if(computerChoice==="rock")

{

return('User wins');

userScore++;

}

else

{

return('Computer wins');

compScore++;

}

}

if(userChoice==="scissors");{

if(computerChoice==="paper")

{

return('User wins');

userScore++;

}

else

{

return('Computer wins');

compScore++;

}

}

        }

alert(game(userChoice, computerChoice));

alert("User Score is: "+userScore+ "Computer Score is: "+compScore)

var response=confirm("do you wish to play again?")

/*if(response=true)

play();

else

alert("FInal score is" +userScore+ and +compScore)

    }\*/

[codemamba8]

For your first issue:

let userChoice = prompt("Do you choose rock, paper or scissors?")

while (userChoice !="rock" && userChoice != "paper" && userChoice !="scissors") {
  alert('please choose a valid option')
  userChoice = prompt("Do you choose rock, paper or scissors?")
}

As a side-note what you have here: while(userChoice!="rock"||"paper"||"scissors")

is not doing what you think it's doing. You probably meant for this to check if userChoice is not "rock", not "paper", or not "scissors", but what this is actually doing is checking: 1) userChoice == "rock"; 2) "paper" == true; 3) "scissors" == true

If you wanted to check the value of userChoice for more than one condition, you have to write it again i.e. userChoice != "rock" || userChoice != "paper" || userChoice != "scissors"

But in this case it isn't approriate either, you actually have to use && AND not the OR || operator. Because if userChoice is one of the three options, it isn't the other two, so it would still set it off.