Math.random con minimo y maximo, sin repeticionesM

[Licrat]

Muy buenas a todos!
a ver si a alguien se le ocurre alguna maneras de hacer un numero aleatorio con maximo y minimo y que el numero resultante no se repita!

tengo este codigo para crearlo,pues mi fin es seleccionar todas las filas de una tabla y enviarlas y al presentarlas en otra pagina que se presenten de manera aleatoria :

function anadir_aleatorias(){
chk3 = document.getElementsByName('check[]');
var num = chk3.length;
for(i=0;i<chk3.length;i++){
if(!cancion3){
var cancion3 = "";
}else{
cancion3 = cancion3;
}
chk3[i].checked = true;
if(chk3[i].checked){
var x = num++;
var numero = Math.floor((chk3.length-x)*Math.random()+x)
alert(numero);
//cancion3 += chk3[numero].value;
//window.open("agregar_cancion.php?canciones="+canci on3 , "reproductor");
}
}
}

graciass!

[venkman]

Por lo que entiendo, lo que en realidad quieres es ordenar aleatoriamente un arreglo de objetos que ya tienes, ¿no? Lo digo porque es un poco distinto que simplemente generar números aleatorios y hay mejores formas de hacer esto.

Código javascript:

function mezclar(x) {
    var i = x.length;
    while (i>0) {
        k = Math.floor(Math.random()*i);
        i--;
        var temp = x[i];
        x[i] = x[k];
        x[k] = temp;
    }
    return x;
}
 

En el código que tienes, podrías llamarlo así:

Código javascript:

chk3 = document.getElementsByName('check[]'); // El array está ordenado
chk3 = mezclar(chk3); // El array ahora está ordenado aleatoriamente
 

[Licrat]

Buenas!
he probado el codigo que me has ofrecido y no me coloca los objetos de manera aleatoria!
los envia tal y como estan ><
el codigo que he utilizado es:

function mezclar(x) {
var i = x.length;
while (i>0) {
k = Math.floor(Math.random()*i);
i--;
var temp = x[i];
x[i] = x[k];
x[k] = temp;
}
return x;
}

function anadir_aleatorias(){
chk3 = document.getElementsByName('check[]');
chk3 = mezclar(chk3);
for(i=0;i<chk3.length;i++){
if(!cancion3){
var cancion3 = "";
}else{
cancion3 = cancion3;
}
chk3[i].checked = true;
if(chk3[i].checked){
cancion3 += chk3[i].value +"|";
window.open("agregar_cancion.php?canciones="+canci on3 , "reproductor");
}
}
}

tengo algo mal? gracias!

[venkman]

Ohm, vaya, no me acordaba que getElementsByName() no devuelve un Array normal...

A ver, prueba así:

Código javascript:

function mezclar(x) {
    var i = x.length;
    while (i>0) {
        k = Math.floor(Math.random()*i);
        i--;
        var temp = x[i];
        x[i] = x[k];
        x[k] = temp;
    }
    return x;
}
function anadir_aleatorias(){
    chk3 = document.getElementsByName('check');
    canciones = new Array();
    for(i=0;i<chk3.length;i++){ canciones.push(chk3[i].value); }
    canciones = mezclar(canciones);
    canciones = canciones.join("|");
alert(canciones); // Este alert es para que veas el orden. lo puedes quitar
    window.open("agregar_cancion.php?canciones="+canciones, "reproductor"); 
}
 

En tu codigo hay algo un poco raro, que es eso de poner el checked a true y luego comprobar si es true. Lo he quitado. Además, tal como lo tenías se estaban abriendo múltiples ventanas.

[Licrat]

gracias!
si es cierto que se abrian varias ventanas y por eso de vez en cuando alguna cancion se enviaba varias veces!
este mismo codigo el he utilizado tambien para enviar la informacion tanto ordenada como aleatoriamente!
gracias de nuevo!