2

Me podrían apoyar ¿en qué parte de mi función la lógica está mal?

Llevo intentando un buen rato y no logro avanzar.

El problema me pide lo siguiente:

Escribir la función asientosDisponibles(), que tome como parámetro un conjunto de asientos disponibles y el asiento que quiere ocupar la persona. La función deberá verificar si el asiento solicitado se encuentra disponible y devolverle un mensaje si está disponible o no.

var asientos = [3, 15, 18, 25];
var solicitado=25;

function asientosDisponibles(asientos,solicitado) {
    for (let i = 0; i < asientos.length; i++) {
        if (solicitado === asientos[i]){
            return `Felicitaciones, el asiento número ${solicitado} está disponible`;
        }else{
             return `Lo sentimos, el asiento número ${solicitado} está ocupado, pero aún quedan ${asientos.length - 1} asientos disponibles`;
        }
    }

    console.log(asientosDisponibles(asientos,solicitado));
}

1
  • Me sigue mandando el siguiente error: Errores: La función asientosDisponibles([3, 15, 18, 25], 25) debe retornar el mensaje "Felicitaciones, el asiento número 25 está disponible"
    – Aura MH
    Commented el 28 feb. 2021 a las 22:07

5 respuestas 5

5

No necesitas un ciclo para verificar si el asiento solicitado está dentro del arreglo, es suficiente con el método Array.includes() que devuelve verdadero si existe o falso si no.

No entiendo porqué usas asientos.length - 1 para devolver la cantidad de asientos que aún están disponibles, sobra el - 1.

function asientosDisponibles(asientos, solicitado) {
    if(asientos.includes(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`;
}

console.log(asientosDisponibles([15, 28, 44, 45, 70], 15));
console.log(asientosDisponibles([15, 28, 44, 45, 70], 50));

2

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.

10
  • Me sigue mandando error aunque agregue a la condición dentro de if doble "==" .
    – Aura MH
    Commented el 28 feb. 2021 a las 22:03
  • @AuraMH tienes razón, ni me había fijado en todos los detalles del código. Edité la respuesta para que aborde todos estos. Commented el 28 feb. 2021 a las 22:11
  • M esta saliendo un último error, y no logro descrifrarlo @E.Betanzos, mira: missing ) after argument list ¿me apoyas por favor?
    – Aura MH
    Commented el 28 feb. 2021 a las 22:20
  • @AuraMH en cuál línea?? Commented el 28 feb. 2021 a las 22:58
  • no dice, solo me manda el mensaje "Errores: missing ) after argument list" , Disuclpa no estaría faltando un else?
    – Aura MH
    Commented el 28 feb. 2021 a las 23:10
0

Estas usando variables globales en la funcion como variables locales, eso vuelve a inicializar las variables y borran sus valores. Borra esas variables del argumento.

1
  • me sigue saliendo error :(
    – Aura MH
    Commented el 28 feb. 2021 a las 22:36
0
function asientosDisponibles(asientos, solicitado) {
  for (var i = 0; i < asientos.length; i++) {
    if (asientos.indexOf(solicitado) !=-1) {
      return ( "Felicitaciones, el asiento número " + solicitado + " está disponible");
    } else {
      return ("Lo sentimos, el asiento número " + solicitado + " está ocupado, pero aún quedan " + asientos.length + " asientos disponibles" );
    }
  }

}

1
  • Hola Estefanía, para que la respuesta sea de mayor calidad y ayude más a la gente, es mejor si pones una explicación de por qué crees que esta es la respuesta correcta. Quizá explicar el código, o lo que haces distinto, o dónde estaba el error, etc.
    – ArianJM
    Commented el 23 mar. 2021 a las 8:34
-1
 var asientos = [15, 28, 44, 45, 70];

var solicitado=25;

function asientosDisponibles(asientos,solicitado) {
   for (var i = 0; i < asientos.length; i++) {
    if (solicitado == asientos[i]){
     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";
    }
   }   
}
1

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.