Wylosowanie 100 liczb które się nie powtarzają

Question

var myArray = [];
min = 1;
max = 99;

    
        for (i=0; i<=100; i++) 
        {
            min = Math.ceil(min);
            max = Math.floor(max);
            x = Math.floor(Math.random() * (max - min)) + min;
            if(myArray.includes(x))
            {
                while(true) //wykonuj dopóki myArray.includes(x) = true
                {
                    x =  Math.floor(Math.random() * (max - min)) + min;
                }
                myArray.push(x);
           }
                else //jeżeli nie ciąg nie zawiera liczby wylosowanej 
                {
                    myArray.push(x);
                }
            }

                
        }
    
     

    console.log(myArray);

Kod ma na celu wypisanie 100 losowych liczb, które nie będą się powtarzać, jednak kiedy odpalam program w przeglądarce to się wysypuje.

Answer 1

A może prościej stworzyć tablicę dowolnych liczb, a następnie ją potasować? Na przykład mając tablicę 10 liczb (którą możesz stworzyć za pomocą odpowiedniej pętli czy jak tam wolisz), a następnie wykonać na niej funkcję:

var numArray = [1, 2, 3, 4, 5, 6, 7, 8, 9, 10];
var ranNums = shuffle(numArray);
console.log(ranNums);

Funkcja shuffle():

function shuffle(array) {
   var i = array.length, j = 0, temp;
   while (i--) {
      j = Math.floor(Math.random() * (i+1));
      temp = array[i];
      array[i] = array[j];
      array[j] = temp;
   }
   return array;
}

 

Comments

Genialne! Dziękuje!

---

NIe ma za co 

---

Propozycja z: Array.from i Array.sort

propozycja

const myArray = fillArrayInRange(1,99);
shuffleArray(myArray);
console.log(myArray);

function fillArrayInRange(min, max, step=1) {
  return Array.from({length: (max - min) / step + 1}, (_, i) => min + (i * step)); 
}
function shuffleArray(array) {
  array.sort(() => Math.random() - 0.5);
}

 

Answer 2

Skąd pobierasz zmienną x? Dlaczego pętla jest zawsze prawdziwa? Do czego służy else wewnątrz ifa? Przecież to część warunku, a nie wolna instrukcja. Przeanalizuj dokładnie, krok po kroku co wykonuje ten kod.