Sorry,but I still don't get it


#1


https://www.codecademy.com/courses/javascript-beginner-en-Bthev-mskY8/1/4?curriculum_id=506324b3a7dffd00020bf661

Oops, try again. There was a problem with your syntax.

var userChoice = prompt("Do you choose rock, paper or scissors?");
var computerChoice = Math.random();
if (computerChoice < 0.34) {
	computerChoice = "rock";
} else if(computerChoice <= 0.67) {
	computerChoice = "paper";
} else {
	computerChoice = "scissors";
} console.log("Computer: " + computerChoice);
var compare = function(choice1,choice2){
    if(choice1 === choice2){
        return "The result is a tie!";
    }else if(choice1 === "rock"){
      if(choice === "scissors"){
         return "rock wins";
  }
    else{
        return "paper wins";
    }
}
    else if(choice1 === "paper"){
        if(choice2 === "rock"){
            return "paper wins";
        }
    }
    else{
        return "Scissor wins";
    }
}
    };
    console.log(userChoice,computerChoice);


#2

Remove one of the closing curly braces above else.


#3

I agree with @mtf but you also need to use your compare function at the bottom rather that console.log.

Your last line should be compare(userChoice, computerChoice)


#4

And, it should not be indented. Not that it matters, but it suggests that it is part of some structure above it.


#5

This topic was automatically closed 7 days after the last reply. New replies are no longer allowed.