Переполнение стека в js

[Полбутылки сентября]

Занимаюсь js на codecademy. Делаю задание по написанию "камень, ножницы, бумага". И на вот этот код он мне пишет Maximum call stack size exceeded javasсript. Однако ж если я закоменчу последний if, где choice1=scissors, то всё окей. Чего я не вижу?

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";
}
var compare = function (choice1, choice2) {
    if (choice1 === choice2) {
        return "The result is a tie!";
    }
    if (compare(choice1) === "rock") {
        if (compare(choice2) === "scissors") {
            return "rock wins";
        }
        else {
            return "paper wins";
        }
    }
    if (compare(choice1) === "paper") {
        if (compare(choice2) === "scissors") {
            return "scissors wins";
        }
        else {
            return "paper wins";
        }
    }
    if (compare(choice1) === "scissors") {
        if (compare(choice2) === "rock") {
            return "rock wins";
        }
        else {
            return "scissors wins";
        }
    }
}
compare(userChoice, computerChoice);

@темы: Вопрос, Java script

Comments

[https]

Полбутылки сентября, Думаю, дело в вызовах compare(choice1) и compare(choice2) - они приводят к бесконечной рекурсии. Вы уверены, что хотите их вызвать а не сделать просто "choiceX === "xxxx" ?

И вот тут, кажется, ещё ошибка:

if (compare(choice1) === "rock") {
if (compare(choice2) === "scissors") {
return "rock wins";
}
else {
return "paper wins";
}
}

"scissors win" должно быть, очеидно.

"scissors win" должно быть, очеидно.

разве? бумага заворачивает камень с вариантами ответов там всё ок)

[https]

ninelya, да, в этом не права, логику невнимательно просмотрела. Прошу прощения

[Полбутылки сентября]

https, действительно перемудрила. Спасибо.

[mikluho]

Да, это не тот случай, когда нужна рекурсия.

Я так понимаю, что должно быть вот так:

var compare = function (choice1, choice2) {
    if (choice1 === choice2) {
        return "The result is a tie!";
    }
    if (choice1 === "rock") {
        if (choice2 === "scissors") {
            return "rock wins";
        }
        else {
            return "paper wins";
        }
    }
    if (choice1 === "paper") {
        if (choice2 === "scissors") {
            return "scissors wins";
        }
        else {
            return "paper wins";
        }
    }
    if (choice1 === "scissors") {
        if (choice2 === "rock") {
            return "rock wins";
        }
        else {
            return "scissors wins";
        }
    }
}

[Полбутылки сентября]

mikluho, таки да, именно так оно и заработало. Теперь буду знать, спасибо.

[https]

Полбутылки сентября,