javascript - 使用javascript的乐透号码随机生成器

Question

我是 javascript 的新手。在我的课上，我们为随机数生成器编写了代码，但我的代码不起作用。我想知道是否有人可以查看它并告诉我我做错了什么。我认为我的语法在循环中是错误的，但不能确定。

function lottoGen() {
  var i = 0; //Variable for increment
  var d = 0; //Variable for decrement

  var arr2 = [0, 0, 0, 0, 0, 0]; //6 array values. Begin at 0

  arr2[5] = Math.random(1, 26); //Choose random number for position 5 in array

  while (i <= 4) { //Perform loop while i <= 4
    arr2[i] = Math.random(1, 69);
    d = i;
    while (d !== 0 && d <= 4) {
      d--;
      if (arr2[i] === arr2[d]) {
        i--;
      }
      i++;
    }
  }

  document.getElementById("lotto").innerHTML = arr2; //Print the array
}

<p>Lottery Number Generator</p>
<form>
  <button onclick="lottoGen()">Generate</button>
  <p id="lotto"></p>

Answer 1

在第一次迭代中，i 为 0，因此 d 也为零，因此此 block :

while (d !== 0 && d <= 4) {
  d--;
  if (arr2[i] === arr2[d]) {
    i--;
  }
  i++;
 }
}

不运行，因为 d 是 0，因此 i 不会递增，最终进入无限循环。您实际上希望始终遍历数组:

 while (i <= 4) { //Perform loop while i <= 4
  arr2[i] = Math.random(1, 69);
  d = i;
  while (d !== 0 && d <= 4) {
    d--;
    if (arr2[i] === arr2[d]) {
      i--;
    }
  }
  i++; // <<<
}

---

此外，Math.random() 不接受任何参数并返回 0 到 1 之间的数字，因此要获得特定范围内的整数，您必须使用一个小实用程序:

 const random = (min, max) => min + Math.floor((max - min) * Math.random());

console.log(random(1, 69));

---

PS:说实话，你的代码其实挺难懂的，注释也没什么用。与其描述代码，不如尝试描述您要在那里实现的目标:

 // Step through the array and fill it with random numbers
 while (i <= 4) { 
  arr2[i] = random(0, 69);
  d = i;
 // Check all positions to the left if the number is already taken
  while (d !== 0 && d <= 4) {
    d--;
    if (arr2[i] === arr2[d]) {
      // If thats the case, stay at this position and genrate a new number
      i--;
    }
  }
  i++;
}

---

我会怎么写:

 function lottoGen() {
   const result = [];

  for(let count = 0; count < 6; count++) {
    let rand;
    do {
      rand = random(0, 69);
    } while(result.includes(random))
    result.push(rand);
  }

  return result;
}