新手程序员需要建议：“字符串索引超出范围” - 爪哇

Question

我对编程还很陌生，并且遇到了一个错误，我确信对于更有经验的人来说这是一个简单的修复方法。

这是我所得到的：

    import java.io.*;
    import java.util.Scanner;

    public class ReadNamesFile
    {
        public static void main(String[] args) throws IOException {
        // make the names.csv comma-separated-values file available for reading
        FileReader f = new FileReader("names.csv");
        BufferedReader r = new BufferedReader(f);
        //
        String lastName="unknown", firstName="unknown", office="unknown";

        // get first line
        String line = r.readLine();

        // process lines until end-of-file occurs
        while ( line != null )
        {
           // get the last name on the line
           //
           // position of first comma
           int positionOfComma = line.indexOf(","); 
           // extract the last name as a substring
           lastName = line.substring(0,positionOfComma); 
           // truncate the line removing the name and comma
           line = line.substring(positionOfComma+1); 

           // extract the first name as a substring
           firstName = line.substring(0,positionOfComma); 
           // truncate the line removing the name and comma
           line = line.substring(positionOfComma+1);

           // extract the office number as a substring
           office = line.substring(0,positionOfComma);
           // truncate the line removing the name and comma
           line = line.substring(positionOfComma+2);
           //
           //        
           //
           // display the information about each person
           System.out.print("\nlast name = "+lastName);
           System.out.print("\t first name = "+firstName);
           System.out.print("\t office = "+office);
           System.out.println();
           //
           // get the next line
           line = r.readLine();
        }
    }
}

基本上，它会在 .csv 文件中查找姓氏、名字和办公室号码并将其打印出来。

当我编译时，我没有收到任何错误，但当我运行它时，我得到：

java.lang.StringIndexOutOfBoundsException: String index out of range: 7
at java.lang.String.substring(String.java:1955)
at ReadNamesFile.main(ReadNamesFile.java:34)

在尝试执行办公室号码部分之前，前两个（姓氏和名字）打印得很好，但办公室号码似乎不起作用。

有什么想法吗？

编辑：感谢所有的帖子，但我仍然无法真正弄清楚。有人可以发布一些真正愚蠢的东西吗？我已经尝试解决这个问题一个小时了，但我无法解决。

Answer 1

让我们通过示例来了解您的代码存在哪些问题。

例如：行：溢出，堆栈
{ 长度：14 }

逐行获取程序语句 -

int positionOfComma = line.indexOf(",");  // returns 9

lastName = line.substring(0,positionOfComma); // should be actually postionOfComma-1

现在lastName 发生溢出。 positionOfComma 有 9。

line = line.substring(positionOfComma+1);

现在line有stack。

firstName = line.substring(0,positionOfComma); 

询问从 0 到 9 的子字符串。但stack的长度仅为5。这将导致字符串索引超出范围异常。希望你明白自己哪里做错了。

Answer 2

来自 JavaDoc：

(StringIndexOutOfBoundsException) - 由 String 方法抛出
指示索引为负数或大于大小
字符串。

在您的情况下，对 .substring 的调用之一被赋予一个 >= 字符串长度的值。如果第 #34 行是注释，那么它就是 #34 上面的行。

Answer 3

您需要：

a) 如果您找不到逗号（即，如果您无法找到并提取姓氏和/或名字字符串），请确保处理该情况

b) 确保“positionOfComma + N”的值永远不会超过字符串的长度。

几个“if”块和/或“Continue”语句就可以很好地实现这一目的;-)

Answer 4

您正确地找到了 positionOfComma，但该逻辑适用于 line 的原始值。当您删除姓氏和逗号时，positionOfComma 不再正确，因为它适用于旧的行值。

Answer 5

int positionOfComma = line.indexOf(",");

这行代码可能找不到逗号，然后positionOfComma将为-1。接下来，您可以使用 (0,-1) 来对某些内容进行子串 - 难怪它会给出 StringIndexOutOfBoundsException。使用类似的内容：

int positionOfComma = 0;
if(line.indexOf(",")!=-1)
{
   positionOfComma = line.indexOf(",");
}

有时您确实必须进行大量检查，尤其是当数据被破坏时:(

http://download.oracle.com/javase/1.4.2/docs/api/java/lang/String.html#indexOf(java.lang.String)

PS我是当然，聪明的人可以让我的编码看起来很破旧，但你明白我希望的意思:)