Pass by value instead of refernce

Asked Jul 12 '17 at 16:28

Active Jul 17 '17 at 17:00

Viewed 50 times

        for (var siteCounter = 0; siteCounter < audit.SessionSiteList.length; siteCounter++) {
        for (var itemCounter = 0; itemCounter < audit.SessionSiteList[siteCounter].AuditItemList.length; itemCounter++) {
            var item = audit.SessionSiteList[siteCounter].AuditItemList[itemCounter];

           item.TotalItems = audit.SessionSiteList[siteCounter].AuditItemList.length;
            item.CurrentItem = itemCounter;

            promises.push(function(){return $scope.submitaudit(item)});
        }
    }

When the promises are executed, they are all using the same data. ie the last assigmnet to item gets used by all promises. How do i pass by value?

Can i clone the data and pass that?

edited Jul 17 '17 at 17:00

Daniel

3,541
3
33
46

asked Jul 12 '17 at 16:28

user2520306

could you create plunker of your code – user93 Jul 12 '17 at 16:30
2

check this SO question [Closure inside loops](https://stackoverflow.com/questions/750486/javascript-closure-inside-loops-simple-practical-example) for insight. – koox00 Jul 12 '17 at 16:35
You need a closure. By the time the function is called the loop has completed and `item` is the last item in the loop. – charlietfl Jul 12 '17 at 16:39
Easy fix is use `forEach()` instead of `for()` – charlietfl Jul 12 '17 at 16:44

Pass by value instead of refernce

0 Answers0