La función que has usado es correcta, pero optimizable:
var array=["FI29","FI09","JM15","FI29"]//devuelve true
//["FI29","FI09","JM15","FI28"]//devuelve false
var repetido = false;
// El primer bucle compara con el segundo, así que no hace falta llegar al último valor (... < array.length-1).
for (var i = 0; !repetido && i < array.length-1; i++) {
// El segundo bucle puede empezar a continuación del primero,
// ya que los anteriores ya se comprovaron (j=i+1)
for (var j = i+1; !repetido && j < array.length; j++) {
// El resultado de la comparación indica si es repetido.
// En caso que sea true, no hace falta buscar mas,
// así que saldrá de los dos bucles (!repetido&&...)
repetido = array[i] == array[j];
}
}
console.log(repetido);
Una variación que podría ser más optima (bajando un nivel de ejecución) seria usar el método indexOf()
para buscar el valor dentro del resto del array usando el método splice()
.
var array=["FI29","FI09","JM15","FI29"]//devuelve true
// var array=["FI29","FI09","JM15","FI28"]//devuelve false
var repetido = false;
for (var i = 0; !repetido && i < array.length-1; i++) {
repetido = array.slice(i+1).indexOf(array[i])!=-1;
}
console.log(repetido);
NOTA: De todas maneras, sigue siendo una solución lenta, la mas rápida sin que modifique el array original, es la que te proporciona @user13558