0

Eclipse 不断告诉我我的分区函数有问题。它在java中,是一个关于排序数组的类的一部分。

分区的工作原理如下:数组中有两个索引 i 和 j,在分区算法的最开始,i 指向数组中的第一个元素,j 指向最后一个元素。然后算法向前移动 i,直到找到值大于或等于枢轴的元素。索引 j 向后移动,直到找到值小于或等于枢轴的元素。如果 i ≤ j 则它们被交换并且 i 步到下一个位置 (i + 1),j 步到前一个位置 (j - 1)。当 i 大于 j 时,算法停止。

你能看到问题是什么,因为我很难找到它。非常感谢您的帮助。

public static int partition(int arr[], int left, int right)
    {
        int x = arr[right];

        int i = left-1;
        int temp=0;

        for (int j=left; j<right; j++)
        {
            if(arr[j]<=x)
            {
                i++;
                temp = arr[i];
                arr[i] = arr[j];
                arr[j] = temp;

            }
        }

        temp = arr[i+1];
        arr[i+1] = arr[right];
        arr[right] = temp;
        return (i+1);

    }

编辑

Eclipse是说有两个问题,一个是分区函数中的这行代码:

int x = arr[right];

并使用我的测试类的这一行:

sort.partition(array, 100, array.length);

这是测试类,其中包括我没有提到的功能。

import java.util.Random;


public class test {

    /**
     * @param args
     */
    public static void main(String[] args) {

        int size = 1000;
        int max = 5000; 
        int[] array = new int[size];
        int loop = 0; 

        Random generator = new Random();
        //Write a loop that generates 1000 integers and 
        //store them in the array using generator.nextInt(max)

        generator.nextInt(max); //generating one

        //I need to generate 1000
        //So I need some kind of loop that will generate 1000 numbers. 

        for (int i =0; i<1000; i++)
        {
            generator.nextInt(max);
        }



        /**
         * After I do this, I'll have the array, array. 
         * Then comes what's under this. 
         * THat method is for measuring the time.
         * System.currentTimeMillis();, 
         * with this, I can collect a time for the start of the method
         * and one for the end. 
         * Time at the end, minus the time at the start
         * gets us the running time. 
         */



        long result;

        long startTime = System.currentTimeMillis();
        sort.quickSort(array,  100,  array.length-1);
        long endTime = System.currentTimeMillis();
        result = endTime-startTime; 

        System.out.println("The quick sort runtime is " + result + " miliseconds");

        long result2;

        long startTime2 = System.currentTimeMillis(); 
        sort.partition(array, 100, array.length);
        long endTime2 = System.currentTimeMillis();
        result2 =  endTime2 - startTime2;
        System.out.println("The partition runtime is "+result2 + " miliseconds");

        long result3;

        long startTime3 = System.currentTimeMillis();
        sort.bubbleSort(array, 100);
        long endTime3 = System.currentTimeMillis();
        result3 = endTime3-startTime3;
        System.out.println("The bubble sort runtime is "+result3 + " miliseconds");

        long result4;

        long startTime4 = System.currentTimeMillis();
        sort.selectionSort(array, 100); //change the second number to change
        //the size of an array. 
        long endTime4 = System.currentTimeMillis();
        result4 = endTime4-startTime4;
        System.out.println("The selection sort runtime is "+result4 + " miliseconds");



    }

}
4

1 回答 1

0

首先,您实际上并没有将随机数存储在数组中,因此它将全为零。

至于您看到的实际错误,您已经得到了一个经典错误。您有两个选择:right指向数组的末尾或数组末尾之后的一个。任何一个都是有效的,但你正在混合两者。

具体来说,您传递了 1000,即数组末尾的一个right值,但随后您立即用它索引数组,自然会引发异常。

于 2013-04-25T02:31:05.787 回答