Javascript forEach an Array of Object recursive

Question

i need some help at looping over an Object with Array of Objects

Here is an example Object:

const node = {
    documents: [{
        name: "test1"
    }],
    children: [{
        documents: [{
            name: "test2"
        }],
        children: [{
            documents: [{
                name: "test3"
            }],
            children: []
        }]
    }]
};

My Problem
I tried some of the solutions from here but it's not quite what I need. Basically i want to move every object from the documents Array into the childrens Array.
I already got that but currently its only doing it in the firts Object so not recursive...
I cant figure it out without creating an Frankenstein-ish code...
I tried like i mentioned before i used

Array.concat()

to append documents to children but only in the first Object.
What it should look like

const node = {
    children: [
    {
    name: "test1"
    },
    {
    children: [
        {
      name: "test2"
      },
      {
      children: [
        {name: "test3"}
      ]
      }
    ]
    }
  ]
};

Now in every "depth" the documents were "appended" to children.
And that should happen for every children key (depth).

Is someone kind enough to help me out?

EDIT:

I guess my example is not good enough. Here is some actual data:

      const node = {
        id: 1,
        name: "Root Node",
        documents: [
          { name: "Doc Root 1", id: 1234 },
          { name: "Doc Root 2", id: 1235 }
        ],
        children: [
          {
            id: 2,
            name: "Sub Node Node1",
            documents: [
              { name: "Doc SubNote 1 1", id: 1236 },
              { name: "Doc SubNote 1 2", id: 1237 }
            ],
            children: [{
              id: 3,
              name: "Sub Sub Node Node 1",
              documents: [
                { name: "Doc SubSubNote 1 1", id: 1238 },
                { name: "Doc SubSubNote 1 2", id: 1239 }
              ],
              children: null,
              addTreeNodeDisabled: true,
              addLeafNodeDisabled: true,
              editNodeDisabled: true,
              delNodeDisabled: true,
            }],
            addTreeNodeDisabled: true,
            addLeafNodeDisabled: true,
            editNodeDisabled: true,
            delNodeDisabled: true,
          }
        ],
        addTreeNodeDisabled: true,
        addLeafNodeDisabled: true,
        editNodeDisabled: true,
        delNodeDisabled: true,
      };

the data structure is quite mixed up. please add the wanted result of it. — Nina Scholz
– Nina Scholz, Commented Jan 6, 2020 at 14:43
Sorry for that, I messed up the export :( Now its a better example — Lwtsde
– Lwtsde, Commented Jan 7, 2020 at 7:20

Nina Scholz · Accepted Answer · 2020-01-07 07:36:26Z

3

You could concat a copy of documents with the childrens by calling the function again.

var f = ({ documents, children, ...o }) => {
        children = [...(documents || []), ...(children || [])].map(f);
        return children.length
            ? { ...o, children }
            : o;
    },
    data = { id: 1, name: "Root Node", documents: [{ name: "Doc Root 1", id: 1234 }, { name: "Doc Root 2", id: 1235 }], children: [{ id: 2, name: "Sub Node Node1", documents: [{ name: "Doc SubNote 1 1", id: 1236 }, { name: "Doc SubNote 1 2", id: 1237 }], children: [{ id: 3, name: "Sub Sub Node Node 1", documents: [{ name: "Doc SubSubNote 1 1", id: 1238 }, { name: "Doc SubSubNote 1 2", id: 1239 }], children: null, addTreeNodeDisabled: true, addLeafNodeDisabled: true, editNodeDisabled: true, delNodeDisabled: true, }], addTreeNodeDisabled: true, addLeafNodeDisabled: true, editNodeDisabled: true, delNodeDisabled: true, }], addTreeNodeDisabled: true, addLeafNodeDisabled: true, editNodeDisabled: true, delNodeDisabled: true },
    result = f(data);

console.log(result);

.as-console-wrapper { max-height: 100% !important; top: 0; }

edited Jan 7, 2020 at 7:36

answered Jan 6, 2020 at 13:56

Nina Scholz

388k26 gold badges367 silver badges417 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Lwtsde Over a year ago

That is very good but I have an error. in some cases children can be null and it throws an length is undefined error.

Aziz.G Over a year ago

add (children && children .length) instead of children .length

Lwtsde Over a year ago

Can you have a look at my Edit? I cant get your code to run with this data. It runs, but is loses some keys :(

Akash Shrivastava · Accepted Answer · 2020-01-06 14:04:47Z

2

I've used recursion here to achieve the results.

const node = {"documents":[{"name":"test1"}],"children":[{"documents":[{"name":"test2"}],"children":[{"documents":[{"name":"test3"}],"children":[]}]}]}

const unwrap = (v) => ({...{...v}[0]});

const merge = (node) => ([...node.documents, {children: (unwrap(node.children).hasOwnProperty("children") ? merge(unwrap(node.children)) : node.children) }]);

console.log(merge(node));

answered Jan 6, 2020 at 14:04

Akash Shrivastava

1,3659 silver badges16 bronze badges

2 Comments

Lwtsde Over a year ago

Hi, maybe you can try with the data from my edit. I cant get yours to work :(

Akash Shrivastava Over a year ago

the datasets are nowhere near the same, i suggest you add a different question for that, and add a smaller object with clear and exact data structure.

Collectives™ on Stack Overflow

Javascript forEach an Array of Object recursive

EDIT:

2 Answers 2

3 Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

EDIT:

2 Answers 2

3 Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related