Question

我有一个递归函数（执行Heap's algorithm以生成排列）。似乎有一个范围可变的问题。

function permute(starting_arr) {
  var all_permutations = [];
  heap(starting_arr, starting_arr.length-1);
  console.log(all_permutations);
  return all_permutations;

  function heap(a, n) {  // a = array, n = max index i.e. length -1
    var temp_an;

    if (a.length-1 == n) {
      all_permutations.push(a); 
    }

    if (n>1) {
      heap(a.slice(), n-1);
    }

    for ( var i=0; i<n ; i++ ) {
      temp_an = a[n];
      if (n%2==0) {
        a[n] = a[0]
        a[0] = temp_an;
      } else {
        a[n] = a[i]
        a[i] = temp_an; 
      }

      all_permutations.push(a); 

      if (n>1) { heap(a.slice(), n-1); }   
    }
  }
}

问题似乎出在“ all_permutations.push（a）”上，因为如果我将其替换为...，事情就解决了。

 all_permutations.push([]); 
 for ( var g=0 ; g<a.length ; g++ ) {
   all_permutations[all_permutations.length-1][g] = a[g];
 }

正如您所看到的，我正在尝试使用slice，我已经读过它作为一种解决方案，它允许您通过值而不是引用来传递数组，例如本例中的

function x(arr){
  arr.push(4);
}

var a = [1,2,3]
x(a);
console.log(a);
var b = [1,2,3]
x(b.slice());
console.log(b);

那可以做些什么，但是仍然不能正常工作。

感谢您的帮助。

Answer 1

如您所见，我正在尝试使用slice，作为解决方案，我允许您使用它来通过值而不是引用传递数组。

是的，您找到了问题以及解决问题的正确方法。但是，您需要将其专门应用于行

all_permutations.push(a);

在您的for循环中：

all_permutations.push(a.slice());

递归Javascript函数中的范围问题

1 个答案: