Angularjs Load object into array via forEach

Question

I have an array of Drug Codes 'BARB', 'HALU', 'METH'. I want to loop thru these codes and create a new object for each one.

var saveObj = {};
saveObj.SelectedList = [];
angular.forEach(self.DrugNamesSelected, function(value, key) {
    //Load Objects into array
    var newObj = {};
    newObj = self.Selected;
    newObj.FK_CLNT_PERSFK_PER = $routeParams.personId;
    newObj.DRUG_TYP_CODE = value;
    saveObj.SelectedList.push(newObj);
});

This is what I'm getting

saveObj.SelectedList[0].DRUG_TYP_CODE = 'METH' saveObj.SelectedList[1].DRUG_TYP_CODE = 'METH' saveObj.SelectedList[2].DRUG_TYP_CODE = 'METH'

This is what I need

saveObj.SelectedList[0].DRUG_TYP_CODE = 'BARB' saveObj.SelectedList[1].DRUG_TYP_CODE = 'HALU' saveObj.SelectedList[2].DRUG_TYP_CODE = 'METH'

Community · Accepted Answer · 2017-05-23 12:17:26Z

3

Seems like newObj = self.Selected; making you a problem. So what happening is each element is storing a reference of self.Selected(i.e. all objects are referring to single memory reference). So as last value updates to METH, all 3 elements has same self.Selected value.

angular.forEach(self.DrugNamesSelected, function(value, key) {
    //Load Objects into array
    var newObj = {};
    //newObj = self.Selected; //removed this line
    newObj.FK_CLNT_PERSFK_PER = $routeParams.personId;
    newObj.DRUG_TYP_CODE = value;
    saveObj.SelectedList.push(newObj);
});

If you need that particular object assignment for some reason, I'd suggest you to use Object.assign which will copy self.Selected properties to newObj.

Object.assign(newObj, self.Selected)

IE11 doesn't support Object.assign out of the box. You could consider adding polyfill for the same from here

edited May 23, 2017 at 12:17

CommunityBot

11 silver badge

answered Mar 14, 2017 at 3:02

Pankaj Parkar

136k23 gold badges241 silver badges307 bronze badges

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

1 Comment

gman Over a year ago

Object.assign doesn't work for ie(11) so I had to add a polyfill function. (stackoverflow.com/questions/35215360/…)

Collectives™ on Stack Overflow

Angularjs Load object into array via forEach

1 Answer 1

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related