Сделать один объект из нескольких, как?

Question

[mamaanarhiya]

Есть объекты вида

{status: 'approved', message: '123'}
{status: 'declined', message: '3123123'}
{status: 'approved', message: '3434342'}
{status: 'approved, message: 'asdsdfdad'}

Нужно создать один объект из них. То есть нужно взять объекты, у которых status = approved и создать один объект со свойством status: 'approved' и в message все message из каждого объекта.
То есть на выходе должно получиться что-то типа этого:

{status: 'approved', message: '123\n 3434342\n asdsdfdad'}

Answer 1

[Антон Спирин]

const objects = [
  {status: 'approved', message: '123'},
  {status: 'declined', message: '3123123'},
  {status: 'approved', message: '3434342'},
  {status: 'approved', message: 'asdsdfdad'},
];

const result = objects.reduce(function(acc, el, i) {
  if (el.status === 'approved') {
    acc.message += (!i ? '': '\n') + el.message;
  }

  return acc;
}, { 
  status: 'approved',
  message: '',
});

Answer 2

[Interface]

Как еще один вариант:

const arr = [
  {status: 'approved', message: '123'},
  {status: 'declined', message: '3123123'},
  {status: 'approved', message: '3434342'},
  {status: 'approved', message: 'asdsdfdad'}
]

const approvedMessages = arr
    .filter(n => n.status === 'approved')
    .map(n => n.message);

const obj = {
    status: 'approved',
    message: approvedMessages.join('\n'),
}

Answer 3

[0xD34F]

const obj = arr
  .filter(n => n.status === 'approved')
  .reduce((p, c) => ({ ...p, message: `${p.message}\n${c.message}` }));

Или, собираем всё, а затем извлекаем, что потребуется:

const grouped = arr.reduce((acc, { status, message }) => {
  acc[status] = acc[status] || { status, messages: [] };
  acc[status].messages.push(message);
  return acc;
}, {});

Object.keys(grouped).forEach(n => grouped[n].messages = grouped[n].messages.join('\n'));


const obj = grouped.approved;

Comments

[Антон Спирин]

лишний проход

[0xD34F]

Антон Спирин, да хоть десять лишних - это приемлемо, пока не доказано обратное.

[Антон Спирин]

0xD34F, не спрою, в данной задаче приемлемо. Но факт остается фактом.

[0xD34F]

Антон Спирин, а ещё Солнце круглое и светит - факт есть факт. Сказать-то что хотели?

[Антон Спирин]

0xD34F, в первом комментарии сказал.

[0xD34F]

Антон Спирин, так, а смысл говорить очевидное? Можно и ваше решение было бы откомментировать: объём - восемь строк, использована стрелочная функция, отступы - два пробела.

[Антон Спирин]

0xD34F, да забейте. Не в обиду и не с целью задеть было сказано. Алгоритмы люблю. Зарядка для ума.

[Interface]

Антон Спирин, почему лишний? У вас в ответе 1 проход с 2-мя операциями (сравнение и сборка объекта), в данном случае 2 прохода с 1 операцией - т.е. +/- одно и тоже.

[Антон Спирин]

Interface, вижу вы не видели исходный код функции reduce. Создается объект(в нашем случае) перед проходом, наполняется данными и возвращается.
В случае же этого алгоритма во время второго прохода, на каждом элементе создается и возвращается новый объект. С учетом дополнительного прохода, работы больше в разы(от двух и более). Чем больше элементов, тем сильнее разница в скорости выполнения задачи. В контексте данной задачи, не существенно. А вот, скажем, плагин для IDE так лучше не писать.

[Петр Мухуров]

Антон Спирин, в каком конкретно месте создается и возвращается новый объект?) В начале второго прохода берется первый элемент массива после первого прохода, формируется объект, а на каждой следующей итерации идет заполнение/дополнение этого объекта

[Антон Спирин]

pterodaktil, в том самом месте где, по-видимому, заканчиваются ваши знания основ js.
Вот пример который призван объяснить вам в каком месте это происходит и почему ваше предположение "а на каждой следующей итерации идет заполнение/дополнение этого объекта" не имеет ничего общего с тем, что происходит в действительности.

[Петр Мухуров]

Антон Спирин, ладно, убедили, но в любом случае, не вам судить о границах моих знаний) Во всяком случае, указанный подход как минимум в двух пунктах лучше:
1) без всякого говнокода аля

if (el.status === 'approved') {
    acc.message += el.message + (index !== arr.length - 1) ? '\n' : '';
  }
// или slice() и т. д. по окончанию работы reduce

который неизменно появится в вашей версии, если нужно будет убрать последний символ переноса строки (ненужный, к слову)
2) расширяемый. А если придет json с 50 предефайненными полями? Писать

