2 ошибки в моей игре по угадыванию случайных чисел в javascript

Результатом prompt является строка; таким образом, guess — это строка. Результатом Math.floor является число, поэтому answer является числом.

> и < — операторы принуждения: если один из операндов — число, то и другой преобразуется в число. Это означает, что guess < answer и guess > answer работают так, как вы надеетесь.

Тем не менее, вы, очевидно, слышали, как люди говорят «никогда не используйте ==, всегда используйте ===», и приняли это близко к сердцу. Причина поговорки в том, что == также является оператором принуждения, и, как следствие, иногда вещи, которые не выглядят равными, в конечном итоге оказываются равными. Однако здесь вам действительно нужно было использовать принудительное ==, потому что 1 == '1' (хотя 1 и '1' — разные типы), но строгое 1 === '1' оказывается false!

Таким образом, вы сначала проверяете, строго ли guess равно answer. Так как их типы разные, это невозможно, так что вы идете дальше. Вы проверяете, находится ли guess над ответом, затем вы проверяете, находится ли он под answer, оба они также возвращают false. Единственный оставшийся вариант — break из цикла, и "winner" никогда не будет напечатано.

Помимо этого, ваша самая большая проблема на самом деле логика. Размещение различных тестов и действий несколько сомнительно. Например, можно было бы ожидать, что проверка, равна ли guessanswer, будет рядом с проверкой, чтобы увидеть, больше она или меньше. Тест на tries был бы наиболее удобен прямо в петле while; но у вас есть бесконечный цикл и независимый тест для tries. Вы вводите первое предположение с уникальной подсказкой, но оно находится внутри бесконечного цикла, несмотря на то, что это может произойти только один раз. У вас есть функция с именем makeGuess, но эта функция выполняет два или три разных действия, ни одно из которых на самом деле не делает предположений.

Вместо того, чтобы сразу начинать работать над кодом, а затем исправлять проблемы по мере их появления, попробуйте сначала представить себе, каким будет процесс. Как бы это сделал человек? потом код записи.

Вот основная идея той же игры:

answer = imagineAnswer()
guess = askForFirstGuess()
status = "lost"
while (--tries) {
  if (guess > answer) {
    guess = askForHigherGuess()
  } else if (guess < answer) {
    guess = askForLowerGuess()
  } else {
    status = "won"
    break
  }
}
reportGameEnd(status)

1. Вы проверили tries === 0 перед tries--, поэтому у вас осталось 0 догадок. Вы должны уменьшить перед проверкой.

2. Догадка на самом деле является строкой, а не числом, поэтому guess === answer никогда не бывает правдой. Следующая итерация после того, как вы угадали правильно, попадает в else{break}, а status все еще играет. Вы либо хотите Number(guess) === answer, либо просто guess == answer.

3. По Math.floor(Math.random() * 11) вы фактически генерируете 0-10. Вы хотите Math.floor(Math.random() * 10) + 1 за 1-10. Просто помните, что Math.floor(Math.random() * n) дает вам n возможных результатов, которые представляют собой целое число в диапазоне [0,n). Затем выполните любой расчет, который вы хотите сопоставить с желаемым выходным значением.

const guessingGame = (tries) => {
    const answer = Math.floor(Math.random() * 10) + 1
    let guess = null
    let status = "playing"
    function makeGuess() {
        tries--;
        if (guess === answer) {
            status = "won"
        } else if (tries === 0) {
            status = "lost"
        } else {
            
            if (tries === 1) {
                console.info(tries + ' guess left')
            } else {
                console.info(tries + ' guesses left')
            }
        }
    }
    while (status === "playing") {
        if (!guess) {
            guess = Number(prompt('Guess a number between 1 and 10'))
            makeGuess()
        } else if (guess > answer) {
            guess = Number(prompt('Lower'))
            makeGuess()
        } else if (guess < answer) {
            guess = Number(prompt('Higher'))
            makeGuess()
        } else {
            break
        }
    }
    console.info("Game Over.  You " + status + "!  The answer was " + answer + ".")
}
guessingGame(5);