我可以简化这个Javascript循环吗？

Question

I may have gone around the most convoluted way to achieve this (see code below). Could someone with experience tell me if there is a better way?

I have two JSON objects resource and data . I want to loop over each new item in data and if the key matches in resource replace the value for that key in resource with the value in data. If this does not make sense the code I have below will.

This works and does what I need it to do, but it does not feel right (don't laugh)

factory.put = function (resource, data) {
        dataArr = [data];
        resourceArr = [resource];


        for (var i = 0; i < dataArr.length; i++) {
            var obj = dataArr[i];
            for (var key in obj) {
                var attrName = key;
                var attrValue = obj[key];


                for (var i = 0; i < resourceArr.length; i++) {
                    var obj = resourceArr[i];
                    for (var key in obj) {
                        var resourceArrName = key;
                        var resourceArrValue = obj[key];

                        if (resourceArrName == attrName){
                           resource[resourceArrName] = attrValue
                        }
                    }

                    }

                }
            }

           resource.put()
        }

Answer 1

Unless I'm missing something, you have no need to stuff the objects into arrays. You just loop through the attributes of one object, data , check if that attribute exists in the other object, resource , and if it does, replace the value in resource with that of data .

for (var key in data) {
    var attrName = key;
    var attrValue = data[key];

    if(resource[attrName]) {
        resource[attrName] = attrValue;
    }
}

Answer 2

You don't need to iterate over resourceArr's keys. Just check if resourceArr[key] exists, if so, set the value.

Something like:

if(resourceArr[key])
 resourceArr[key] = attrValue;

Answer 3

One way to simplify the loops, is to use the forEach function, which angular provides:

factory.put = function (resource, data) {
    dataArr = [data];
    resourceArr = [resource];

    angular.forEach(dataArr, function(obj) {
        angular.forEach(obj, function(value, key) {
            angular.forEach(resourceArr, function(resourceObj) {
                if(resourceObj[key]) {
                    resourceObj[key] = value;
                }
            }
        }
    }

    resource.put()
}