Group array of object into shorter array of object

Question

I want to make this

const arr = [
    {
        "name": "mac"
    },
    {
        "group": "others",
        "name": "lenovo"
    },
    {
        "group": "others",
        "name": "samsung"
    }
]

into this:



 [
   {
     name: 'mac',
   },
   {
     name: 'others',
     group: [
       {
         name: 'lenovo',
       },
       {
         name: 'samsung',
       },
     ],
   }
 ]

I tried to use normal forEach loop but it didn't turn out well:

let final = []
const result = arr.forEach(o => {
    if(o.group) {
        group = []
        group.push({
            name: o.name
        })
        final.push(group)
    } else {
        final.push(o)
    }
});

Not sure if reduce might help? before I try lodash groupBy I want to use just pure js to try to make it.

Using the same key for different purposes looks like really bad object design. — connexo, Jul 28 '22 at 06:22

score 0 · Answer 1 · answered Jul 28 '22 at 05:19

0

instead of creating and pushing group array in final again and again u should just push group in the end of foreach like this--

let final = []
let group = []
const result = arr.forEach(o => {
    if(o.group) {
        
        group.push({
            name: o.name
        })
    } else {
        final.push(o)
    }
})
final.push({name: "others", group})

answered Jul 28 '22 at 05:19

Dev Garg

1
3

and also `const result` is not neccessary in `forEach` – Dev Garg Jul 28 '22 at 05:20
you hardcoded "others" which is incorrect. – Jenny Le Jul 28 '22 at 05:36
oh kk sry i thought he only wanted others array i will correct it Thanx :D – Dev Garg Jul 28 '22 at 05:41

Mr.Developer · Accepted Answer · 2022-07-28T06:14:11.240

0

Hope this answer will work for you.

const arr = [
  {
    name: "mac",
  },
  {
    group: "others",
    name: "lenovo",
  },
  {
    group: "others",
    name: "samsung",
  },
];

const temp = [];
arr.forEach((e) => {
  if (!e.group) {
    temp.push(e);
  } else {
    const index = temp.findIndex((ele) => ele.name === e.group);
    if (index === -1) {
      obj = {
        name: e.group,
        group: [{ name: e.name }],
      };
      temp.push(obj)
    } else {
      temp[index].group.push({ name: e.name });
    }
  }
});

console.log(temp);

edited Jul 28 '22 at 06:14

answered Jul 28 '22 at 05:30

Mr.Developer

489
2
8

check my output, your output is wrong – Jenny Le Jul 28 '22 at 05:46
please check your question, what you want ? @JennyLe – Mr.Developer Jul 28 '22 at 05:48
you want above output in your question or below ? – Mr.Developer Jul 28 '22 at 05:49
you need to read my question again, your output is wrong. this is the output `[{"name":"mac"},{"name":"others","group":[{"name":"lenovo"},{"name":"samsung"}]}]` run your own script. – Jenny Le Jul 28 '22 at 05:55
so please check your question first, what you want from into this – Mr.Developer Jul 28 '22 at 06:02
if you want to above output so please set this output in I want to content – Mr.Developer Jul 28 '22 at 06:03
please check now, i have edited my answer, hope it will help you – Mr.Developer Jul 28 '22 at 06:14
it worked! findIndex is interesting! – Jenny Le Jul 28 '22 at 06:53
1

add vote or give useful answer if it's worked – Mr.Developer Jul 28 '22 at 06:56

Group array of object into shorter array of object

2 Answers2