ArrayIndexOutOfBoundsException - help debug ?

Question

Can someone help me debug this? I keep getting the out of bounds exception and I've tried adding 1 to the length....
     public int[] AscendingSort(int[] Students)
               int temp = 0;
               for(int i = 0 ; i < Students.length ; i++)
                         if(Students[i] > Students[i + 1])
                              temp = Students[i] ;
                              Students[i] = Students[i + 1];
                              Students[i + 1] = temp;
          return Students;
          }

Answer 1

Suppose your array is like this
3,2,1.
So
for(int i = 1 ; i > Students.length ; i++)this line evaluates to
for(int i = 1 ; i > 3 ; i++)which is true.So the control enters the loop.
Now the next line is
if(Students[i] < Students[i - 1])which inturn evaluates to
if(Students[1] < Students[0])i.e
if(2<3)which again evaluates to true.So the control now enters the if block
Now the block of code that follows i.e
temp = Students[i] ;     
Students[i] = Students[i + 1];
Students[i + 1] = temp;evaluates to
temp = Students[1] ;     
Students[1] = Students[2];
Students[2] = temp;which inturn evaluates to
temp = 2;     
Students[1] = 1;
Students[2] = 2;So instead of swaping 3 and 2 (Students[0] and Students[1]) your code is swapping 2 and 1 (Students[1] and Students[2])
So what you need to do is to swap only those array variables that you are checking
Again if you are going for sorting then you probably would be needing another nested loop
which would look something like this
for(int i = 0 ; i <Students.length-1 ; i++)
         for(int j = i+1;j < Students.length;j++)
                         if(Students[j] < Students) { temp=Sutdents[i];     
                              Students[i] = Students[j];
                              Students[j] = temp;
Now I believe it would sort the array
Edited by: phoenix_frm_ashes on Feb 10, 2009 11:36 PM
Edited by: phoenix_frm_ashes on Feb 10, 2009 11:39 PM
Edited by: phoenix_frm_ashes on Feb 10, 2009 11:40 PM