问题描述
我正在研究来自在线编码社区的挑战。 目的是编写一个函数,该函数使用CID数组(抽屉中的现金)中显示的面额生成从最高到最低的精确变化。 该函数接受三个参数,价格,现金,cid。 Cid是现金抽屉。
我正在尝试将cid的面额现金金额与精确更改计算的现金金额进行比较。
例如,如果change = 96,则从最高到最低的确切变化为
['twenty', 80], ['ten', 1], ['five', 1],
['one', 1]
。 但是,如果CID显示['twenty',60],['ten',20],[“ five,55],则代码必须使用cid中的数量来计算精确的变化。在这种情况下,结果将是,
['twenty', 60], ['ten', 20],
['five', 15], ['one', 1]
在下面的代码中,display()函数包含一个条件语句,用于进行比较并在必要时切换CID值。 但是,这导致出现[“ TWENTY”]的重复值。 我的问题是什么原因造成的,以及如何解决。
该函数的输入产生以下错误结果:
[ [ 'TWENTY', 60 ],
[ 'TWENTY', 20 ],
[ 'TEN', 10 ],
[ 'FIVE', 5 ],
[ 'ONE', 1 ],
[ 'QUARTER', 0.5 ],
[ 'DIME', 0.2 ],
[ 'PENNY', 0.04 ] ]
我已经在下面提供了代码,但是我在建立了模型。
function drawer(price, cash, cid){
var change = cash - price;
change = change.toFixed(2);
change = +change;
var combine = [];
var cidMap = [ cid[0], cid[1], cid[2], cid[3], cid[4], cid[5], cid[6], cid[7], cid[8] ];
var denomMap = [
100, 'ONE HUNDRED', 20, 'TWENTY', 10, 'TEN', 5, 'FIVE', 1, 'ONE', 0.25, 'QUARTER', 0.1, 'DIME', 0.05, 'NICKEL', 0.01, "PENNY"
];
function total(cid){
var cashAmts = [];
cid.forEach(function(v){
cashAmts.push(v[1]);
},0);
var sum = cashAmts.reduce(function(a, b){return a + b;
}).toFixed(2);
return +sum;
}
if(cash < price){
return price;
}
else if(total(cid) < change){
return "Insufficient Funds";
}
else if(total(cid) === change){
return "Closed";
}
else if(total(cid) > change){
return display(change, cid);
}
function display(change) {
var temp;
num = change;
var num = change;
var value = '';
for (var i = 0; change > 0 && i < denomMap.length; i += 2) {
while (change >= denomMap[i]) {
change += denomMap[i + 1];
change = change.replace(/\d+/g, "");
change = change.replace(/\./g, "");
value += denomMap[i];
var div = Math.floor(num/+value) * denomMap[i];
if(change === cidMap[0][0] && div > cidMap[0][1]){
div = cidMap[0][1];
}
else if(change === cidMap[1][0] && div > cidMap[1][1]){
div = cidMap[1][1];
}
else if(change === cidMap[2][0] && div > cidMap[2][1]){
div = cidMap[2][1];
}
else if(change === cidMap[3][0] && div > cidMap[3][1]){
div = cidMap[3][1];
}
else if(change === cidMap[4][0] && div > cidMap[4][1]){
div = cidMap[4][1];
}
else if(change === cidMap[5][0] && div > cidMap[5][1]){
div = cidMap[5][1];
}
else if(change === cidMap[6][0] && div > cidMap[6][1]){
div = cidMap[6][1];
}
else if(change === cidMap[7][0] && div > cidMap[7][1]){
div = cidMap[7][1];
}
else if(change === cidMap[8][0] && div > cidMap[8][1]){
div = cidMap[8][1];
}
temp = [change, div];
combine.push(temp);
num = (num - div).toFixed(2);
num = +num;
display(num);
}
}
return combine;
}
}
drawer(3.26, 100.00, [['PENNY', 1.01], ['NICKEL', 2.05], ['DIME', 3.10], ['QUARTER', 4.25], ['ONE', 90.00], ['FIVE', 55.00], ['TEN', 20.00], ['TWENTY', 60.00], ['ONE HUNDRED', 100.00]]);
1楼
是什么原因造成的?
您正在从抽屉中取出所有三个'TWENTY'
笔记,但没有更新抽屉以反映出现在没有'TWENTY'
笔记了。
因此,在下一个循环迭代中,抽屉错误地认为您还有另一张'TWENTY'
笔记要给。
我如何解决它?
提款后更新抽屉中的现金,并包括一张剩余现金支票。 这是一种对原始代码进行最少修改的方法,可以帮助您获得一些见识。
注意changeAvailable
函数及其调用位置。
还要注意对抽屉的更新(例如,对于三个TWENTY
注释: cidMap[7][1] = 0;
)。
(function () {
"use strict";
function drawer(price, cash, cid){
var change = cash - price;
change = change.toFixed(2);
change = +change;
var combine = [];
var cidMap = [ cid[0], cid[1], cid[2], cid[3], cid[4], cid[5], cid[6], cid[7], cid[8] ];
var denomMap = [
100, 'ONE HUNDRED', 20, 'TWENTY', 10, 'TEN', 5, 'FIVE', 1, 'ONE', 0.25, 'QUARTER', 0.1, 'DIME', 0.05, 'NICKEL', 0.01, "PENNY"
];
function display(change) {
var temp;
//num = change;//use global variables sparingly, preferably within a namespace, suspect type-o
var num = change;
var value = '',
changeAvailable = function (nameOfCurrency) {
var queryCash = cid.filter(function (currentCurrency) {
return currentCurrency[0] === nameOfCurrency;
}),
cashRemaining = queryCash[0][1];
return (cashRemaining !== 0);
};
for (var i = 0; change > 0 && i < denomMap.length; i += 2) {
while (change >= denomMap[i] && changeAvailable(denomMap[i + 1])) {
change += denomMap[i + 1];
change = change.replace(/\d+/g, "");
change = change.replace(/\./g, "");
value += denomMap[i];
var div;
div = Math.floor(num/+value) * denomMap[i];
if(change === cidMap[0][0] && div > cidMap[0][1]){
div = cidMap[0][1];
cidMap[0][1] = 0;
}
else if(change === cidMap[1][0] && div > cidMap[1][1]){
div = cidMap[1][1];
cidMap[1][1] = 0;
}
else if(change === cidMap[2][0] && div > cidMap[2][1]){
div = cidMap[2][1];
cidMap[2][1] = 0;
}
else if(change === cidMap[3][0] && div > cidMap[3][1]){
div = cidMap[3][1];
cidMap[3][1] = 0;
}
else if(change === cidMap[4][0] && div > cidMap[4][1]){
div = cidMap[4][1];
cidMap[4][1] = 0;
}
else if(change === cidMap[5][0] && div > cidMap[5][1]){
div = cidMap[5][1];
cidMap[5][1] = 0;
}
else if(change === cidMap[6][0] && div > cidMap[6][1]){
div = cidMap[6][1];
cidMap[6][1] = 0;
}
else if(change === cidMap[7][0] && div > cidMap[7][1]){
div = cidMap[7][1];
cidMap[7][1] = 0;
//temp = [change, div];//duplicated code
}
else if(change === cidMap[8][0] && div > cidMap[8][1]){
div = cidMap[8][1];
cidMap[8][1] = 0;
}
temp = [change, div];
combine.push(temp);
num = (num - div).toFixed(2);
num = +num;
display(num);
}
}
return combine;
}
function total(cid){
var cashAmts = [];
cid.forEach(function(v){
cashAmts.push(v[1]);
},0);
var sum = cashAmts.reduce(function(a, b){return a + b;
}).toFixed(2);
return +sum;
}
if(cash < price){
return price;
}
else if(total(cid) < change){
return "Insufficient Funds";
}
else if(total(cid) === change){
return "Closed";
}
else if(total(cid) > change){
return display(change, cid);
}
}
console.log(drawer(3.26, 100.00, [['PENNY', 1.01], ['NICKEL', 2.05], ['DIME', 3.10], ['QUARTER', 4.25], ['ONE', 90.00], ['FIVE', 55.00], ['TEN', 20.00], ['TWENTY', 60.00], ['ONE HUNDRED', 100.00]]));
}());