Как сделать игру Камень-Ножницы-Бумага на js?

Question

[Иван Медведев]

Задание:
Сделайте игру "камень-ножницы-бумага". Пользователь вводит свой вариант через prompt (1 или 2 или 3), программа генерирует свой вариант через Math.random() и выводит через alert. Следующий alert выводит имя победителя или "ничья"
Используем только условия и массивы (если необходимо). Циклы не применять.

Я написал такой вот код. Но при любых комбинациях чисел программа выводит последний алерт из условия (The friendship won!)

let userValue = Number(prompt("Enter the number from 1 to 3. 1 is stone, 2 - scissors, 3 - paper"))
function progValue(min, max) {
    return Number(Math.floor(Math.random() * (max - min + 1)) + min)
}
alert(`The programm number is ${progValue(1, 3)}`)
if (userValue === 1 && progValue === 2 || userValue === 1 && progValue === 3 || userValue === 2 && progValue === 3) {
    alert("Yeah! You won!")
}
else if (userValue === 3 && progValue === 1 || userValue === 3 && progValue === 2 || userValue === 2 && progValue === 1) {
    alert("Sorry! You lost")
} else {
    alert("The friendship won!")
}

Что не так?

Comments

[Михаил]

progValue === 2 если функция равняется 2м
найс

[Иван Медведев]

Михаил, что вы имели ввиду?

[Даша Циклаури]

Иван Медведев, у тебя в progValue записан указатель на виртуальную функцию. он никак не может равняться 2

[Роман]

Иван Медведев, то, что в условии надо вызывать функцию. например так:
progValue(1, 3) === 2

а по сему возникают серьезные сомнения в вашем авторстве

этого кода

function progValue(min, max) {
return Number(Math.floor(Math.random() * (max - min + 1)) + min)
}
alert(`The programm number is ${progValue(1, 3)}`)

потому, что вы совсем не понимаете как это работает.

[Иван Медведев]

Роман, код мой. Я просто учу JS и естественно делаю ошибки))

Answer 1

[Иван Медведев]

Разобрался! Как я понял, мне не обязательно было использовать функцию. И условия я немного упростил. В общем, в итоге, получился вот такой рабочий код:

let userValue = Number(prompt("Enter the number from 1 to 3. 1 is stone, 2 - scissors, 3 - paper"))
let progValue = Math.ceil(Math.random() * 3)
alert(`The programm number is ${progValue}`)
if (userValue === 1 && progValue > 1 || userValue === 2 && progValue === 3) {
    alert("Yeah! You won!")
}
else if (userValue === 3 && progValue < 3 || userValue === 2 && progValue === 1) {
    alert("Sorry! You lost")
} else {
    alert("The friendship won!")
}

Answer 2

[Karpion]

Мне кажется, userValue содержит строку, а не число. Соответственно, они никогда не м.б. === числу. Наверно, там надо ==, а не === ...

Ещё в операторах if - надо расставить скобки. А то полагаться на приоритет операций - так себе идея.

И ещё есть хорошая оптимизация: надо из userValue вычесть progValue. И анализировать разность оператором switch; или есть альтернативы...

Comments

[Иван Медведев]

- userValue точно не строка, так как я изначально промпт перевожу в число, еще на этапе присвоения переменной
- в чем приоритет, если там только операции сравнения записаны через условный оператор
- не знаю, зачем там вычитание

[Karpion]

Иван Медведев, Про userValue - Вы правы, там Number() .

Приоритет м.б. в чём угодно. Например, я навскидку не готов сказать, у кого приоритет выше - у сравнений, у "и" или у "или". Поставьте скобки, не жадничайте, они бесплатны.

Рассмотрим Вашу программу: userValue === 1 && progValue > 1 - вот что это значит? Тут нельзя "больше одного", тут можно только "равно двум"! Потому что "равно трём" - это совсем наоборот!

Рассмотрим разные варианты:
{1:1}, {2:2}, {3:3} = ничья. Это легко детектируется сравнением на равенство.
{1:2}, {2:3}, {3:1} = победа первого игрока. Если вычесть первое из второго - получается 1,1,-2.
{1:3}, {2:1}, {3:2} = победа второго игрока. Если вычесть первое из второго - получается 2,-1,-1.

Значит, берём разность. Если она меньше нуля - то прибавим к ней три. В результате получим: 0 = ничья; иначе - номер победителя.
Правда, красиво? Это математика, учите её!

Answer 3

[Александр]

Результат функции progValue надо бы сохранять и уже его использовать в выводе и сравнениях.

Comments

[Karpion]

А разве это - функция?

[Александр]

Karpion,