Question

I’ve got a very annoying problem with some code throwing an IndexOutOfBoundsException and I really cannot understand why. The logcat points to the “addTimetableItem” of the following code which ill explain more on:

if(sortedFridayTimes.size()>0){
    insertDay("Friday");
    for(int i=1; i<sortedFridayTimes.size()+1;i++){
        addTimetableItem(sortedFridayTimes.get(i));
    }
}

“sortedFridayTimes” is an ArrayList containing my own “Timetable Entry” objects which I have sorted into order already. First the size is checked to see if there are any objects and if there is then “insertDay” runs which creates a new textview for a title and adds it to the layout (This works fine..).
Inside the for loop the idea is to then add all the objects from the arraylist into the layout. Now I know that the “addTimetableItem” code works as ive tested it already, but my problem is that i cant seem to get the last object out of the arraylist. If I declare the for loop to only run for

"i<sortedFridayTimes.size()" 

then the program runs fine but I don’t get the last entry in the arraylist which I know exists because I’ve debugged and watched my variables. On adding the “+1” as shown above I now get the IndexOutOfBoundsException and I really don’t know why. As I’ve said, I’ve debugged and I know that an entry exists in the arraylist where I’m trying to point to, but it just crashes. I can provide more code if needs be, but does anyone have any ideas please?

Answer 1

i<sortedFridayTimes.size()+1

You are looping past the last element in the array. Why the +1?

If there are N elements in the array, then the elements are from indexes 0 through to N-1.

So it should be:

for(int i=0; i<sortedFridayTimes.size(); i++) {