2

我试图实现一个基本的快速排序算法,我认为我已经正确地实现了它。但是,这些函数根本不影响数组。可能是什么原因?我不知道出了什么问题,所以我决定在这里咨询我的程序员同事:

#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;
}
4

2 回答 2

5

您通过值而不是通过引用传递 A,因此quickSort制作 A 的副本并对其进行排序。而是尝试通过引用传递向量:

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

... 和

void quickSort(vector<int>& A,int low,int high)
于 2012-04-25T17:58:14.083 回答
0

因为您通过值而不是通过引用传递参数。实际上,您应该有一个带有迭代器的函数,用于数组的开始和结束(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
于 2012-04-25T18:04:27.783 回答