Question

我试图实现一个基本的快速排序算法，我想我已经正确实现了它。但是这些函数根本不会影响数组。可能是什么原因？我无法弄清楚出了什么问题所以我决定在这里咨询我的同事：

#include <iostream>
#include <vector>

using namespace std;

int partition(vector<int> A,int low,int high)
{
    int pivot=A[high-1];
    int boundryForLowerArray=low-1;
    for(int i=low;i<high-2;i++)
    {
        if(A[i]<=pivot)
        {
            boundryForLowerArray++;
            swap(A[i],A[boundryForLowerArray]);
        }
    }
    swap(pivot,A[boundryForLowerArray+1]);
    return boundryForLowerArray+1;
}
void quickSort(vector<int>A,int low,int high)
{
    if(low<high)
    {
        int q=partition(A,low,high);
        quickSort(A, low, q-1);
        quickSort(A, q+1, high);
    }
}


int main(int argc, const char * argv[])
{

    vector<int>A,sorted;
    A.push_back(2);
    A.push_back(8);
    A.push_back(7);
    A.push_back(1);
    A.push_back(3);
    A.push_back(5);
    A.push_back(6);
    A.push_back(4);
    quickSort(A, 0, A.size());
    for(int i=0;i<A.size();i++)
        cout<<A[i]<<" ";
    return 0;
}

Answer 1

您通过值而不是引用传递A，因此quickSort正在复制A并对其进行排序。而是尝试通过引用传递向量：

int partition(vector<int>& A,int low,int high)

...和

void quickSort(vector<int>& A,int low,int high)

Answer 2

因为您按值传递参数而不是参考。实际上你应该有一个带有迭代器的函数，用于数组的开始和结束（vec.begin（），vec.end（））作为参数。此外，您的算法应该接受任何类型的迭代器。所以你应该使用模板！

template<class Iterator>
void quick_sort(Iterator begin, Iterator end) { 
   for(auto iter = begin;iter != end;iter++) 
      *iter; // access to the value of the iterator

C ++：不受Void函数影响的变量

2 个答案: