0

Программа играет с пользователем, загадывает число, а пользователь должен его отгадать. Всё работает хорошо, только если не ввести некорректное значение (число больше 100 или меньше 0, либо вообще не число), тогда программа, отрабатывает отмену, но всё равно выполняет код после (сравнение в функции checkNumber()). В чём может быть проблема?

И ещё, мне сказали что у меня неправильная генерация случайного числа, но я не понял почему. Можете подсказать?

let userNumber;

const randomNumber = Math.floor(Math.random() * 101);
console.log(randomNumber);

alert('Привет человек! Я загадал число от 0 до 100. Попробуй меня победить!')

function enterNumber() {
    userNumber = prompt('Введите число от 0 до 100: ');
    // если нажали отмена, то завершить программу
    if (userNumber == null) {
        alert('Пока!')
        return;
    }
    getUserNumber(userNumber);
    checkNumber(userNumber);
}

function checkNumber(num) {
    // сравнивает число которое "загадал" компьютер с числом пользователя
    if (randomNumber > num) {
        alert('Меньше!');
        enterNumber();
    }

    if (randomNumber < num) {
        alert('Больше!');
        enterNumber();
    }

    if (randomNumber == num) {
        alert ('Число ' + userNumber + ', это верное число! Человек, ты победил!');
        return;
    }

}

function getUserNumber(num) {
    // Проверка на число и на поподание в промежуток от 0 до 100
    if (!isNaN(parseFloat(num)) && isFinite(num) && num >= 0 && num <= 100) {
        num = Number(num);
        return num;
    }
    else {
        alert('Некорректно');
        enterNumber();
    }


}

enterNumber();

3
  • вы не хотите рекурсию на цикл заменить случаем?
    – teran
    Commented 11 сент. 2020 в 14:35
  • 1
    нормальный random у вас. возможно, того кто сказал, смущает 101.
    – teran
    Commented 11 сент. 2020 в 14:39
  • @teran, насчёт random согласен. А вот насчет рекурсии вас не понял. Commented 11 сент. 2020 в 14:44

1 ответ 1

1

только если не ввести некорректное значение (число больше 100 или меньше 0, либо вообще не число), тогда программа, отрабатывает отмену, но всё равно выполняет код после (сравнение в функции checkNumber())

Проблема здесь:

function enterNumber() {
  ..
  getUserNumber(userNumber); // даже если здесь проверка проверка определит, что число некорректно
  checkNumber(userNumber); // эта функция останется в стеке и будет вызвана
}

Поэтому вызывайте ее только когда уверены в корректности значения:

if (!isNaN(parseFloat(num)) && isFinite(num) && num >= 0 && num <= 100) {
   num = Number(num);
   checkNumber(num);
}

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.