你能帮忙指出我的代码有什么问题吗: 目标是计算 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
最佳答案
您可以使用 .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/
<小时/>关于您的代码的一些注释:
永远不要使用 for/in 来迭代数组。
for/in
迭代对象的所有属性,其中包括数组元素,但也可以包括其他可枚举属性(如果已添加到对象中)。相反,使用.forEach()
,.map()
,.reduce()
或常规for (var i = 0; i < array.length; i++)
循环或在 ES6 中,for/of
.您的销售税计算不正确。您将每个小计值相乘,这意味着早期值将乘以销售税值多次。最简单的方法是累积常规总额,然后在最后应用销售税。或者,仅将销售税乘以每个新价格,而不是乘以累计总额。
您必须决定小费是在税前还是税后计算。这是偏好问题,但我已经展示了小费是在税前计算的。
关于javascript - 如何正确访问像这样的javascript "dictionary",我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/38967799/