Как правильно склонировать объеты?

Question

[ivanDoligov]

Добрый день! Подскажите пожалуйста, как правильно склонировать любое число объектов и прибать id каждого объекта на единицу?

const clonObj = (obj, count) => {
  const objects = [];

  for (let i = 0; i < count; i++) {
    objects.push({ ...obj });
    objects[i].id = objects[i].id + 1;
  }

  return [...objects];
};

использование потом, примерно такое:

const result = clonObj({id: 5}, 10);
должно получиться 10 объектов и id начинаться с 5-ки

В каждом obj есть заранее уже поле id!
если можно, подскажите как правильно это сделать прибавив единицу к каждому следующему id и не использовать цикл for

Answer 1

[Александр]

Без цикла никак.

const clone = (base, count, key = 'id') => [...new Array(count)].map((_, index) => {
  const cloned = { ...base };
  cloned[key] += index;
  
  return cloned;
});

const a = {
  id: 7,
  type: 'apple'
};
console.log(clone(a, 8));
console.log(a);
/**
[
  { id: 7, type: 'apple' },
  { id: 8, type: 'apple' },
  { id: 9, type: 'apple' },
  { id: 10, type: 'apple' },
  { id: 11, type: 'apple' },
  { id: 12, type: 'apple' },
  { id: 13, type: 'apple' },
  { id: 14, type: 'apple' }
]
{ id: 7, type: 'apple' }
*/

Answer 2

[Wondermarin]

const cloneObject = (o, count = 1) => {
  const objects = new Array(count).fill(o);

  return objects.map((obj, index) => ({ ...obj, id: obj.id + index }));
};

Comments

[ivanDoligov]

Почему то у меня id в объекте не прибавляется, а остается моё, но добавляется ещё одно поле

console.log(cloneObject({id: 5}, 2);

/**
[
  0: { id: 5, type: 'apple' и т.д }, id: NaN
  1: { id: 5, type: 'apple'  и т.д}, id: NaN
]
{ id: 7, type: 'apple' }
*/

[Wondermarin]

ivanDoligov, проверил на codesandbox, все работает. Возможно у вас где-то ошибка, отправьте полный код.

[ivanDoligov]

Wondermarin,

const merge = require('lodash/merge');

const createList = (data = {}, countCampaigns = 1) => {
  const dataObj = [{
      id: data.id || defaultObj.id,
      // разные другие поля
  }];

  const preparedObj = merge(defaultObj, dataObj);

  const preparedObjList = cloneObject(preparedObj, countCampaigns);

  return [...preparedObjList];
};

[ivanDoligov]

Wondermarin, получается у меня есть defaultObj где уже заполнены поля объекта по умолчанию, но если нужно поменять какое-то поле я тогда в функции cloneObject указываю какие поля необходимо поменять и указываю сколько похожих объектов создать ). Только id нужно увеличивать всегда

[WbICHA]

ivanDoligov, а по какой причине у вас dataObj это массив?
Зачем тащить мердж лодаша, когда это спокойно желается нативно?
И зачем копировать только что созданный массив?
return [...preparedObjList];

[ivanDoligov]

WbICHA,
а по какой причине у вас dataObj это массив?
согласен, убрал!

Зачем тащить мердж лодаша, когда это спокойно желается нативно?
он умеет откидывать undefined, если не все свойства переданы.

а как тогда нативно поступить? Object.assign undefined не пропускает, а записывает в свойства

[WbICHA]

он умеет откидывать undefined, если не все свойства переданы.

ivanDoligov, откуда у вас там вообще андефайнды всплывают?

[ivanDoligov]

WbICHA, хотя merge тоже не как мне нужно объединил объекты... У меня при таком раскладе получилось на выходе 2 объекта
1-й со свойствами defaultObj
2-й с поменяным id, но остальные свойства undefined

[ivanDoligov]

WbICHA, я не знаю... вроде все свойсва имеют значения (в defaultObj)

[WbICHA]

ivanDoligov, тогда это проблема явно где-то у вас в коде.
{ ...defaultObj, ...dataObj }

[ivanDoligov]

WbICHA, ну вот весь код:

const defaultObj = [
  {
    id: 5310,
    прочие свойства: значения
  },
];

const merge = require('lodash/merge');

const createList = (data = {}, count = 1) => {
  const dataObj = {
      id: data.id || defaultObj.id,
      // разные другие поля
  };

  const preparedObj = merge(defaultObj, dataObj);

  const preparedObjList = cloneObject(preparedObj, count);

  return preparedObjList;
};

const cloneObject = (o, count) => {
  const objects = new Array(count).fill(o);

  return objects.map((obj, index) => ({ ...obj, id: obj.id + index }));
};

затем вызываю 
const result = createList({id: 44}, 2);
console.log(result)

и получаю в консоли:

[{...}, {...}]

разворачиваю

0: {0: {объект со свойствами defaultObj}, id: 44, name: undefined и т.д все свойства с undefined}

[ivanDoligov]

WbICHA, нашёл баг, Спасибо!

[WbICHA]

ivanDoligov, кстати, я бы переписал функцию, а то она нагромождена.

const createList = (data = {}, count = 1) => {
  const preparedObj = { ...defaultObj, ...data };
  return cloneObject(preparedObj, count);
};

Answer 3

[Антон Антон]

вместо { ...obj } используй JSON.parse(JSON.stringify(obj)), если объекты сложные (их свойства являются также объектами), но без методов или что-нибудь типа https://lodash.com/docs/4.17.15#cloneDeep если есть методы.
Ну а так - использоавние цикла тут логично, просто можно его немного поменять:

for (let i = 0, id = obj.id+1; i < count; i++, id++) {
    const clone = {...JSON.parse(JSON.stringify(obj)), id}
    objects.push(clone);
  }

Comments

[ivanDoligov]

Здесь у меня выдает ошибку, не дает использовать несколько раз let в цикле

[Антон Антон]

ivanDoligov, ну значит просто через запятую

[Дмитрий Беляев]

как же на qna не хватает отметки ненравится...
клонировать объекты через сериализацию - верх расточительства, да еще и клон будет с потерями...

[Антон Антон]

Дмитрий Беляев, вы предлагаете в каждом проекте пихать lodash или свою (скорее всего) кривую реализацию?
Ну а про потери я написал в ответе.

[Дмитрий Беляев]

Антон Антон, можно и lodash, хотя я лично предпочитаю ramda, если сборка tree-shake'ается нормально, то это будет в разы оптимальнее, чем через сериализацию гонять