Question

我正在练习编写排序算法作为一些面试准备的一部分，我想知道是否有人可以帮我发现为什么这种快速排序不是很快？它似乎具有正确的运行时复杂性，但它比我的合并排序慢了大约2的常数因子。我也很感激任何可以改进我的代码的注释，但不一定能回答这个问题。

非常感谢你的帮助！如果我犯了礼仪错误，请不要犹豫，告诉我。这是我的第一个问题。

private class QuickSort implements Sort {

        @Override
        public int[] sortItems(int[] ts) {
            List<Integer> toSort = new ArrayList<Integer>();
            for (int i : ts) {
                toSort.add(i);
            }
            toSort = partition(toSort);
            int[] ret = new int[ts.length];
            for (int i = 0; i < toSort.size(); i++) {
                ret[i] = toSort.get(i);
            }
            return ret;
        }

        private List<Integer> partition(List<Integer> toSort) {
            if (toSort.size() <= 1)
                return toSort;
            int pivotIndex = myRandom.nextInt(toSort.size());
            Integer pivot = toSort.get(pivotIndex);
            toSort.remove(pivotIndex);
            List<Integer> left = new ArrayList<Integer>();
            List<Integer> right = new ArrayList<Integer>();
            for (int i : toSort) {
                if (i > pivot)
                    right.add(i);
                else
                    left.add(i);
            }
            left = partition(left);
            right = partition(right);
            left.add(pivot);
            left.addAll(right);
            return left;
        }

}

非常感谢所有帮助过的人！

这是我后代的改进课程：

private class QuickSort implements Sort {

        @Override
        public int[] sortItems(int[] ts) {
            int[] ret = ts.clone();
            partition(ret,0,ret.length);
            return ret;
        }

        private void partition(int[] toSort,int start,int end) {
            if(end-start<1) return;
            int pivotIndex = start+myRandom.nextInt(end-start);
            int pivot = toSort[pivotIndex];
            int curSorted = start;
            swap(toSort,pivotIndex,start);
            for(int j = start+1; j < end; j++) {
                if(toSort[j]<pivot) {
                    if(j!=curSorted+1) 
                        swap(toSort,curSorted,curSorted+1);
                    swap(toSort,j,curSorted++);
                }
            }
            // Now pivot is at curSorted
            partition(toSort,start,curSorted);
            partition(toSort,curSorted+1,end);
        }
    }

Answer 1

Quicksort的一大优势是它可以作为就地算法实现。不要创建新列表，而是在原地对元素进行排序。

Answer 2

除了不重复使用列表之外，还可以在每个步骤中在Integer和int之间进行转换：

        for (int i : toSort) {  // converts from Integer to int
            if (i > pivot)
                right.add(i);  // converts from int to Integer
            else
                left.add(i);   // converts from int to Integer
        }

请注意，从int到Integer的转换通常需要创建一个新对象。

最后，random.nextInt（）可能是一个非常重要的操作。如果toSort超过一定的大小并且使用更简单的枢轴选择策略，那么最好只选择一个随机的枢轴（测量它！）。

为什么我的快速排序这么慢？

2 个答案: