IndexOutOfBoundsException error добавление длинных чисел

Question

Я пытаюсь wrtie java-приложение, которое добавляет до 5 длинных чисел, используя LinkedLists. В конце пробега я получаю это:

Исключение в потоке "основной" java.lang.IndexOutOfBoundsException: Индекс: 0 Размер: 0 в java.util.LinkedList.checkElementIndex (LinkedList.java: 555) в java.util.LinkedList.remove (LinkedList.java:525) при Assignment1.LongNumbers.remove (LongNumbers.java:33) при Assignment1.LongNumbers.main (LongNumbers.java:92)

Введите код:

import java.util.*; 
/** 
* 
* @author ..... 
*/ 

public class LongNumbers 
{ 
private List<Integer> [] theLists; 
public LongNumbers() { 
    this.theLists = new LinkedList[6]; 
    for (int i=0; i<6; i++) 
    this.theLists[i]= new LinkedList<>(); 
} 

public void add(int location, int digit) { 
    //add digit at head of LinkedList given by location 
    theLists[location].add(digit); 
} 

public int remove(int location) { 
    //remove a digit from LinkedList given by location 
    return theLists[location].remove(location); //LongNumbers.java:33 
} 

public boolean isEmpty(int location) { 
    //check for an empty LinkedList given by location 
    return theLists[location].isEmpty(); 
} 

public static void main(String[] args) { 
    Scanner stdIn = new Scanner(System.in); 

    //Local Variables 
    int digit; 
    int carry = 0; 
    int numberAt = 0; 
    int largestNumLength = 0; 
    char[] digits; 
    String number; 
    boolean userWantstoQuit = false; 
    LongNumbers Lists = new LongNumbers(); 

    System.out.println("The program will enter up to 5 numbers and add them up."); 
    System.out.println(); 

    while(!userWantstoQuit && numberAt != 5){ 
     System.out.print("Enter a number, enter -1 to quit entry phase: "); 
     number = stdIn.nextLine(); 

     if((number.compareTo("-1")) == 0) 
      userWantstoQuit = true; 
     else{ 
      digits = new char[number.length()]; 
      for(int i=0;i<number.length();i++) 
       digits[i] = number.charAt(i); 
      for(int i=0;i<number.length();i++){ 
       int tempValue = digits[i] - 48; 
       try{ 
        Lists.add(numberAt, tempValue); 
       } 
        catch(NumberFormatException nfe){ 
         System.out.println("Invalid Input. Please try again."); 
         break; 
        } 
       if(i == (number.length() - 1)) 
        numberAt++; 
       if(number.length() > largestNumLength) 
        largestNumLength = number.length(); 
      } 
     } 
    } 

    for(int j=0;j<largestNumLength;j++){ 
     int tempDigit = 0; 
     int index = 0; 

     while(index < numberAt){ 
      if(Lists.theLists[index].get(0) != null){ 
       tempDigit += Lists.theLists[index].get(0); 
       Lists.remove(0); //LongNumbers.java:99       
      } 
     index++; 
     } 

     digit = carry + tempDigit; 

     if(j < numberAt){ 
      carry = digit/10; 
      digit = digit%10; 
     } 
    Lists.add(5, digit); 
    } 

    System.out.print("The sum of the numbers is: "); 

    for(int i=0;i<Lists.theLists[5].size();i++){ 
     System.out.print(Lists.theLists[5].get(i)); 
    } 

    System.out.println(); 
    System.out.println(); 
    System.out.println(); 

}//end main 
}//end class 

Answer 1

Для начала, я не думаю, что вы можете иметь массив объектов List<E> ...

Вы также должны убедиться, что ваш список инициализируется и имеет элемент в данном location.

Так что ваш метод может выглядеть следующим образом:

public int remove(int location) 
{ 
    if(theLists != null) 
      if(theLists.size() > location) 
       return theLists.remove(location); 
    return 0; 
} 

Если вам нужны 2 размеры списков, вы можете попробовать использовать List<List<E>>

Обрабатывать все E в Integer.

Answer 2

Посмотрите на код здесь:

while(index < numberAt){ 
      if(Lists.theLists[index].get(0) != null){ 
       tempDigit += Lists.theLists[index].get(0); 
       Lists.remove(0); //LongNumbers.java:99       
      } 
     index++; 
     } 

Вы проверяете ли первый элемент index -м список не пуст. Если это так, вы добавляете его и вызываете метод remove. Однако, что, если вы уже обработали первый список и index 'th значение 1? В этом случае theLists[1].get(0) != null истинно, но Lists.remove(0) проходит 0 как location. Посмотрите на этот код:

public int remove(int location) { 
    //remove a digit from LinkedList given by location 
    return theLists[location].remove(location); //LongNumbers.java:33 
} 

В сценарии я описал, location равно 0. Но ваш 0'th список уже пуст ...

EDIT: Перепишите метод remove, как это :

public int remove(int location, int index) { 
    //remove a digit from LinkedList given by location 
    return theLists[index].remove(location); //LongNumbers.java:33 
} 

И всякий раз, когда вы вызываете этот метод, передать index списка для работы с. Пример:

while(index < numberAt){ 
      if(Lists.theLists[index].get(0) != null){ 
       tempDigit += Lists.theLists[index].get(0); 
       Lists.remove(0, index); //LongNumbers.java:99       
      } 
     index++; 
     } 

Наконец: В будущем, пожалуйста, структура ваш код, это была реальная боль, чтобы прочитать его в этом, неструктурированный состояние, прочитать о том, как код.