How to add Json items using forEach loop?

Question

This is part of GoJS (diagramming) project and "properties" is itemArray, which is defined inside of one node in diagram.

1) This works (hard coded values):

properties: [
    { "property_name": "Prop1", "property_value": "100"},
    { "property_name": "Prop2", "property_value": "101" },
    { "property_name": "Prop3", "property_value": "102" }
]

2) This doesn't work (from data source):

properties: [
    data.ObjectPropertiesList.forEach(function (item, i) {
        properties.push({ "property_name": item.Item1.toString(), "property_value": item.Item2.toString() });
    })
]

2.1) With surrounding code:

myPalette.model = new go.GraphLinksModel([
    { key: "B", text: "some block", color: "blue" },
    { key: "G", text: "Macro", isGroup: true },
    { category: "circle", key: "Gc", text: "A", color: "black", group: "G", loc: "0 0" },
    {
        category: "table", key: "Ga", group: "G", loc: "60 0",
        properties: [
            data.ObjectPropertiesList.forEach(function (item, i) {
                properties.push({ "property_name": item.Item1.toString(), "property_value": item.Item2.toString() });
            })
        ]
    }
], [
    { from: "Gc", to: "Ga" }
]);

can you please also post the sorrounding code of the second example you have posted here — Parv Sharma
– Parv Sharma, Commented Jun 21, 2017 at 11:15

Bourbia Brahim · Accepted Answer · 2017-06-21 11:19:59Z

2

Why pushing data inside var declaration ? just declare your array then push values , see below working snippet

var data= {};
  data.ObjectPropertiesList = [
  {Item1:"Prop1",Item2:"100"},
  {Item1:"Prop2",Item2:"101"},
  {Item1:"Prop3",Item2:"102"},
]

properties = [];

data.ObjectPropertiesList.forEach(function (item, i) {
   properties.push({ "property_name": item.Item1.toString(), "property_value": item.Item2.toString() });
})

console.log(properties);

answered Jun 21, 2017 at 11:19

Bourbia Brahim

14.7k4 gold badges45 silver badges55 bronze badges

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

1 Comment

Bourbia Brahim Over a year ago

Great :) @tesicg

str · Accepted Answer · 2017-06-21 11:19:03Z

0

Don't use forEach when you want map:

properties: data.ObjectPropertiesList.map(function (item) {
    return { "property_name": item.Item1.toString(), "property_value": item.Item2.toString() };
})

answered Jun 21, 2017 at 11:19

str

45.5k18 gold badges117 silver badges134 bronze badges

1 Comment

str Over a year ago

Because it is a prime example to use map. You have an input array for which you want to modify each element and return the new array. This is exactly what map does, it transforms array items and returns them.

Parv Sharma · Accepted Answer · 2017-06-21 11:16:55Z

0

var properties = [];
for(var i=0; i < data.length; i++){
    var item = data[i];
    properties.push({ 
        "property_name": item.Item1.toString(), 
        "property_value": item.Item2.toString() 
    })
}

answered Jun 21, 2017 at 11:16

Parv Sharma

12.7k5 gold badges52 silver badges83 bronze badges

Collectives™ on Stack Overflow

How to add Json items using forEach loop?

3 Answers 3

1 Comment

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related