Skip to main content
se añadieron 4 caracteres en el cuerpo
Origen Enlace
E. Betanzos
  • 5k
  • 1
  • 10
  • 24

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.

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} 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.

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.

se añadieron 528 caracteres en el cuerpo
Origen Enlace
E. Betanzos
  • 5k
  • 1
  • 10
  • 24

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:

iffunction asientosDisponibles(asientos,solicitado) ==={
 asientos[i]   for (var i = 0; i < asientos.length; i++) {// 
 Notar el uso del    if (solicitado === asientos[i]){
            return "Felicitaciones`Felicitaciones, el asiento número 25${solicitado} está disponible";disponible`;
        }
    } 
 else {  
    return "Lo`Lo sentimos, el asiento número 78${solicitado} está ocupado, pero aún quedan 4${asientos.length} asientos disponibles";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.

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.

La solución es usar alguno de los operadores de comparación:

if (solicitado === asientos[i]) {// Notar el uso del ===
    return "Felicitaciones, el asiento número 25 está disponible";
} else {
    return "Lo sentimos, el asiento número 78 está ocupado, pero aún quedan 4 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.

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} 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.

Origen Enlace
E. Betanzos
  • 5k
  • 1
  • 10
  • 24

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.

La solución es usar alguno de los operadores de comparación:

if (solicitado === asientos[i]) {// Notar el uso del ===
    return "Felicitaciones, el asiento número 25 está disponible";
} else {
    return "Lo sentimos, el asiento número 78 está ocupado, pero aún quedan 4 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.