java - 回文程序输出超出范围

Question

这个问题已经有答案了:

How do I compare strings in Java? (23 个回答)

已关闭 9 年前。

我对编程很陌生。你能告诉我为什么会输出:

string index out of range: -1

class Palindrome {

    public static String reverse(String s) {
        if (s == "") {
            return s;
        }
        else {
            return reverse(s.substring(1,s.length())) + s.charAt(0);
        }
    }

    public static void main(String args[]) {
        System.out.print(reverse("galskjdf"));
    }
}   

Answer 1

使用

class Palindrome {
  public static String reverse(String s) {
    if ("".equals(s)) {
      return s;
    } else {
      return reverse(s.substring(1,s.length())) + s.charAt(0);
  }
}

  public static void main(String args[]) {
    System.out.print(reverse("galskjdf"));
  }
}   

参见How do I compare strings in Java?和 Reverse a string in Java可能也会有帮助(即使您可能将上述代码作为编程练习来完成，但知道通常人们会使用现有的工具来完成此类事情也没有什么坏处)。

使用原因

"".equals(s)

而不是

s.equals("")

如下:如s是 null ，后者会抛出 NullPointerException (因为您尝试在 null 上调用方法)，而当您在“正确”字符串上调用方法时，前者仍然有效，另请参阅 Gracefully avoiding NullPointerException in Java然而，我记得也有人批评这种方法，因为这可能会让你错过 s 的事实。是 null当它不应该的时候(在这种情况下你应该明确检查处理这种情况)。