Skip to main content
se añadieron 183 caracteres en el cuerpo
Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

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'));

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, colocando accessories en el objeto global.

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 newAcces = 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'));

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, y 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
  */
  if ( theCat.accessories.includes(catAccessory) ) {
    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'));

Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

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, colocando accessories en el objeto global.

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 newAcces = 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'));