El problema principal es que estás usando el operador de asignación =
en vez de uno de comparación ==
o ===
. Al hacer esto lo que ocurre es que a la variable solicitado
le asignas el valor que hay en la posición i
del arreglo y luego JavaScript intenta convertir este valor en un booleano que es lo que usa el if
. Como el valor es diferente de 0
este se avalúa a true
y por esto siempre te indica que el asiento está disponible.
Además no puedes poner el return
dentro del if
y del else
ya que solo comprobaría la primera posición. Te recomiendo poner solo el if
y si se evalúa a true
retornar ahí mismo y poner el otro return
al fuera del bucle.
Por último, para que los mensajes sean correctos deben mostrar el valor del asiento que está buscando en vez de uno fijo. Para hacer esto puede usar los string templates.
La solución es usar alguno de los operadores de comparación:
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 - 1} asientos disponibles`;
}
Si quieres profundicar en la conversión automática de tipo en JavaScript te recomiendo comenzar por esta sección del libro Eloquent JavaScript: Automatic Type Conversion.