尝试创建一个返回输入数组的所有组合的递归函数。我不断收到类型错误“无法读取未定义的属性”

Question

我很困惑为什么会收到错误，因为我传入了一个已定义且应该有长度的数组。然而，我不断收到“第 XXX 行的类型错误：无法读取未定义的属性（读取'长度'）”

这是我正在处理的代码...

function getAllCombos(arr) {
    // declare an array to hold results
  let results = [];
  // declare an innerfunction which takes a prefix and the original array passed in
  function recurse(prefix, arr) { 
    // loop through the original array, our base case will be when the loop ends
    for (let i = 0; i < arr.length; i++) { 
    // push the results of spreading the prefix into an array along with the current element both in an array
    results.push([...prefix, arr[i]]);
    // recursive case: now we build the prefix, we recurse and pass into our prefix parameter an array consisting of the prefix spread in, the current element being iterated on, and the original array sliced after our current element
    recurse([...prefix, arr[i], arr.slice(i+1)])
    }
  }
  // call the inner function with an empry prefix argument and the original array
  recurse([], arr);
  // return the results array
  return results;
}

这里有一些测试用例...

console.log(getAllCombos(['a', 'b'])); // -> [['a','b'], ['a'], ['b'], []]
console.log(getAllCombos(['a', 'b', 'c']));
// -> [
//   ['a', 'b', 'c'],
//   ['a', 'b'],
//   ['a', 'c'],
//   ['a'],
//   ['b', 'c'],
//   ['b'],
//   ['c'],
//   [],
// ]

有人可以向我解释为什么即使我传入一个有长度的数组，我仍然会收到此错误消息吗？

感谢您的指点！

Answer 1

哪里出错了

我认为你打错了recurse -

recurse([...prefix, arr[i], arr.slice(i+1)]) // <- notice `]`

将] 移动到arr[i] 之后-

recurse([...prefix, arr[i]], arr.slice(i+1))

新的开始

也就是说，我认为使用更简单的功能可以改善一切 -

function* combinations(t) {
  if (t.length == 0) return yield []
  for (const combo of combinations(t.slice(1))) {
    yield [ t[0], ...combo ]
    yield combo
  }
}

// array
for (const combo of combinations(["a", "b"]))
  console.log(`(${combo.join(",")})`)
  
// or even strings
for (const combo of combinations("abc"))
  console.log(`(${combo.join(",")})`)

(a,b)
(b)
(a)
()

(a,b,c)
(b,c)
(a,c)
(c)
(a,b)
(b)
(a)
()

一些优化

.slice 以上是有效的，但确实会创建一些不必要的中间值。我们可以使用索引i，就像您在原始程序中所做的那样 -

function combinations(t) {
  function* generate(i) {
    if (i >= t.length) return yield []
    for (const combo of generate(i + 1)) {
      yield [ t[i], ...combo ]
      yield combo
    }
  }
  return generate(0)
}

for (const combo of combinations(["?","?","?","?"]))
  console.log(combo.join(""))
  

????
???
???
??
???
??
??
?
???
??
??
?
??
?
?

要将生成器中的所有值收集到一个数组中，请使用Array.from -

const allCombos = Array.from(combinations(...))

• 要计算固定大小的组合，n 选择 k，请参阅 this Q&A
• 要使用类似技术计算排列，请参阅this related Q&A

没有生成器

使用生成器解决组合问题的主要优点是每个结果都是延迟提供的，并且可以随时停止/恢复计算。但是，有时您可能想要所有结果。在这种情况下，我们可以跳过使用生成器并急切地计算包含每个可能组合的数组 -

const combinations = t =>
  t.length == 0
    ? [[]]
    : combinations(t.slice(1)).flatMap(c => [[t[0],...c], c])
  
const result =
  combinations(["a", "b", "c"])
  
console.log(JSON.stringify(result))

[["a","b","c"],["b","c"],["a","c"],["c"],["a","b"],["b"],["a"],[]]