Javascript乱数生成器

Question

私は1から80までの8つの乱数を取得し、指定されたdivに入れようとしています。

var numbers = [] 
 
var luckyNumber; 
 
var x = 1; 
 

 
while (x < 8) { 
 
    luckyNumber = Math.floor(Math.random() * 80 + 1); 
 
    if (numbers.indexOf(luckyNumber) == -1) { 
 
    for (var i = 1; i < 9; i++) { 
 
     document.getElementById('k' + i).innerHTML = numbers[i] 
 
    } 
 
    numbers.push(luckyNumber); 
 
    x++; 
 
    } 
 
} 
 

 
document.write(numbers);

<div id="k1">K1</div> 
 
<div id="k2">K2</div> 
 
<div id="k3">K3</div> 
 
<div id="k4">K4</div> 
 
<div id="k5">K5</div> 
 
<div id="k6">K6</div> 
 
<div id="k7">K7</div> 
 
<div id="k8">K8</div>

最後の2つの数値は、HTML結果に未定義示されています。しかし、document.writeで数字を確認すると、最後の2つの数字が表示されます。 この問題を解決するにはどうすればよいですか？

Answer 1

• whileループは、JavaScriptではなく、8
• 配列は0インデックスされている7つの数字を生成する、1から7まで行くので、あなたは内の配列を通過するとき、 1から始まるforループは、最初の番号をスキップしています。
• 新しい番号を除いて、古い番号で新しい番号を生成するたびにdivを埋めています。最後に生成された番号は、このプロセスに決して含まれません。

最も単純な修正は、whileループを修正し、forループを削除し、xを使用して、最新の数字を表示する場所を見つけます。

var numbers = [] 
 
var luckyNumber; 
 
var x = 1; 
 

 
while (x <= 8) { 
 
    luckyNumber = Math.floor(Math.random() * 80 + 1); 
 
    if (numbers.indexOf(luckyNumber) == -1) { 
 
    document.getElementById('k' + x).innerHTML = luckyNumber; 
 
    numbers.push(luckyNumber); 
 
    x++; 
 
    } 
 
} 
 

 
document.write(numbers);

<div id="k1">K1</div> 
 
<div id="k2">K2</div> 
 
<div id="k3">K3</div> 
 
<div id="k4">K4</div> 
 
<div id="k5">K5</div> 
 
<div id="k6">K6</div> 
 
<div id="k7">K7</div> 
 
<div id="k8">K8</div>

Answer 2

ループロジックが間違っているようです。数字の配列を最初に入力し、htmlを更新する必要があります。

var numbers = [] 
 
var luckyNumber; 
 
var x = 1; 
 

 
while (x < 9) { 
 
    luckyNumber = Math.floor(Math.random() * 80 + 1); 
 
    if (numbers.indexOf(luckyNumber) == -1) { 
 
    numbers.push(luckyNumber); 
 
    x++; 
 
    } 
 
} 
 
for (var i = 1; i < 9; i++) { 
 
     document.getElementById('k' + i).innerHTML = numbers[i-1] 
 
    } 
 

 
document.write(numbers);

<div id="k1">K1</div> 
 
<div id="k2">K2</div> 
 
<div id="k3">K3</div> 
 
<div id="k4">K4</div> 
 
<div id="k5">K5</div> 
 
<div id="k6">K6</div> 
 
<div id="k7">K7</div> 
 
<div id="k8">K8</div>