Question

所以我试图实现快速排序算法，但事实上，因为我在一个调试器中跟随代码，所有元素都按排序顺序，但是，这个算法在元素之后的三个递归调用中取消了所有内容分类。请告诉我什么是错误的

#include <iostream>
#include <vector>

using namespace std;

void swap(int& a, int& b)
{
    int temp = a;
    a = b;
    b = temp;
}

void quicksort(vector<int> A, int start, int end)
{
    if (start < end)
    {
        int pivot = A[start];
        int i, j = start;
        for (i = start + 1;i < end;i++)
        {
            if (pivot>A[i])
            {
                swap(A[i], A[j + 1]);
                j++;
            }
        }
        swap(A[start], A[j]);
        quicksort(A, start, j - 1);
        quicksort(A, j + 1, end);
    }
}

int main()
{
    vector<int> A = { 2,8,7,1,3,5,6,4 };

    quicksort(A, 0, A.size());

    for (int i = 0; i < A.size(); i++)
        cout << A[i] << endl;

    return 0;
}

Answer 1

您将向量的副本传递给函数。

您需要一个参考参数，就像您使用swap所做的那样：

void quicksort(vector<int>& A, int start, int end)

（顺便说一句，swap在标准库中。）

Answer 2

快速浏览后，我发现了以下内容：

您需要将矢量作为参考传递。请记住， quicksort 中变量的生命周期只是暂时的（直到函数结束）。

void quicksort(vector<int>& A, int start, int end)

我也改变了

quicksort(A, start, j -1 );
quicksort(A, j + 1, end);

到

quicksort(A, start, j );
quicksort(A, j + 1, end);

因为你似乎跳过了A [j]。我认为它现在有效。您的主要问题是通过引用传递向量。

返回原始值的快速排序实现

2 个答案: