0

我已经为我的 CS 类编写了代码来识别用户输入的字符串是否是回文。我已经让代码工作了。但是,每当我执行代码时,我都会收到以下消息:

"Exception in thread "main" java.lang.StringIndexOutOfBoundsException: String index out of range: -1
    at java.lang.String.charAt(String.java:658)
    at palindrome.Palindrome.main(Palindrome.java:14)"

我认为我的有问题lengthi但我不确定是什么。我的代码如下。我用于aabbaa用户输入的字符串。

package palindrome;

import java.util.Scanner;

public class Palindrome {

    public static void main(String[] args) {
        System.out.println ("Enter A String:");
        Scanner input = new Scanner (System.in);
        String s = input.nextLine();
        int length = s.length();
        int i = 0;
        while (length>= 0) {
            if (s.charAt(i) == s.charAt(length-1)) { 
                i++;
                length--;
                if (length == i) {
                    System.out.println ("Your string is a palindrome");
                    length = 0;
                }
            }
            else {
                System.out.println ("Your string is not a palindrome");
                length = 0;
            }
        }
    }
}
4

6 回答 6

2

当长度等于 0 时,while 循环中的第一个 if 语句会尝试获取 s.charAt(0-1)、iescharAt(-1),这就是发生错误的地方。

也许尝试以下(未测试):

public static void main(String[] args) {
System.out.println ("Enter A String:");
Scanner input = new Scanner (System.in);
String s = input.nextLine();
int length = s.length();
int i = 0;
while (length > 0) {
    if (s.charAt(i) == s.charAt(length-1)) { 
        i++;
        length--;
        if (length < 1) {
            System.out.println ("Your string is a palindrome");
            break;
        }
    }
    else {
        System.out.println ("Your string is not a palindrome");
        break;
    }
}
于 2013-02-13T02:28:07.933 回答
1

您编写while(length >= 0) {}并在您编写的main方法中length = 0;尝试循环。

那应该是while(length > 0){}

于 2013-02-13T02:29:45.673 回答
1

只需创建一个如下所示的方法并将您的字符串作为参数传递来测试字符串是否为回文。如果字符串是回文,方法将返回 true

想法是将您的字符串反转并与原始字符串进行比较,如果两者相同,则它是回文。

public static boolean isPalindrome(String str) {
    return str.equals(new StringBuffer().append(str).reverse().toString());
}
于 2013-02-13T03:21:39.103 回答
0

当您在那个时候迭代while循环时,长度值变为负数,并且数组索引从这0就是为什么您将异常作为StringIndexOutOfBoundsException.

你应该试试这个,它会很好用。

import java.util.Scanner;

public class Palindrome
{
    public static void main(String[] args)
    {
        System.out.println("Enter A String:");
        Scanner input = new Scanner(System.in);
        String s = input.nextLine();
        int length = s.length();
        int i = 0;
        while (length >= 0)
        {
            if (s.charAt(i) == s.charAt(length - 1))
            {
                i++;
                length--;
                if (length-1 == i)
                {
                    System.out.println("Your string is a palindrome");
                    length = 0;
                    break;
                }
            }
            else
            {
                System.out.println("Your string is not a palindrome");
                length = 0;
                break;
            }
        }
    }
}
于 2014-04-29T10:54:45.270 回答
0

你应该做这样的事情:

package palindrome;

import java.util.Scanner;

public class Palindrome {

    public static void main(String[] args) {
        System.out.println ("Enter A String:");
        Scanner input = new Scanner (System.in);
        String s = input.nextLine();
        if ( s.equals(new StringBuffer(s).reverse().toString()) ){
            System.out.println ("Your string is a palindrome");
        else
            System.out.println ("Your string is not a palindrome");
    }
}
于 2014-04-29T11:24:41.973 回答
0

您的问题是,在每一步中,您都会迈出两步,但只会将长度减少 1

所以一个踪迹:用户输入foooof.

length = 6
i = 0
f == f
i = 1
length = 5
o == o
i = 2
length = 4
o == o
i = 3
length = 3 (You've now passed yourself)
o == o
i = 4
length = 2
length-i = -2 =>     index out of range
于 2013-02-13T02:27:31.603 回答