非反復乱数ピッカー

Question

私は非反復乱数ピッカーを作成しようとしていますが、誰かが間違っていたことを指摘できますか？おかげ

var resultArray = []; 

function pickRandom(){ 
    var min = parseInt(document.getElementById("min").value); 
    var max = parseInt(document.getElementById("max").value); 
    var result = Math.floor((Math.random() * max) + min); 

    resultArray.push(result); 

    for (var i = 0; i < resultArray.length; i++){ 
    if(resultArray[i].includes(result)) 
    { 
     pickRandom(); 
    } else { 
     document.getElementById("result").value = result; 
    } 
    } 
} 

Answer 1

に変更し、それを：あなたのコードと2つの問題がありました

var resultArray = []; 
function pickRandom(){ 
    var min = parseInt(document.getElementById("min").value); 
    var max = parseInt(document.getElementById("max").value); 
    var result = Math.floor((Math.random() * max) + min); 
    if(resultArray.includes(result)) 
    { 
    pickRandom(); 
    } else { 
    resultArray.push(result); 
    document.getElementById("result").value = result; 
    } 
} 

：
1.あなたは、最初の配列に新しい要素をプッシュし、それが配列であるかどうかをチェックします常に真実を返します。これにより、無限ループが発生します。
2.メソッドは、要素が配列内にあるか文字が文字列内にあるかをチェックするために定義されています。あなたのコードでは、arrayの各要素を繰り返し処理した後、includeメソッドを使用しています。

Answer 2

resultArrayの長さが可能な数値の数と等しい場合にチェックを使用してループを終了することができます。

function pickRandom() { 
 
    var min = +parseInt(document.getElementById("min").value), 
 
     max = +parseInt(document.getElementById("max").value), 
 
     result = Math.floor((Math.random() * (max - min + 1)) + min); 
 

 
    if (resultArray.length === max - min + 1) { 
 
     document.getElementById("result").value = 'no more available'; 
 
     return; 
 
    } 
 
    if (resultArray.includes(result)) { 
 
     pickRandom(); 
 
    } else { 
 
     resultArray.push(result); 
 
     document.getElementById("result").value = result; 
 
    } 
 
} 
 

 
var resultArray = [];

<input type="text" id="min" placeholder="min"> <input type="text" id="max" placeholder="max"> <button onclick="pickRandom()">Generate Random Number</button> <input type="text" id="result" placeholder="random number">