この関数をより簡潔にすることができます

Question

関数の1回のパスでamountとusersを計算できるかどうか、このjavascriptスニペットがありますか？

root.children.forEach(function(v) { 

    v.amount = v.children.reduce(function(a, b) { 
     console.log(a); 
     return { 
      amount: a.amount + b.amount 
     } 
    }, { 
     'amount': 0 
    }).amount 

    v.users = v.children.reduce(function(a, b) { 
     console.log(a); 
     return { 
      users: a.users + b.users 
     } 
    }, { 
     'users': 0 
    }).users 

}) 

Answer 1

あなたは可能性があり、単一のArray#forEachループ。

var root = { 
 
     children: [ 
 
      { children: [ 
 
       { amount: 2, users: 3 }, 
 
       { amount: 7, users: 5 } 
 
      ]} 
 
     ] 
 
    }; 
 

 
root.children.forEach(function(v) { 
 
    v.amount = 0; 
 
    v.users = 0; 
 

 
    v.children.forEach(function(a) { 
 
     v.amount += a.amount; 
 
     v.users += a.users; 
 
    }); 
 
}); 
 
console.log(root);

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

Answer 2

はい、あなたは文字通り一つに二つの方法を組み合わせることができますように

root.children.forEach(function(v) { 
    var obj = v.children.reduce(function(a, b) { 
     a.amount += b.amount; 
     a.users += a.users; 
    }, {'amount': 0, 'users' : 0 }); 
    v.amount = obj.amount; 
    v.users = obj.users; 
}); 

Answer 3

、以下のように見えますことを行うことができます。

root.children.forEach(function(v) {   
    var result = v.children.reduce(
     function(a, b) { 
      return { 
       amount: a.amount + b.amount, 
       users: a.users + b.users 
      }; 
     }, 
     { amount: 0, users: 0 } 
    ); //^Note that I left out the quotes there. In this case, they're optional. 

    v.amount = result.amount; 
    v.users= result.users; 
});