无限循环,虽然我放了个断点
Infinite loop although I put a break;
此算法的目的是返回一个有组织的数组(包含货币)的更改。
我建立了一个嵌套的while循环来循环,只要"change"不等于0,那么while循环函数中的每个循环当"change"大于0时(即:不是-ve)。
我错过了什么?
//
function checkCashRegister(price, cash, cid) {
var change = cash-price;
var chArr = [["PENNY", 0],
["NICKEL", 0],
["DIME", 0],
["QUARTER", 0],
["ONE", 0],
["FIVE", 0],
["TEN", 0],
["TWENTY", 0],
["ONE HUNDRED", 0]];
while (change !== 0) {
while (change - 100 > 0) {
chArr[8][1] += 100;
change -=100;
if (change <= 0) {break;}
}
while (change - 20 > 0) {
chArr[7][1] += 20;
change -=20;
if (change <= 0) {break;}
}
while (change - 10 > 0) {
chArr[6][1] += 10;
change -=10;
if (change <= 0) {break;}
}
while (change - 5 > 0) {
chArr[5][1] += 5;
change -=5;
if (change <= 0) {break;}
}
while (change - 1 > 0) {
chArr[4][1] += 1;
change -=1;
if (change <= 0) {break;}
}
while (change - 0.25 > 0) {
chArr[3][1] += 0.25;
change -=0.25;
if (change <= 0) {break;}
}
while (change - 0.1 > 0) {
chArr[2][1] += 0.1;
change -=0.1;
if (change <= 0) {break;}
}
while (change - 0.05 > 0) {
chArr[1][1] += 0.05;
change -=0.05;
if (change <= 0) {break;}
}
while (change - 0.01 > 0) {
chArr[0][1] += 0.01;
change -=0.01;
if (change <= 0) {break;}
}
if (change <= 0) {break;}
}
// Here is your change, ma'am.
return chArr;
}
checkCashRegister(17.46, 20.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]]);
您只需要检查单个现金部分,并且只有在其余部分大于找零时才使用支票。在while循环中不需要断点
缺少的是检查更改是否足够。
function checkCashRegister(price, cash, cid) {
var change = cash-price;
var chArr = [["PENNY", 0], ["NICKEL", 0], ["DIME", 0], ["QUARTER", 0], ["ONE", 0], ["FIVE", 0], ["TEN", 0], ["TWENTY", 0], ["ONE HUNDRED", 0]];
while (change >= 100 && cid[8][1] >= 100) {
chArr[8][1] += 100;
cid[8][1] -= 100;
change -=100;
}
while (change >= 20 && cid[7][1] >= 20) {
chArr[7][1] += 20;
cid[7][1] -= 20;
change -=20;
}
while (change >= 10 && cid[6][1] >= 10) {
chArr[6][1] += 10;
cid[6][1] -= 10;
change -=10;
}
while (change >= 5 && cid[5][1] >= 5) {
chArr[5][1] += 5;
cid[5][1] -= 5;
change -=5;
}
while (change >= 1 && cid[4][1] >= 1) {
chArr[4][1] += 1;
cid[4][1] -= 1;
change -=1;
}
while (change >= 0.25 && cid[3][1] >= 0.25) {
chArr[3][1] += 0.25;
cid[3][1] -= 0.25;
change -=0.25;
}
while (change >= 0.1 && cid[2][1] >= 0.1) {
chArr[2][1] += 0.1;
cid[2][1] -= 0.1;
change -=0.1;
}
while (change >= 0.05 && cid[1][1] >= 0.05) {
chArr[1][1] += 0.05;
cid[1][1] -= 0.05;
change -=0.05;
}
while (change > 0 && cid[0][1] >= 0.01) {
chArr[0][1] += 0.01;
cid[0][1] -= 0.01;
change -=0.01;
}
// Here is your change, ma'am.
return chArr;
}
var register1 = [["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]],
register2 = [["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]];
console.log(checkCashRegister(17.46, 20.00, register1));
console.log(register1);
console.log(checkCashRegister(19.50, 20.00, register2));
console.log(register2);
.as-console-wrapper { max-height: 100% !important; top: 0; }
我认为你应该重新考虑你的代码:
function checkCashRegister(price, cash) {
var change = cash - price;
var chObj = {
PENNY: 0,
NICKEL: 0,
QUARTER: 0,
ONE: 0,
FIVE: 0,
TEN: 0,
TWENTY: 0,
ONE_HUNDRED: 0
}
// Performs an euclidian division to get the quotient
// (in 750$, there are 7 times 100 ie quotient = 7)
chObj.ONE_HUNDRED = Math.floor(change / 100);
// Now get the remainder (750 minus 7 times 100)
change = change % 100;
// So on...
chObj.TWENTY = Math.floor(change / 20);
change = change % 20;
chObj.TEN = Math.floor(change / 10);
change = change % 10;
chObj.FIVE = Math.floor(change / 5);
change = change % 5;
chObj.ONE = Math.floor(change);
change = Math.floor((change - Math.floor(change)) * 100);
chObj.QUARTER = Math.floor(change / 20) / 100;
change = change % 25;
chObj.NICKEL = Math.floor(change / 10) / 100;
change = change % 10;
chObj.NICKEL = Math.floor(change) / 100;
return chObj;
}
var chObj = checkCashRegister(245, 1000);
for (var key in chObj) {
if (chObj.hasOwnProperty(key)) {
console.log(key + " : 't" + chObj[key]);
}
}
结果(可通过控制台查看):
PENNY : 0
NICKEL : 0
QUARTER : 0
ONE : 0
FIVE : 1
TEN : 1
TWENTY : 2
ONE_HUNDRED : 7
我认为还可以改进。
你的问题是,你检查等于0,你的实际值是0.00..[某事]你可以看到这是一个已知的问题
修改最后一个if语句为if (change.toFixed(2) <= 0) {break;}
顺便说一句,我建议你把你所有的钱的计算改为美分,以避免这种问题(而不是1.0美元使用100美分)。
原因如下:每种类型的最终硬币都不会被添加到数组中。
假设金额是0.02,2美分。然后在最后一个循环中,
while (change - 0.01 > 0) {
chArr[0][1] += 0.01;
change -=0.01;
if (change <= 0) {break;}
}
保护(0.02 - 0.01 > 0
)为真,因此进入循环,变化变为0.01。
下一次,0.01 - 0.01 > 0
为false,因此不进入循环。
然后外部循环一次又一次地运行,因为变化仍然是0.01,它永远不会变得更低。
所以>
应该是>=
。然后所有的break以及整个外部循环都可以被删除。
相关文章:
- </脚本>标签放错地方了
- JQuery Masonry.Ajax图标和“;加载新项目”;绳子放错地方了
- Css在每行的底部放了一整行,有角度ng重复
- 为什么这个文件上传拖放不起作用,我完全复制了 CodePen 代码
- 当我放入一个库时,幻灯片就停止工作了
- 控制器上没有达到断点,但返回了来自控制器的数据
- 缩放更改了诺基亚此处地图的事件
- javascript innerHTML print 放错了位置
- 如何为在Chrome中应用了css缩放的img获得正确的jQuery offset()
- Html5拖放意外激活了css:悬停样式
- 如果我捕获了onclick事件以在SVG文档中缩放和平移,我如何到达锚点href
- 在windows的safari浏览器上,html5的拖放功能被破坏了吗
- 缩放后删除自定义地理标记(加载了传单)
- 日期表没有'如果我放了日期选择器函数,它就根本不起作用
- twitter引导程序工具提示放错地方了
- Javascript值从webservice返回,但不会显示,除非使用了断点
- 无限循环,虽然我放了个断点
- dojo xhr.post,我在xhr里放了一个数组.发布数据,但数据被更改的原因
- 当前不会命中该断点.调试过程加载了不同版本的脚本文件
- 我已经获取了一个动态表的高度,现在我需要将这个高度放入另一个元素中