能不能帮我看看这段 js 代码哪里出问题了,我找了好久不知道错在哪里

2016-10-24 09:26:21 +08:00
 pumpkin

我想收集一个不重复数组的全排列集合,但是这段代码怎么都得不到结果

function permute(a){
	var n = a.length
	var ret = []
	helper(a, 0, a.length-1,ret)
	console.log(ret)
	function helper(arr, start, end,ret){	
		var m = arr.length;
		if(start === end){
			ret.push(arr) 
			console.log(arr)
			return 
		}else {
			for(var i = start; i <= end; i++){
				swap(arr ,i, start)
				helper(arr, start+1, end,ret)
				swap(arr, start, i)
			}
		}
	}
	function swap(arr, i, n){
		var t = arr[i]
		arr[i] = arr[n]
		arr[n] = t
	}
}

每次进去调用 helper 函数之后 ret 变量都被重新赋值了,不是以前改变的那个数组,可我这里好像没有把 ret 重新 赋值吧,每次操作都是在原对象上 push ,求各位大大解释下。

1895 次点击
所在节点    问与答
7 条回复
xxxyyy
2016-10-24 09:56:23 +08:00
你看错了吧,你在 helper 里 console 的是 arr 的,不是 ret 。
另外, helper 的第一个参数最好不要自己传原来的数组,而是复制原来的数组,比如 helper(arr.slice(), ...)。当然这一步也可以在 helper 函数里的开始位置做。
pumpkin
2016-10-24 10:03:19 +08:00
@xxxyyy 没有看错,我想把 arr push 到 ret 中去, 不过真的把 arr 换成 arr.slice()后就实现了。。太感谢了,可是为什么会这样呢,我一直以为问题出在 ret 在递归过程可能存在重新赋值的问题
loy6491
2016-10-24 11:00:40 +08:00
你的代码里, ret 中 push 的一直是同一个 arr 的引用
用 slice 会复制出一个新的 arr ,不会被之后的 swap 影响
zuotech
2016-10-24 11:57:54 +08:00
女程员 vs 女司机
pumpkin
2016-10-24 12:16:38 +08:00
@loy6491 嗯嗯,明白啦,感觉这种递归很难定位错误。。
pumpkin
2016-10-24 12:16:46 +08:00
@zuotech 嘻嘻
SilentDepth
2016-10-24 19:25:11 +08:00
循环嵌递归……一开始还真愣住不知道该怎么查。
不过涉及数组状态的保存,就一定要想到数组赋值是引用传递,即使它存到别的数组里也是引用传递。需要固定数组的某个状态时一定要 slice (或者别的「复制」数组方法)。

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

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

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

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

© 2021 V2EX