tratata.reduce(..., {
  k1: 'v1',
  k2: 'v2',
  k3: 'v3',
  ...
})

Отличная идея

[Interface]

Антон Спирин, да, прошу прощения, прочитал ваш код по-диагонали. Однако суть это меняет слабо. У вас 1 цикл, 2 операции (сравнение, склейка строк), у 0xD34F - 2 цикла (сравнение, сборка нового объекта), если его код немного поменять:

var arr = [
  {status: 'approved', message: '123'},
  {status: 'declined', message: '3123123'},
  {status: 'approved', message: '3434342'},
  {status: 'approved', message: 'asdsdfdad'}
]

var obj = arr
    .filter(n => n.status === 'approved')
    .reduce((p, c) => {
        p.message += '\n' + c.message;
        return p;
    }, { 
      status: 'approved',
      message: '',
})

(тут опять всплывает проблема о лишнем пересносе строки, о которой писали выше)

Получится также 2 прохода (и я не могу назвать какой-то из них лишним). Производительность кода будет примерно одного уровня.

Если я правильно понимаю ваши доводы - то вы считаете неэффективным (что имеет место) выполнять в цикле сложную операцию по формированию нового объекта.
В то время как изначально пишете о принципиальной разнице в количестве подходов.

Говоря о вашем решении: я бы не стал здесь (в вашем ответе) использовать reduce, в решении 0xD34F оно оправдано тем, что решается проблема первого переноса строки, у вас же просто итерация по массиву.

P.s. добавил свое решение, там тоже 2 цикла + join

[Антон Спирин]

Interface, насчет reduce согласен, лишние операции. Но удобство.
Функцию reduce оправдывает ее читаемость и лаконичность, но там где нужна производительность ее использовать не следует. Но это точно не про текущую задачу, тут ее использовать удобно.

Доводы вы поняли не правильно и то и другое просадка в производительности. Внизу будет бенчмарк.

А вот этот алгоритм эффективней вашего решения, только из-за одного подхода.

const result = {
  status: 'approved',
  message: [],
};
  
for (var i = 0; i < arr.length; i++) {
  if (arr[i].status === 'approved') {
    result.message.push(arr[i].message);
  }
}

result.message.join('\n');

Вот бенчмарк на средний показатель за 500 проходов по 10000 элементам.

https://jsfiddle.net/rockon404/mqpha6qg/

В бенчмарке в комментарии ниже решение с reduce, да еще и с дополнительным условием, и оно дает результаты не хуже вашего решения.
Так что думайте сами, что менее оправданно reduce или ваши два прохода.

[Антон Спирин]

pterodaktil,
1. Вариантов много без "всякого говнокода":
acc.message += (!i ? '': '\n') + el.message;
Даже с дополнительным условием алгоритм завершает операцию с массивом 10000 элементов в среднем 5 раз быстрей в Chrome и более чем в 10 раз быстрей в Firefox.
https://jsfiddle.net/rockon404/gvu1vg0r/

2. Тут во-первых другая задача, а во вторых такое решение, как вы предлагаете не отличная, а провальная идея. При таких условиях надо искать другое более изящное решение, нежели хардкод 50 ключей в алгоритме.

Но вы верно заметили, я не подумал о последнем переносе. В условиях задачи в конце строки переноса нет. Моя невнимательность. Исправил ответ.

[Interface]

Антон Спирин, спасибо за бенчмарк. Я добавил (заменил первый тест) ваш вариант: https://jsfiddle.net/jumdndm6/4/ и он оказался медленее (в большинстве запусков на моей машине) моего. Хотя там один проход вместо 2 моих. Ваш последний вариант, конечно, быстрее

[Антон Спирин]

Interface, Вот этот вариант выполняется в среднем то хуже, то лучше. а разница доли 1 тысясчной секунды для 10000 элементов(по сути разницы, пока счет не на миллионы, нет)
https://jsfiddle.net/rockon404/f6e8pmz7/

Answer 4

[Егор Живагин]

let your_array = [ // Положим объекты в массив
{status: 'approved', message: '123'},
{status: 'declined', message: '3123123'},
{status: 'approved', message: '3434342'},
{status: 'approved', message: "asdsdfdad"},
]

// Результат будет объектом со свойствами === статусами
// result.approved = [ 123, 3434342,  asdsdfdad , ...]
let result = concatAtStatus(your_array);


function concatAtStatus ( arrayOfObjects = [] ) {
   let result = {};
   arrayOfObjects.forEach( elem => {
      let { status, message } = elem;
      result[status] ? result[status].push(message) : result.status = [ message ];
   });
   return result;
}