java.lang.StringIndexOutOfBoundsException：字符串索引超出范围

问题描述：

嗨我写了一个Java代码来查找由其他字组成的最长的单词。我的逻辑是从文本文件中读取单词列表，并将每个单词添加到数组中（在文本中，单词是排序的，每行只会有一个单词）之后，我们检查数组中的每个元素是否其他元素作为子字符串。如果是这样，我们计算子串的数量。具有最多子字符串的元素将成为结果java.lang.StringIndexOutOfBoundsException：字符串索引超出范围

代码正在运行时，我给一个文本文件只有两个字。但是，当有两个以上的字我收到以下错误

java.lang.StringIndexOutOfBoundsException：字符串索引超出范围：3

我觉得错误是在这行，如果（s.charAt（I1存在的）== w.charAt（J1））

import java.util.*; 
    import java.io.*; 
    import java.lang.reflect.Array; 
    public class Parser 
    { 
public static void main (String[] args) throws IOException 
{ 
    String [] addyArray = null; 

    FileReader inFile = new FileReader ("sample.txt"); 
    BufferedReader in = new BufferedReader (inFile); 
    String line = ""; 
    int a = 0; 
    int size=0; 
    String smallestelement = ""; 

    while(in.ready()) 
    { 
     line=in.readLine(); 
     while (line != null && line != "\n") 
     { 
      size++; 
      System.out.println(size); 
      line = in.readLine(); 
      if (line == null) line = "\n"; 
     } 
    } 
    addyArray = new String[size]; 
    FileReader inFile2 = new FileReader ("sample.txt"); 
    BufferedReader in2 = new BufferedReader (inFile2); 
    String line2 = ""; 

    while(in2.ready()) 
    { 
     line2 = in2.readLine(); 


     while (line2 != null && line2 != "\n") 
     { 

      addyArray[a] = line2; 


      System.out.println("Array"+addyArray[a]); 
      line2 = in.readLine(); 
      a++; 
      if (line2 == null) line2 = "\n"; 
     } 

    } 


    int numberofsubstrings=0; 
    int[] substringarray= new int[size]; 

    int count=0,no=0; 

for(int i=0;i<size;i++) 
{  
    System.out.println("sentence "+addyArray[i]); 
    for(int j=0;j<size;j++) 
    { 
     System.out.println("word "+addyArray[j]); 

     String w,s; 
     s=addyArray[i].trim(); 
     w=addyArray[j].trim(); 

     try{ 
      for(int i1=0;i1<s.length();i1++) 
      { 
       if(s.equals(w)&& s.indexOf(addyArray[j-1].trim()) == -1) 
       {} 
      else 
      { 
       if(s.charAt(i1)==w.charAt(0)) 
       {     
        for(int j1=0;j1<w.length();j1++,i1++) 
        { 
        if(s.charAt(i1)==w.charAt(j1)) //I feel the error is occuring here 
        { count=count+1;} 
         if(count==w.length()) 
         {no=no+1;count=0;}; 

        } 
       } 
       } 
      } 
       System.out.println(no); 
     } 
     catch(Exception e){System.out.println(e);} 
     substringarray[i]=no; 
     no=0; 

     } 
    } 



     for(int i=0;i<size;i++) 
     { 
      System.out.println("Substring array"+substringarray[i]); 
     } 
    Arrays.sort(substringarray); 
    int max=substringarray[0]; 

    System.out.println("Final result is"+addyArray[max]+size); 

} 
    }

答

这就是问题所在：

for(int j1=0;j1<w.length();j1++,i1++)

通过循环每次迭代，你递增i1一以及j1。 i1可能已经在s的末尾，所以在你增加它之后，s.charAt(i1)将会失效。

两个旁白：

你应该看看String.regionMatches
使用一致缩进和明智的空白可以使你的代码多更容易阅读。

答

当您使用string.charAt（x）时，您必须检查它是否超出字符串长度。 Documentation表明如果索引参数为负数或不小于此字符串的长度，您将得到“IndexOutOfBoundsException”。而在你的特殊情况下，你只是在循环中验证你正在使用的长度，所以它会失败。

正如SO已经说过的，循环只考虑w的长度，所以如果你有更短的s，它会引发这个异常。检查条件，以便更短的字符串或重新考虑过程。

是的，每次做charAt检查都可以解决问题，只会在问题上提出一个绷带。这不是问题的根本原因。因此，我不会推荐使用该解决方案。这也可能导致代码后面的其他问题。 – 2012-02-09 23:09:50

嗯，并不意味着每一次迭代。猜猜我必须编辑 – Alfabravo 2012-02-09 23:11:01

答

一些提示：

首先，总是包含完整的堆栈跟踪，当你所要求的调试帮助。它应该指向问题发生的确切行号。

其次，你的问题很可能在你最内环for(int j1=0;j1<w.length();j1++,i1++)你除了j1递增i1这将导致i1最终超越串的大小s

最后，你应该考虑使用String.contains()方法对于字符串甚至正则表达式。

java.lang.StringIndexOutOfBoundsException：字符串索引超出范围

相关推荐