StringIndexOutOfBoundsException

Question

我正在编写的这个程序没有编译正确，我不断收到错误： 线程“main”中的异常java.lang.StringIndexOutOfBoundsException：字符串索引超出范围：-9 at java.lang.String .charAt（String.java:687） 在pro1.main（pro1.java:161）

这里是我的代码：

import java.io.*; 
import java.util.*; 
public class pro1 { 
    static String str=""; 
    static String str1=""; 
    static int range=250; 
    static int length; 
    static String key=""; 
    static String ep=""; 
     static String pt=""; 
    static char temp; 
    static int t,p,h; 
    static int r,x,y,z,w; 
    static Random generator = new Random(); 
static public String getContents(File aFile) 
    { 
    StringBuilder contents = new StringBuilder(); 
    try { 
     BufferedReader input = new BufferedReader(new FileReader(aFile)); 
     try { 
     String line = null; 
     while ((line = input.readLine()) != null){ 
      contents.append(line); 
      contents.append(System.getProperty("line.separator")); 
     } 
     } 
     finally { 
     input.close(); 
     } 
    } 
    catch (IOException ex){ 
     ex.printStackTrace(); 
    } 
    str1=contents.toString(); 
    return str1; 
    } 



public static void main (String args[]) throws IOException { 
     File testFile = new File("/home/amritha/Desktop/sam.txt"); 
     System.out.println("Original file contents: " + getContents(testFile)); 
     System.out.println("message:"+str1); 
     String sbox="abcdefghijklmnopqrstuvwxyz"; 
     length=str1.length()-1; 
     for(int i=0;i<length;i++) 
     { 
      t=(int)str1.charAt(i); 
      if(t==32) 
       { 
       int t1=32; 
       temp=(char)t; 
       } 
      else 
       { 
       range=generator.nextInt(26)+1; 
       temp=sbox.charAt(range); 
       } 
       key+=""+temp; 
     } 
     System.out.println("Key:"+key); 
     for(int i=0;i<length;i++) 
     { 
     t=(int)str1.charAt(i); 
      { 
      if(t==32) 
      { 
       t=32; 
       temp=(char)t; 
      } 
      else 
      { 
       t-=97; 
      } 
      } 
     p=(int)key.charAt(i); 
      { 
      if(p==32) 
      { 
       p=32; 
       temp=(char)p; 
      } 
      else 
      { 
       p-=97; 
      } 
      } 
     if((t==32)&&(p==32)) 
      { 
       int v=32; 
       temp=(char)v; 
      } 
      else 
      { 
      r=(t+p)%26; 
      temp=sbox.charAt(r); 
      } 
      ep+=""+temp; 
     } 
     System.out.println("Encrypted Text:"+ep); 

    for(int i=0;i<length;i++) 
     { 
     y=(int)ep.charAt(i); 
      { 
      if(y==32) 
      { 
       y=32; 
       temp=(char)y; 
      } 
      else 
      { 
       y-=97; 
      } 
      } 
     x=(int)key.charAt(i); 
      { 
      if(x==32) 
      { 
       x=32; 
       temp=(char)x; 
      } 
      else 
      { 
       x-=97; 
      } 
      } 
     if((x==32)&&(y==32)) 
      { 
       int w=32; 
       temp=(char)w; 
      } 
      else 
      { 
      z=(y-x)%26; 
      temp=sbox.charAt(z); 
      } 
      pt+=""+temp; 
     } 
     System.out.println("deccrypted Text:"+pt); 
    } 
} 

Answer 1

你的代码看起来腥以各种方式，我无法想象任何人想要阅读这段代码的170行。

看看例外情况：它告诉你到底发生了什么问题：你将charAt（）作为索引传递给-9，显然超出了范围，因为你应该只传递0 ...（length- 1）在那里。

它也给你行号......所以去行号161，看看里面有什么，以及它是如何到达那里。

Answer 2

我的猜测是它有事情做这一行：

z=(y-x)%26; 

如果x比y的％操作的结果可能是负的较大（或0）。 charAt（这是z给出的参数）预期为正值。 你应该尝试：

z=Math.abs(y-x)%26; 

编辑：作为一个方面说明，这不应该是很难，因为指出别人找出你自己，通过查看异常，或者在最坏的情况下使用调试器并准确查看不同变量具有的值以及发生异常时的原因。