字符串模式搜索算法堆栈溢出

Question

我正在尝试将this code 修改为递归版本，以提高执行时间效率。代码所做的是在 Text 中查找所有模式的第一个索引并将它们添加到 ArrayList。我的代码如下：

public boolean search(String text, int n, String pattern, int d) {
    if (n > text.length() || n < d) return true;
    while (d >= 0 && d < pattern.length() && n < text.length() && text.charAt(n) != pattern.charAt(d)) d = next[d];
    if (d == pattern.length()) {
    (***) if (ans == null || !ans.contains(n-d)) ans.add(n-d);                    
          n = n-d;
          d = -1;
    }
    if( n < text.length() && n >= d ) search(text, n+1, pattern, d+1);
    return false;
}

我的代码的循环版本：

public void search(String text) {
int m = this.pattern.length();
int n = text.length();
int i, j;
for (i = 0, j = 0; i < n && j < m; i++) {
    while (j >= 0 && text.charAt(i) != this.pattern.charAt(j)) {
        j = next[j];
    }
    j++;
    if (j >= m) {
    if (ans == null || !ans.contains(i-m+1) ) ans.add(i-m+1);     
        j = 0;
        i = i - m + 1;
    }
}}

更糟糕的是，它是指数级的，这就是我无法满足时间限制的原因。

数组next[] 与示例代码相同。我的问题是：

1. 当参数文本字符串变大时，比如超过 2200 个字符，它总是引发java.lang.StackOverflowError；特别是在括号内的三个星号的行中。即使只剩下ans.add(n-d)，仍然是同样的问题。返回值boolean 毫无意义，即使我将函数return 类型更改为void 并删除最后一行，它仍然是同样的问题。

2. 有没有更好的方法来做到这一点？

Answer 1

目前 JVM (Hotspot) 没有尾调用优化。所以每次递归都会消耗栈。所以你得到stackoverflow错误。目前最好坚持迭代方法。