Getting error "ArrayIndexOutofBound"

[Yatin Shah]

My Problem is resolved.

[Rob Spoor]

The exception comes from while ( (S[rightIndex] >= pivot) && (c.compare(S[rightIndex], pivot)>=0)). The reason is that you never check the bounds. Change both loops to make sure they don't go out of bounds:

while ( leftIndex < rightIndex && (S[leftIndex] <= pivot) && (c.compare(S[leftIndex], pivot)<=0) )

while ( leftIndex < rightIndex && (S[rightIndex] >= pivot) && (c.compare(S[rightIndex], pivot)>=0))

This will get rid of the exception, although now there will be an infinite loop. I didn't check why tet.


The Comparator class should be made generic (using Number as bound wildcard), or use Integer as parameter types. That way you don't need to cast all these values. Also, Integer implements Comparable<Integer> so you can change your compare method to this:

public int compare(Integer v1, Integer v2) {
    return v1.compareTo(v2);
}

[Randall Twede]

rob, you are my hero. i sure don't have the patience to read through all that code to find the problem. unless i wrote myself that is

[Yatin Shah]

Thanks, I did it some different way. I really appreciated your help. Thanks once again.

[Campbell Ritchie]

Randall Twede wrote: . . . all that code . . .

What code?

Yatin Shah, you do not delete code after it has been replied to