想问下这个 js 的栈溢出怎么改呀~刚开始用 js 写算法 Orz

2021-09-27 00:06:01 +08:00
 shermer
一个硬币的动态规划问题:

var coinChange = function(coins, amount) {
return dp(coins,amount);
};

var dp=function(coins,amount){
if(amount==0) return 0;
if(amount<0) return -1;
let res =Infinity;
for(let coin in coins)
{
let sub=dp(coins,amount-coin);
if(sub==-1) continue;
res=Math.min(res,sub+1);
}
return res==Infinity?-1:res;
};


然后在第二个函数里的第一句报了栈溢出
1082 次点击
所在节点    前端开发
5 条回复
shermer
2021-09-27 00:10:26 +08:00
在线等~急~愁的睡不着了
hsfzxjy
2021-09-27 00:54:20 +08:00
数据规模多大?
momocraft
2021-09-27 01:09:26 +08:00
递归改成循环
Pipecraft
2021-09-27 01:50:01 +08:00
@shermer
问题出在 array 的 for 循环用错了。
`for(let coin in coins)`

改成 `for (let coin of coins)`

```
for (let i = 0; i < coins.length; i++) {
const coin = coins[i];
```
就可以了。
Pipecraft
2021-09-27 01:54:09 +08:00
`for in` 循环的是下标,0,1,2...
`amount-coin` 这里减 0 就导致无限循环了。

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/804487

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX