Почему переменная всегда выводит 1, даже если она увеличивается в Javascript

Я попытался решить проблему, которую нашел в Интернете. Я успешно решил проблему, но есть небольшая ошибка, которую я не смог решить.

Проблема

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

Объясните, пожалуйста, что я делаю не так.

Спасибо.

Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
В настоящее время производительность загрузки веб-сайта имеет решающее значение не только для удобства пользователей, но и для ранжирования в...
Безумие обратных вызовов в javascript [JS]
Безумие обратных вызовов в javascript [JS]
Здравствуйте! Юный падаван 🚀. Присоединяйся ко мне, чтобы разобраться в одной из самых запутанных концепций, когда вы начинаете изучать мир...
Система управления парковками с использованием HTML, CSS и JavaScript
Система управления парковками с использованием HTML, CSS и JavaScript
Веб-сайт по управлению парковками был создан с использованием HTML, CSS и JavaScript. Это простой сайт, ничего вычурного. Основная цель -...
JavaScript Вопросы с множественным выбором и ответы
JavaScript Вопросы с множественным выбором и ответы
Если вы ищете платформу, которая предоставляет вам бесплатный тест JavaScript MCQ (Multiple Choice Questions With Answers) для оценки ваших знаний,...
1
0
30
3
Перейти к ответу Данный вопрос помечен как решенный

Ответы 3

Ответ принят как подходящий

Вы повторно инициализируете 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++;
}

О, я этого не видел! Спасибо.

user9045638 01.07.2018 15:45

Ваши переменные декали 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++;
}

Спасибо за ответ! Но он работает без преобразования. Не делать этого - плохая практика?

user9045638 01.07.2018 16:44

Это не так, но иногда в некоторых браузерах это вызывает ошибку, поскольку представляет число в виде строки. Если вы этого не сделаете, то все в порядке, но перед выпуском я бы проверил разные браузеры и разные версии этих браузеров.

Andrey Seregin 01.07.2018 16:50

Хорошо, и, думаю, я бы тоже не смог использовать оператор '===' как 'input === num'.

user9045638 01.07.2018 16:52

Извините не заметил оператора. Если вы используете '==', проблем не будет

Andrey Seregin 01.07.2018 17:22

Другие вопросы по теме