-1 
public static void sort(int[] a){
        if (a.length>1){
            int pivot=a[a.length-1];
            int left=0;
            int right=a.length-1;
            while(left<=right){
                while(a[left]<pivot)
                    left++;
                while(a[right]>pivot)
                    right--;
                if(left<=right){
                    int tmp=a[right];
                    a[right]=a[left];
                    a[left]=tmp;
                    left++;
                    right--;
                }
            }
            int[] tmp1=new int[right];
            for(int i=0;i<tmp1.length;i++)
                tmp1[i]=a[i];
            int[] tmp2=new int[a.length-right-1];
            for(int i=left;i<a.length;i++)
                tmp2[i-left]=a[i];
            sort(tmp1);
            sort(tmp2);
        }
    }

我试图用一个函数编写一个快速排序算法，但它不起作用。任何帮助都是值得的。谢谢

编辑：我解决了谢谢大家的意见。

4

2 回答 2

2

我认为问题在于最后你没有使用tmp1andtmp2来符合新数组a......这是一种不创建其他数组的方法：

    public static void sort(int[] a, int left, int right){
        if (left < right){
            int pivot = a[right];
            int pos = left - 1;
            for (int i = left; i < right; i++)
                if (a[i] <= pivot)
                    Swap(a, ++pos, i);
            Swap(a, pos + 1, right);
            sort(a, left, pos);
            sort(a, pos + 1, right);
        }

    }

    public  static void Swap(int[] a, int i, int j){
        int temp = a[j];
        a[j] = a[i];
        a[i] = temp;
    }

排序的第一个调用必须是sort(a, 0, a.length - 1)

我希望这可以帮助你

于 2013-03-19T03:20:06.597 回答
0

与大多数 int 排序方法相比，您的排序方法似乎非常极端。这是一个快速简便的方法。

public static void sort(int[] intArray){
    int n = intArray.length;
    int temp = 0;
    for(int i=0; i < n; i++){
        for(int j=1; j < (n-i); j++){
            if(intArray[j-1] > intArray[j]){
                temp = intArray[j-1];
                intArray[j-1] = intArray[j];
                intArray[j] = temp;
            }
        }
    }
}

这只是一个冒泡排序。我没有看到递归的意义。还有很多其他类型的排序，但对于较短的数组长度，这是最简单的（IMO）。查找其他一些，它们的工作有点酷（排序算法）。

回答你的问题....就像@RobinCurbelo 说的那样，你没有正确使用temp1和使用temp2。你的想法是存在的，但我认为你对你需要做的事情考虑得太多了。

于 2013-03-19T03:28:52.147 回答