function array_rand ( input, num_req ) { // Pick one or more random entries out of an array
//
// + original by: _argos
var Indexes = [];
var Ticks = num_req || 1;
var Check = {
Duplicate : function ( input, value ) {
var Exist = false, Index = 0;
while ( Index < input.length ) {
if ( input [ Index ] === value ) {
Exist = true;
break;
}
Index++;
}
return Exist;
}
};
if ( input instanceof Array && Ticks <= input.length ) {
while ( true ) {
var Rand = Math.floor ( ( Math.random ( ) * input.length ) );
if ( Indexes.length === Ticks ) { break; }
if ( !Check.Duplicate ( Indexes, Rand ) ) { Indexes.push ( Rand ); }
}
} else {
Indexes = null;
}
return ( ( Ticks == 1 ) ? Indexes.join ( ) : Indexes );
}
ошибка в строке if ( !Check.Duplicate ( Indexes, Rand ) ) { Indexes.push ( Rand ); }
должно быть if ( !Check.Duplicate ( Indexes, Rand ) ) { Indexes.push ( input[Rand] ); }
Сегодня 21 год, эти вещи делаются проще...
function array_rand(data, num){ num = Math.min(num || 1, data.length); var rand = []; while(num--) { rand.push( data.splice(Math.floor(Math.random() * data.length) % data.length, 1)[0] ); } return rand; }А чтоб исходный массив не изменялся, добавьте вначале функции его клонирование