El break
esta funcionando bien lo que pasa es que se ingresa al else
en la primera instancia y detiene el ciclo sin llegar al max.
Esto ocurre porque estas comparando fibonacci[2]
y ese valor no existe, la comparación se convierte en undefined > 89 = false
. Para solucionar esto debes obtener el siguiente numero de la serie antes comparar tu solución con q - 1
, quedando de esta manera:
function p(max){
var fibonacci = []; fibonacci[0] = 0; fibonacci[1] = 1; var q = 2;
for(;q<max;q++){
var next = fibonacci[q-1] + fibonacci[q-2];
if(max > fibonacci[q - 1]next) {
fibonacci[q] = fibonacci[q-1] + fibonacci[q-2];next;
}
else break; // INTENTO CON BREAK <--
}
return fibonacci;
}
console.log(p(89143));