1

首先,这是我的代码有问题的部分;这些是非常基础的课程

public Passenger(String Name, String adress, String number, String password){
    count++;
    accId+=count;

    this.Name=Name;

    this.adress=adress;
    this.number=number;

    if(checkPw(password)==true){
        this.password=password;
    }

}

private boolean checkPw(String password){
    int length;
    length = password.length();

    if(length != 6){
        return false;
    }
    else if(password.charAt(0)==0){
        return false;
    }
    else {
        for (int i = 0; i < password.length();i++){
            if((password.charAt(i))==(password.charAt(i+1))){
                return false;
            }
        }
    }
    return true;        
}

测试类:

public static void main(String[] args){
    Passenger gokhan=new Passenger("Gokhan","Istanbul","xxx","254651");

    System.out.println(gokhan.password);
}

所以,我认为问题出在乘客班上。这是我第一次在课堂上上课(我的意思是 if(checkPw(password)==true) 部分)。在测试类中,看起来很清晰,没想到会出现这个错误。如何避免此消息?

完整错误:

Exception in thread "main" java.lang.StringIndexOutOfBoundsException: String index out of range: 6
    at java.lang.String.charAt(String.java:658)
    at project1.Passenger.checkPw(Passenger.java:45)
    at project1.Passenger.<init>(Passenger.java:27)
    at project1.testClass.main(testClass.java:11)

Java 结果:1

4

2 回答 2

5

问题在这里:

for (int i = 0; i < password.length();i++){
    if((password.charAt(i))==(password.charAt(i+1))){
        return false;
    }
 }

当您在最后一次迭代中时,您试图访问不存在charstring位置。i+1

    text
       ^
       |
when i = 3 charAt(i) will return t and charAt(i+1) will throw the exception
于 2013-10-29T22:28:49.030 回答
2

这条线似乎是问题所在:

if((password.charAt(i))==(password.charAt(i+1))){

for循环的最后一次迭代中,iis 5andi+1或 or6离开字符串的末尾,因为索引的范围从0to length() - 1。这里的解决方案是在for倒数第二个字符而不是最后一个字符之后停止循环迭代。改变

for (int i = 0; i < password.length();i++){

for (int i = 0; i < password.length() - 1; i++){

所以循环中的最大值是i,所以or不在字符串的末尾。for4i+15

于 2013-10-29T22:30:57.530 回答