Виталий, желаю вам удачи в освоении программирования. Игры вполне для этого годятся. Что касается вашего кода в нем есть недочеты - поэтому он и не работал в частности проблема с условиями (лишний заход).
Предлагаю другое решение в плане кода, будет хорошо если вы в нем разберетесь, если что-то не будет понятно я отвечу в комментариях.
Итак у нас есть игра. У игры есть правила давайте их запишем. Это своеобразная таблица подстановки
:
|
камень |
ножницы |
бумага |
камень |
0 |
1 |
-1 |
ножницы |
-1 |
0 |
1 |
бумага |
1 |
-1 |
0 |
Такие таблицы используются в программировании, и документации, они реально могут приносить пользу. Чтобы было удобно читать можно записать ее в коде в виде объекта. Очень важно чтобы код был читаем - Вы это позже оцените, всегда старайтесь писать так чтобы было меньше сложностей при чтении. Вложенные if
достаточно сложны для восприятия.
Вот правила игры:
const gameRules = {
"камень" : {
"камень" : 0,
"ножницы" : 1,
"бумага" : 0, //-1,
},
"ножницы" : {
"камень" : 0,//-1,
"ножницы" : 0,
"бумага" : 1,
},
"бумага" : {
"камень" : 1,
"ножницы" : 0, //-1,
"бумага" : 0,
}
}
это запись нашей таблицы истинности, для того чтобы вычислить результат мы пишем gameRules['камень']['бумага']
т.е. если у нас камень
а у соперника бумага
то мы ничего не заработаем (или -1 бал - смотря как считать) тоже самое делаем со стороны компьютера.
В итоге остальной код выглядит так:
function getComputer() {
const random = Math.random();
if (random <= 0.33){
computer = 'камень';
} else if (random <= 0.64){
computer = 'ножницы';
} else {
computer = 'бумага';
}
return computer;
}
while (true){
const number = prompt('Введите один из вариантов\n1. камень \n2. ножницы\n3. бумага');
if (!number) {
break;
}
const user = ['камень', 'ножницы', 'бумага'][parseInt(number)-1];
computer = getComputer();
compScore += gameRules[computer][user] // считаем баллы компьютера
userScore += gameRules[user][computer] // считаем баллы пользователя
console.log(`Пользователь ${user} (${userScore})\nКомпьютер ${computer}(${compScore})`)
}
По объему строк практически то же самое, но такой код при определенной практике читается проще, легче контролируется и меняется. Конечно этот код не идеален и его тоже можно улучшить, в частности слова в коде обычно подменяют константами чтобы не наделать опечаток, и получение рандомного числа можно записать гораздо короче.
По поводу возможных улучшений вашего последнего варианта (разумеется нужно еще ввести проверку ввода пользователя, ну и это не предел улучшений):
const STONE = 0;
const SCISSORS = 1;
const PAPER = 2;
let userScore = 0;
let computerScore = 0;
function getComputerChoise() {
return Math.floor(Math.random() * 3);
}
function getUserChoise (){
const userChoise = prompt('Введите один из вариантов\n1. камень \n2. ножницы\n3. бумага');
if (userChoise === null) {
return userChoise;
}
return parseInt(userChoise) - 1; /* от 0 до 2*/
}
function gameStep() {
const computerChoise = getComputerChoise();
const userChoise = getUserChoise();
if (!userChoise) return false;
if (computerChoise === STONE && userChoise === PAPER
|| computerChoise === SCISSORS && userChoise === STONE
|| computerChoise === PAPER && userChoise === SCISSORS)
{
userScore++
} else if (computerChoise === STONE && userChoise === SCISSORS
|| computerChoise === SCISSORS && userChoise === PAPER
|| computerChoise === PAPER && userChoise === STONE){
computerScore++
}
return true;
}
function showResults() {
let result = 'Счет равен';
if (userScore > computerScore){
result ='Пользователь выиграл'
}else if(userScore < computerScore){
result = 'Компьютер выиграл';
}
console.log('Пользователь ' + userScore + '\nКомпьютер ' + computerScore + '\n' + result);
}
while (gameStep()) {}
showResults();
if (computer === 'камень' && user === 3) ...