Asumiendo que cats
tiene esta estructura:
let cats=[
{name:'Misu', accessories:[]},
{name:'Miau', accessories:[]}
];
Convendría crear una referencia de todo el objeto buscando por el nombre, así, ya usas esa referencia para todo: si la referencia es undefined
no hay gatos con ese nombre, esa validación queda hecha. Luego, la usas para verificar si el accesorio existe, y si no existe, usas la referencia para agregarlo.
En tu lógica parece que los accesorios son globales a todos los gatos, pero esto no tendría sentido, pues ¿cómo sabrías que Misu
ya tiene un Collar
?.
Si me equivoco modifica el código, no será difícil hacerlo, colocandoy accessories
debe ir en el objeto global., entonces colócalo allí, y para verificar haz: if ( cats.accessories.includes(catAccessory) )
, y para agregar: cats.accessories.push(catAccessory)
, aunque, en ese caso sería interesante que nos digas cómo sabrás a qué gato pertenece qué accesorio ¿?
let cats = [{
name: 'Misu',
accessories: []
},
{
name: 'Miau',
accessories: []
}
];
const addCatAccessory = function(catName, catAccessory) {
/*
Referencia a todo el gato
que servirá para verificar/agregar sus accesorios
*/
let theCat = cats.find(o => o.name === catName);
/*
Si no hay gatos con catName
la referencia será undefined
*/
if (theCat === undefined) throw new Error(`El gato ${catName} no existe`);
/*
Usamos includes para ver si
el accesorio está en el array accessories
*/
let newAccesif =( theCat.accessories.includes(catAccessory);
if (newAcces) {
throw new Error(`El gato ${catName} ya tiene el accesorio puesto`)
}
theCat.accessories.push(catAccessory);
return `El accesorio ${catAccessory} fue agregado a ${catName} con éxito`
}
console.log(addCatAccessory('Misu', 'Collar'));
console.log(cats);
console.log(addCatAccessory('Miau', 'Arena'));
console.log(cats);
console.log(addCatAccessory('Misu', 'Casita'));
console.log(cats);
// Casos de error
console.log(addCatAccessory('Misu', 'Collar'));
console.log(addCatAccessory('Fake', 'Collar'));