1

Verán cree un array que crea un texto random que coloca en un link, reemplazando los espacios por - pero me arroja undefined.

Este es mi código:

var array = ["texto 1", "text 2", "text 3", "text 4"];
 var random = Math.floor(Math.random() * array.length);
 $('a').attr('href', array[random].replace(/\s/g, "-"));
3
  • ¿el array es creado correctamente y el valro random está dentro del rango del arreglo? Commented el 13 mar. 2017 a las 22:42
  • Los array son universales entre los lenguajes de programación: los índices van desde 0 hasta n - 1; donde n es la cantidad de elementos.
    – gugadev
    Commented el 13 mar. 2017 a las 23:26
  • Salvo en sql donde los rownum y autoincrementales parten de 1 (aunque no necesariamente)
    – ffflabs
    Commented el 14 mar. 2017 a las 0:13

1 respuesta 1

3

Los arrays en javascript parten desde el cero y tienen índices [0 ... length-1].

Tú estás haciendo un random que en ocasiones apuntará al índice [length], que no existe. (aunque esta posibilidad es minúscula y me sorprende que siempre te esté dando undefined.

Para asegurarte de no apuntar nunca al índice 4, debieras hacer:

var array = ["texto 1", "text 2", "text 3", "text 4"];
var random = Math.floor(Math.random() * (array.length-1));
$('a').attr('href', array[random].replace(/\s/g, "-"));

Pero esto no te va a dar un resultado equiprobable. En esa solución la posibilidad de que salga el texto 4 es ínfima.

Yo lo corregiría contemplando el caso de borde (random===array.length) y corrigiéndolo para apuntar al índice anterior:

var array = ["texto 1", "text 2", "text 3", "text 4"];
var random = Math.floor(Math.random() * array.length);
if(random===array.length) {
   random=(array.length-1)
}
console.log(array[random].replace(/\s/,'-'));

PD: el nombre "array" es un muy mal nombre para un array.

8
  • +1 por: el nombre "array" es un muy mal nombre para un array xD.
    – gugadev
    Commented el 13 mar. 2017 a las 23:25
  • Jajajajaja ya lo se, solo lo puse de ejemplo. xD Commented el 14 mar. 2017 a las 0:00
  • Y por siertó gracias por la explicación, veré si me da un resultado positivo. Commented el 14 mar. 2017 a las 0:02
  • Por sierto, se me olvido mencionar que el error me resalta en el replace. xD Commented el 14 mar. 2017 a las 0:05
  • Un pregunta mas, como podría generar el texto aleatorio de manera ordenada. Commented el 14 mar. 2017 a las 0:12

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.