10 различных чисел в массиве

Question

Мне нужно 10 различных случайных чисел в массиве.

Я использую следующую функцию:

for (k=0;k<10;k++) { 
     x[k] = Math.floor((Math.random()*10)); 
     for (l=0;l<k;l++){ 
     if (x[l]==x[k]){fail=true;} 
     } 
     if (fail){k=k-1;} 
    } 

THX для ответа.

Answer 1

Убедитесь, что все переменные объявляются:

var x = [], k, l, fail; 

Убедитесь, что сброс «глючить» перед внутренним контуром:

fail = false; 
    for (l = 0; l < k; l++) 
    // ... 

Теперь, когда сказал, если вы хотите 10 номеров в массиве , а массив будет содержать 10 элементов, вам лучше заполнить массив цифрами от 1 до 10 (или от 0 до 9; что угодно), а затем shuffling the array. Если вы делаете что-то по-своему, то производительность процесса непредсказуема. Когда вы дойдете до последних двух элементов, это может занять две или три попытки, или это может занять сотни; вы никогда не знаете, что может сделать последовательность случайных чисел.

Answer 2

Было бы лучше, чтобы заполнить массив с номерами от 1 до 10, а затем перетасовать ее, как это:

var myArray = [1,2,3,4,5,6,7,8,9,10]; 

//shuffle array 
//code from http://stackoverflow.com/a/2450976/1223693 
var i = myArray.length, j, temp; 
if (i === 0) return false; 
while (--i) { 
    j = Math.floor(Math.random() * (i + 1)); 
    temp = myArray[i]; 
    myArray[i] = myArray[j]; 
    myArray[j] = temp; 
} 

//done 
alert(myArray); 

Ваш текущий метод очень медленно, потому что

• это имеет внутренний цикл
• он может генерировать неправильные номера много много раз, прежде чем он генерирует правильный номер

Answer 3

Вы действительно хотите иметь массив с 10 элементами со случайными номерами от 0 до 9, и ни один из них не может быть одним и тем же? Итак, вы хотите, чтобы цифры 0-9 в 10 элементах перетасовывались вокруг?

Тогда просто инициализировать их и перетасовать их:

//+ Jonas Raoni Soares Silva 
//@ http://jsfromhell.com/array/shuffle [v1.0] 
function shuffle(o){ //v1.0 
    for(var j, x, i = o.length; i; j = parseInt(Math.random() * i), x = o[--i], o[i] = o[j], o[j] = x); 
    return o; 
}; 


// init the array: 
var a = new Array(); 
for (var i=0;i<10;i++){ 
    a.push(i); 
} 
// shuffle it. 
a = shuffle(a); 

Answer 4

может быть simpler-

var nums= [0, 1, 2, 3, 4, 5, 6, 7, 8, 9].sort(
function(){return Math.random()-.5});