Я попытался решить проблему, которую нашел в Интернете. Я успешно решил проблему, но есть небольшая ошибка, которую я не смог решить.
Проблема
Write a guessing game where the user has to guess a secret number. After every guess the program tells the user whether their number was too large or too small. At the end the number of tries needed should be printed.
Вот мой код:
// Generate a random number between 1 and 100
var num = Math.floor(Math.random() * (100)) + 1;
var running = true;
while(running) {
var tries = 1;
var input = prompt("Take a guess");
if (input == num) {
console.info("Correct!");
console.info("Number of tries: " + tries);
running = false;
}else if (input > num) {
console.info("Too big");
}else if (input < num) {
console.info("Too small");
}
tries++;
}
Bug
Even if the user takes more than 1 try, the program still says,
Number of tries: 1
Объясните, пожалуйста, что я делаю не так.
Спасибо.



![Безумие обратных вызовов в javascript [JS]](https://i.imgur.com/WsjO6zJb.png)


Вы повторно инициализируете tires на каждой итерации цикла while:
while(running) {
var tries = 1;
...
}
Попробуйте выполнить инициализацию вне вашего цикла.
var num = Math.floor(Math.random() * (100)) + 1;
var running = true;
var tries = 1;
while(running) {
var input = prompt("Take a guess");
if (input == num) {
console.info("Correct!");
console.info("Number of tries: " + tries);
running = false;
}else if (input > num) {
console.info("Too big");
}else if (input < num) {
console.info("Too small");
}
tries++;
}
Ваши переменные декали tries внутри корпуса while. Итак, при любой итерации цикла эта переменная получает значение 1. Чтобы решить эту проблему, вы должны объявить и инициализировать переменную вне цикла:
// Generate a random number between 1 and 100
var num = Math.floor(Math.random() * 100) + 1;
var running = true;
var tries = 1;
while (running) {
var input = prompt("Take a guess");
if (input == num) {
console.info("Correct!");
console.info("Number of tries: " + tries);
running = false;
} else if (input > num) {
console.info("Too big");
} else if (input < num) {
console.info("Too small");
}
tries++;
}
Примечание: Не используйте console.info(), используйте вместо него alert(). Консоль предназначена для отладки, а не для уведомлений пользователей.
Я вижу твою проблему. Если вы поместите переменную внутри функции while, она будет сбрасываться каждый раз, что означает, что каждый раз она будет равна 1.
Если вы поместите его снаружи, он будет работать нормально. Кроме того, я считаю, что promt возвращает строку, а не число, поэтому я бы порекомендовал вам либо преобразовать число в строку, либо promt в число.
Вот рабочий код:
var num = Math.floor(Math.random() * (100)) + 1;
var running = true;
var tries = 1;
while(running) {
var input = parseFloat(prompt("Take a guess"));
if (input == num) {
console.info("Correct!");
console.info("Number of tries: " + tries);
running = false;
}else if (input > num) {
console.info("Too big");
}else if (input < num) {
console.info("Too small");
}
tries++;
}
Спасибо за ответ! Но он работает без преобразования. Не делать этого - плохая практика?
Это не так, но иногда в некоторых браузерах это вызывает ошибку, поскольку представляет число в виде строки. Если вы этого не сделаете, то все в порядке, но перед выпуском я бы проверил разные браузеры и разные версии этих браузеров.
Хорошо, и, думаю, я бы тоже не смог использовать оператор '===' как 'input === num'.
Извините не заметил оператора. Если вы используете '==', проблем не будет
О, я этого не видел! Спасибо.