Как исправить "java.lang.ArrayIndexOutOfBoundsException: 1" injava - PullRequest
Новая
       18

Как исправить "java.lang.ArrayIndexOutOfBoundsException: 1" injava

0 голосов
/ 03 ноября 2019

Я делаю шифр замещения с использованием периодической таблицы, и я столкнулся с некоторыми проблемами, пытаясь определить некоторую переменную с индексами массива.

Я пытался использовать некоторые условные операторы, чтобы обойти проблему, но всегда кажется, что она разрушает логику и приводит к сбою приложения. 

public class ENcryptionProcedures2 {
    public static  String strOriginal;
    public static String[] arrofStr;
    public static boolean val;
    public static int newval;

    private static final String[] arrPeriodicTableValues = {"h", "he", "li", "be", "b", "c", "n", "o", "f", "ne", "na", "mg", "al", "si", "p", "s", 
        "cl", "ar", "k", "ca", "sc", "ti", "v", "cr", "mn", "fe", "co", "ni", "cu", "zn", "ga", "ge", "as", "se", "br", "kr", "rb", "sr", "y", "zr", "nb", "mo", 
        "tc", "ru", "rh", "pd", "ag", "cd", "in", "sn", "sb", "te", "i", "xe", "cs", "ba", "la", "ce", "pr", "nd", "pm", "sm", "eu", "gd", "tb", "dy", "ho", "er", "tm", "yb",
        "lu", "hf", "ta", "w", "re", "os", "ir", "pt", "au", "hg", "tl", "pb", "bi", "po", "at", "rn", "fr", "ra", "ac", "th",
        "pa", "u", "np", "pu", "am", "cm", "bk", "cf", "es", "fm", "md", "no", "lr", "rf", "db", "sg", "bh", "hs", "mt", "ds", "rg", "cn", "nh", "fl", "mc", "lv", "ts", "og"};

    public static String sc(){
        Scanner sc = new Scanner(System.in);
        String strOriginal = sc.nextLine();
        return strOriginal;
    }

    public static int scI(){
        Scanner sc = new Scanner(System.in);
        int sh = sc.nextInt();
        return sh;
    }

    private static int checkingfunc(String letters){
        int num = 0;
        for(int f=0; f<=117; f++){
            if(arrPeriodicTableValues[f].equalsIgnoreCase(letters)){
               num= f+1;
               return num;
            }  else{
             num = 0;
            }
        }
        return 0;
    }

    public static String Periodicencrypt(String strOriginal, String[] arrofStr){

    String encryptedString=""; 

    arrofStr = strOriginal.split("");

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

        String oneletter = arrofStr[i];
        String twoletters = arrofStr[i]+arrofStr[i+1];
        System.out.println(twoletters);

        if(checkingfunc(twoletters)!=0){
           encryptedString= encryptedString + Integer.toString(checkingfunc(twoletters));

        } else if(checkingfunc(oneletter)!=0){
           encryptedString=encryptedString + Integer.toString(checkingfunc(oneletter));
        } else{
           encryptedString=encryptedString +oneletter;
        }
    }
    return encryptedString;
}

Ответы [ 2 ]

4 голосов
/ 03 ноября 2019

Итерация массива превышает его размер, измените 

for(int i=0; i<=iLen; i++ ){

на 

for(int i=0; i<iLen; i++ ){

Редактировать:

Другая проблема может быть здесь

String twoletters = arrofStr[i]+arrofStr[i+1];

когда ваше значение i равно max (iLen-1), это также приведет к ошибке, вам нужно исправить свой алгоритм здесь

0 голосов
/ 03 ноября 2019

В дополнение к тому, что написал Шайлеш Чандра. Вы также можете использовать foreach следующим образом: 

for(String st: someStringArray ){
    do something with "st"
}

В этом методе вы будете перебирать все строки в массиве одну за другой, и вам не придется беспокоиться о выходе за пределы

...