Розыгрыш не работает

Проблема с кодом для розыгрыша не сработает!

var i = 0;
var count;
var names = [
    "Stefon",
    "Garret",
    "Brandon"
];

function GetRandomInt(){
    return Math.floor(Math.random()*i+1);
}

function CallWinner(){
    var ID = GetRandomInt();
    document.write("<hr>"+names[ID]+" has won with the ID of "+id+"!");
}

do {
    i++;
    for(count=0;i<=names.length;){
        count++;
        document.write(names[count]+" has been assigned to the raffle ID, "+count+"<br>");
    }
} while (i<=names.length);

По какой-то причине это не работает, это действует как бесконечный цикл или, возможно, происходит сбой вкладки, он работает, но затем он вызывает сбой вкладки. Пожалуйста помоги.

Индекс в length неверен, и for-loop, использующий i, а не count, также неверен!

— 23.04.2018 23:27

Посмотрите внимательно на for(count=0;i<=names.length;). Ваше условие i<=names.length всегда будет выполняться => бесконечный цикл

— 23.04.2018 23:27

Я, честно говоря, не понимаю, какова должна быть логика этих вложенных циклов while и for, и у меня есть подозрения, как и у вас. Ваш цикл for увеличивает count, но использует i в условии, что вызывает бесконечный цикл. Прочтите снова о циклах и переосмыслите эту концепцию.

— 23.04.2018 23:29

javascript

23.04.2018 23:25

Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией

В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.

Концепция локализации и ее применение в приложениях React ⚡️

Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...

Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer

В настоящее время производительность загрузки веб-сайта имеет решающее значение не только для удобства пользователей, но и для ранжирования в...

Безумие обратных вызовов в javascript [JS]

Здравствуйте! Юный падаван 🚀. Присоединяйся ко мне, чтобы разобраться в одной из самых запутанных концепций, когда вы начинаете изучать мир...

Система управления парковками с использованием HTML, CSS и JavaScript

Веб-сайт по управлению парковками был создан с использованием HTML, CSS и JavaScript. Это простой сайт, ничего вычурного. Основная цель -...

JavaScript Вопросы с множественным выбором и ответы

Если вы ищете платформу, которая предоставляет вам бесплатный тест JavaScript MCQ (Multiple Choice Questions With Answers) для оценки ваших знаний,...

Перейти к ответу Данный вопрос помечен как решенный

Ответы 1

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

document.write - это документ. Неверный. Пожалуйста, используйте что-нибудь более современное и менее запутанное. Эта функция пытается записать в текущий документ. Если документ уже был обработан, он будет заменен пустым с вашим аргументом. Вы этого не хотите; вместо этого используйте правильные методы DOM.

Ваша функция GetRandomInt нарушена; он должен возвращать случайный доступный индекс в массиве, а не статическое число.

Попробуйте вместо этого что-нибудь вроде этого:

const names = [
  "Stefon",
  "Garret",
  "Brandon"
];

function GetRandomIndex() {
  return Math.floor(Math.random() * names.length);
}

function CallWinner() {
  const index = GetRandomIndex();
  const hr = document.body.appendChild(document.createElement('hr'));
  hr.textContent = names[index] + " has won with the ID of " + index + "!";
}
names.forEach((name, count) => {
  const div = document.body.appendChild(document.createElement('hr'));
  div.textContent = name + " has been assigned to the raffle ID, " + count;
});
CallWinner();

24.04.2018 00:46