function seAbre(horarios,minimo){
let aTiempo = 0;
for(let i =0 ;i< horarios.length;i++){
if(horarios[i]<=0){
aTiempo ++;
}
if(aTiempo >= minimo){
return true;
} else{
return false;
}
}
}
console.log(seAbre([4, 5, -1], 1));
Como vemos no funciona bien, retorna false
como tu dices, sin embargo... y esto a que se debe?
Para darnos cuenta del porque de este comportamiento debemos definir y a la vez recordar tres sentencias basicas, return
, break
y continue
.
Donde return
hace parte de cualquier cosa que sea una función y break
y continue
solo hacen parte de los ciclos.
return
: hace que la función 'hable', es decir que nos devuelva a nosotros un valor que podamos usar fuera de la función, todo lo que este debajo de return no será ejecutado.
break
: rompe el ciclo en el que te encuentras para seguir con la ejecución del código.
continue
: salta a la siguiente ejecución del ciclo (si es que hay una proxima).
El problema se encuentra es en el return
, debido a lo que dijimos antes 'lo que sea que este debajo de return ya no será ejecutado', y cuando nosotros ponemos un return
dentro de un ciclo, el return
comienza a funcionar como si hubiesemos puesto un break
statement en el ciclo, es decir, se sale del ciclo y directamente detiene la ejecución de nuestra función en ese punto.
Y esto que tiene que ver con el problema??
Pues que sí por ejemplo, al momento de contar todos los tiempos, aún no se ha llegado al minimo requerido el resultado siempre va ser false
y en tu caso en la mayoria de casos nunca se van a llegar a contar todos los casos debido al return
statement que pusiste (no se alcanza a sumar todas las coincidencias de tu condición).
Para solucionarlo solo debes hacer uso de return cuando ya tengas toda tu información, es decir cuando el ciclo haya hecho lo suyo y no antes.
checa el siguiente código corregido:
function seAbre(horarios,minimo){
let aTiempo = 0;
for(let i =0 ;i< horarios.length;i++){
if(horarios[i]<=0){
aTiempo ++;
}
}
if(aTiempo >= minimo){
return true;
} else{
return false;
}
}
console.log(seAbre([4, 5, -1], 1));
Como vemos ahora funciona correctamente, y lo unico que tuvimos que hacer es esperar a que el ciclo hiciese lo suyo para poder retornar ya sea true
o false
.