Esta es la razón
Tomando como punto de partida el comentario hecho por @gibianchi, cito:
Lee tu código.. ponele comentarios.. si dentro del if, los dos son return, entonces no hay for que valga, porque solo comparas una sola vez no?
Encontré lo siguiente:
for (var i = 0; i < asientos.length; i++) {
if (solicitado == asientos[i]) {
return "Si encuentra el elemento, se detiene aquí";
} else {
return "Pero también se detiene aquí si no lo encuenta";
}
}
Por lo tanto, el bucle solo iterará una vez porque cualquiera va a devolver un valor. Una forma simple de corregirla es así:
for (var i = 0; i < asientos.length; i++) {
if (solicitado == asientos[i]) {
return "Si lo encontró devolverá esta cadena y detendrá el bucle";
}
return "Mientras que esta es la cadena por defecto que devolverá si no encuentra un elemento";
}
Es decir, se debe sacar del bucle la palabra reservada return
para evitar una sola iteración. Entonces, si aplicamos los siguientes cambios tendremos:
function asientosDisponibles(asientos, solicitado) {
for (var i = 0; i < asientos.length; i++) {
if (solicitado == asientos[i]) {
return (
"Felicitaciones, el asiento número " + solicitado + " está disponible"
);
}
}
return (
"Lo sentimos, el asiento número " +
solicitado +
" está ocupado, pero aún quedan " +
asientos.length +
" asientos disponibles"
);
}
let A = asientosDisponibles([15, 28, 44, 45, 70], 15);
let B = asientosDisponibles([15, 28, 44, 45, 70], 75);
console.log(A);
console.log(B);
Pero existen soluciones muchos más descriptivas o elegantes:
Si se utiliza el bucle for
:
function asientosDisponibles(asientos, solicitado) {
for (let asiento of asientos) {
if (asiento === solicitado) {
return (
"Felicitaciones, el asiento número " + solicitado + " está disponible"
);
}
}
return (
"Lo sentimos, el asiento número " +
solicitado +
" está ocupado, pero aún quedan " +
asientos.length +
" asientos disponibles"
);
}
let A = asientosDisponibles([15, 28, 44, 45, 70], 15);
let B = asientosDisponibles([15, 28, 44, 45, 70], 75);
console.log(A);
console.log(B);
O la solución propuesta por @Triby (por cierto, es una de la más elegante), que consiste en la utilización del método includes
que su función, dicho de forma corta, es devolver un true
o false
como consecuencia de haber encontrado o no el elemento incluido en el array que se pretende evaluar.
Solo que en esta oportunidad se modificó un poco para utilzarse con un operador ternario.
Por ejemplo:
function asientosDisponibles(asientos, solicitado) {
const [A, B] = [
`Felicitaciones, el asiento número ${solicitado} está disponible`,
`Lo sentimos, el asiento número, ${solicitado} está ocupado, pero aún quedan ${asientos.length} disponibles`
];
return asientos.includes( solicitado ) ? A : B;
}
// Probar la función:
let A = asientosDisponibles([15, 28, 44, 45, 70], 15);
let B = asientosDisponibles([15, 28, 44, 45, 70], 75);
console.log(A);
console.log(B);
En esta última oportunidad, se desestructuró un array en elementos A
y B
, donde A
representa el mensaje que se enviará al usuario si el asiento está disponible y B
es el caso contrario.
Y luego, un operador ternario, es decir:
return condicion ? verdadero : falso;
Que expresado en el ejemplo anterior queda así:
return asientos.includes(solicitado) ? A : B;
Aunque existen soluciones muchos más elegantes todavía.