Question

我正在尝试在Ruby中实现最长的公共子序列算法，但是我收到了stack level too deep错误消息。我知道这可能意味着我有一个无限循环，但我无法发现它。以下是我最好的尝试 - 我哪里错了？

def lcs(string1, string2)
  if !(string1.chars.any? { |x| string2.include?(x)})
    return ""
  elsif string1[-1] == string2[-1]
    return lcs(string1[0..-2], string2[0..-2]) + string1[-1]
  else
    x = lcs(string1, string2[0..-1])
    y = lcs(string1[0..-1], string2)
    x.length > y.length ? x : y
  end
end

n.b。我试图返回子序列本身，而不是它的长度。

Answer 1

尝试使用示例字符串：abcd和bc并逐步完成。

你会发现这两行存在问题：

x = lcs(string1, string2[0..-1])
y = lcs(string1[0..-1], string2)

因为'abcd'[0..-1] -> 'abcd'所以当它到达x = ...行时，它就会被卡住。

应该是[0..-2]。

另外，您可以使用if string1.chars.none? { |x| string2.include?(x)} 替换if !(string1.chars.any? { |x| string2.include?(x)})

最常见的子序列 - 我的无限循环在哪里？

1 个答案: