Могу ли я сделать эту функцию более кратким

Question

У меня есть этот фрагмент javascript, и мне интересно, могу ли я рассчитать amount и users за один проход функции reduce?

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; 
});