Question

所以我有一个任务，我必须在大量随机生成的列表上运行不同的排序算法。然后，我必须提交一份报告，比较各种算法的运行时间。到目前为止，我已经编写了3种排序算法的代码：quicksort，mergesort和heapsort。我只剩下radix-sort。下面是代码。这段代码在这一行上给我一个ArrayIndexOutOfBoundsException：

b[--bucket[(a[i] / exp) % 10]] = a[i];

但我无法弄清楚如何更改代码以使其正确。

public class RadixSort {

    public static void main(String[] args) {
        Random generator = new Random( System.currentTimeMillis() );
        Scanner scan = new Scanner(System.in);
        int size = scan.nextInt();
        int[] x = new int[size];

        long start = System.currentTimeMillis();

        for (int i = 0; i < size; i++)
            x[i] = getRandomNumberInRange(0, 100);

        radixSort(x);
        System.out.println(Arrays.toString(x));
        long runtime = System.currentTimeMillis() - start;
        System.out.println("Runtime: " + runtime);
    }    

    private static int getRandomNumberInRange(int min, int max) {
        if (min >= max)
            throw new IllegalArgumentException("max must be greater than min");

        return (int)(Math.random() * ((max - min) + 1)) + min;
    }

    public static void radixSort( int[] a) {
        int i, m = a[0], exp = 1, n = a.length;
        int[] b = new int[10];

        for (i = 1; i < n; i++)
            if (a[i] > m)
                m = a[i];

        while (m / exp > 0) {
            int[] bucket = new int[10];

            for (i = 0; i < n; i++)
                bucket[(a[i] / exp) % 10]++;
            for (i = 1; i < 10; i++)
                bucket[i] += bucket[i - 1];
            for (i = n - 1; i >= 0; i--)
                b[--bucket[(a[i] / exp) % 10]] = a[i];
            for (i = 0; i < n; i++)
                a[i] = b[i];
            exp *= 10;        
        }
    }    
}

Answer 1

之所以发生这种情况是因为您明确定义了int[] b数组的固定大小：

int[] b = new int[10];

如果输入大于10，那就是它溢出的原因。

从参数中将其更改为数组的可变长度。

int[] b = new int[a.length];

此外，我建议您仅在(0; n>区间内修复数字输入。

RadixSort算法运行时

1 个答案: