Question

您好，我试图编写QuickSort，但是我总是遇到索引超出范围的问题吗？我的代码如下：

public class QuickSort
{
    public void quickSort(ArrayList<Integer> A, int p, int r)
    {
        if (p < r) {
            int q = partition(A, p, r);
            quickSort(A, p, q - 1);
            quickSort(A, q + 1, r);
        }
    }
    public int partition(ArrayList<Integer> A, int p, int r) {
        int x = A.get(r);
        int i = p - 1;
        for (int j = p ; j < r; j++) {
            if (A.get(j) <= x) {
                i++;
                Collections.swap(A, A.get(i), A.get(j));
            }
        }
        Collections.swap(A, A.get(i + 1), A.get(r));
        return (i + 1);
    }
}

我正在使用书中的代码：“算法简介”

我正在尝试对ArrayList A进行快速排序

public class TestDriver
{
    public static void testQuick() {
        //Laver et random array A
        ArrayList<Integer> A = new ArrayList<>();
        for (int i = 1; i <12; i++) {
            A.add(i);
        }
        Collections.shuffle(A);
        int n = A.size();
        QuickSort qs = new QuickSort();
        System.out.println("The Array");
        System.out.println(A);
        qs.quickSort(A, 0, (n - 1));
        System.out.println("The Array after QuickSort");
        System.out.println(A);
        System.out.println("");

    }
}

Answer 1

问题是Collections.swap(A, A.get(i), A.get(j));-这将尝试使用A.get(i)中的值作为列表中的索引，并且如果i中的值显然会引发异常大于A.size()。

因此，仅将其替换为您想交换的职位：

Collections.swap(A, i, j);

和

Collections.swap(A, (i + 1), r);

QuickSort IndexOutOfBound异常arraylist

1 个答案: