0

Tengo este problema: La funcion 'checkInventario' recibe como argumento un array de objetos llamado 'inventario' y el nombre de un item llamado 'item' Cada objeto tiene una propiedad 'nombre' y 'cantidad'. La funcion debe devolver la cantidad de items que hay. Si el item no existe la funcion tiene que devolver 0 (cero). Ej:

var inventario = [
   {    nombre: 'tenedor',
       cantidad: 6
     },
    {
       nombre: 'cuchara',
       cantidad: 4,
     },
   ]
   checkInventario(inventario, 'tenedor') devuelve => 6
   Tu código aca:

Lo que yo hice fue esto:

function checkInventario(inventario, item) {
for (i = 0; i < inventario.length; i++) {
    if (inventario[i].nombre === item) {
      return inventario[i].cantidad;
    } else {
      return 0;
    }
  }
}

Me salta error me pasa la cantidad de tenedores pero no de "cuchara".

3
  • Ayudaría mucho que agregues que error obtienes
    – user128299
    Commented el 25 ago. 2020 a las 15:02
  • El return 0; debería estar al salir del ciclo y eliminar la parte del else, porque ahora solo analizas el primer elemento y si no coincide, devuelves cero en lugar de seguir recorriendo el arreglo.
    – Triby
    Commented el 25 ago. 2020 a las 15:04
  • si lo puse al final me da la cantidad de en mi caso "tenedores" y no de "cuchara"
    – user189423
    Commented el 25 ago. 2020 a las 15:04

3 respuestas 3

2

function checkInventario(inventario, item) {
    return inventario.find((e)=> e.nombre === item )?.cantidad || 0;
}

var inventario = [
   {    nombre: 'tenedor',
       cantidad: 6
     },
    {
       nombre: 'cuchara',
       cantidad: 4,
     },
];
console.log(checkInventario(inventario, 'tenedor')); //6
console.log(checkInventario(inventario, 'vaso')); //0

solo tener en cuenta que al validar te valida por mayusculas minusculas y pudieses usar un tipo

function checkInventario(inventario, item) {
    return inventario.find((e)=> e.nombre.toString().toLowerCase() === item.toString().toLowerCase() )?.cantidad || 0;
}

var inventario = [
   {    nombre: 'tenedor',
       cantidad: 6
     },
    {
       nombre: 'cuchara',
       cantidad: 4,
     },
];

console.log(checkInventario(inventario, 'Tenedor')); //6
console.log(checkInventario(inventario, 'Vaso')); //0

Array.prototype.find()

0

Necesitas sacar el return 0 fuera del bucle asi:

function checkInventario(inventario, item) {
  for (i = 0; i < inventario.length; i++) {
    if (inventario[i].nombre === item) {
      return inventario[i].cantidad;
    } 
  }
  return 0;
}

De lo contrario si el item que buscas no esta en la primera posicion siempre va a retornar 0;

1
  • 3
    Ayudaría que le expliques que diferencia hace tener o no ese return 0
    – user128299
    Commented el 25 ago. 2020 a las 15:05
0

//Esto es lo que buscas:

for (i = 0; i < inventario.length; i++) {
  if (inventario[i].nombre === item) {
    return inventario[i].cantidad;
  } 
  else{
    return 0;
  }
}

}

1
  • Bienvenido/a a Stack Overflow en español: haz el recorrido de bienvenida para conocer el funcionamiento del sitio. Debes explicar brevemente lo que hace este código: ten en cuenta que la respuesta debe ser útil no solo para el OP, sino para los demás miembros de la comunidad. Este sitio no es un repositorio de código. Lee Cómo responder.
    – padaleiana
    Commented el 22 oct. 2021 a las 23:18

Tu Respuesta

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