javascript - 如何正确访问像这样的javascript "dictionary"

Question

你能帮忙指出我的代码有什么问题吗:目标是计算 3 人就餐时每人点 2 道菜的总消费金额。

我知道还有其他方法可以以更 oop 的方式使用“this”和“new”来做到这一点，但我认为我在这里的方法更具可读性……如果我能让它发挥作用的话。如果每个顾客只点一道菜，代码就可以正常工作，所以我尝试访问 JS 字典的方式有问题。

很抱歉这个新手问题。如有任何帮助，我们将不胜感激!

var diners=[
 {name:'adam', entree1:'ramen', price1:11.5, entree2: 'miso', price2 : 3},

 {name:'bobby', entree1: 'udon', price1 :10.69, entree2: 'gyoza', price2 :4.5},

{name:'carly', entree1: 'teriyaki chicken', price1:12, entree2: 'miso', price2 : 3},
];

  var entrees1_total=0;
  for (var d in diners){


 entrees1_total +=  diners[d].price1; //total expense of entree1 for all diners  

 diners[d].tax1 = diners[d].price1*0.082; // method for sales tax

 entrees1_total += diners[d].tax1; //total entree1 price including sales tax
 }

 var entrees2_total=0;
 for (var d in diners){


entrees2_total += diners[d].price2;

diners[d].tax2 = diners[d] * price2 * 0.082;

entrees2_total += diners[d].tax2;
}

 var total = entree1_total + entree2_total;

 var total_bill = total*1.2;   //tips

 console.log("total is: " + total_bill.toString());

 for (var d in diners) {
 console.log(diners[d].name + " spends " + (diners[d].price1 + diners[d].tax1)+(diners[d].price2 + diners[d].tax2));
    }   // print out total spent for each patron

Answer 1

您可以使用 .reduce()迭代数组并保持运行总计:

var rawTotal = diners.reduce(function(cum, obj) {
   return cum + obj.price1 + obj.price2;
}, 0);

// add in tax and tip
var tax = rawTotal * 0.082;
var tip = rawTotal * 0.2;    // assumes you don't include taxes in tip calculation
var finalTotal = rawTotal + tax + tip;

工作演示:https://jsfiddle.net/jfriend00/ure3r2jg/

<小时/>

关于您的代码的一些注释:

1. 永远不要使用 for/in 来迭代数组。 for/in迭代对象的所有属性，其中包括数组元素，但也可以包括其他可枚举属性(如果已添加到对象中)。相反，使用 .forEach() , .map() , .reduce()或常规for (var i = 0; i < array.length; i++)循环或在 ES6 中，for/of .

2. 您的销售税计算不正确。您将每个小计值相乘，这意味着早期值将乘以销售税值多次。最简单的方法是累积常规总额，然后在最后应用销售税。或者，仅将销售税乘以每个新价格，而不是乘以累计总额。

3. 您必须决定小费是在税前还是税后计算。这是偏好问题，但我已经展示了小费是在税前计算的。