快速排序不起作用(分段错误)

时间:2015-04-19 15:53:03

标签: c++ quicksort

任何人都可以告诉我为什么这段代码不起作用。它一直给我一个分段错误,我无法确切地找出错误的确切位置。真的很感激。我已经好几个小时了,但仍然无法弄明白。

vector<long> QuickSortArray(vector<long> nums)
{
        long arr[nums.size()];
        for(int i = 0; i < nums.size(); i++)
        {
                arr[i] = nums[i];
        }
        if(nums.size() == 1)
        {return nums;}
        else
        {
                long pivot = arr[0];
                int m = 0;
                int n = 0;
                for(int j = 1; j < nums.size(); j++)
                {
                        if(arr[j] < pivot)
                        {
                                m++;
                        }
                        else
                        {
                                n++;
                        }
                }
                long left[m+1];
                long right[n+1];
                int a = 0;
                int b = 0;
                for(int j = 1; j < nums.size(); j++)
                {
                        if(arr[j] < pivot)
                        {
                                left[a] = arr[j];
                                cout<<left[a]<<endl;
                                a++;
                        }
                        else
                        {
                                right[b] = arr[j];
                                cout<<right[b]<<endl;
                                b++;
                        }
                }
                vector<long> l;
                vector<long> r;
                for(int x = 0; x < m+1; x++)
                {l.push_back(left[x]);}
                for(int y = 0; y < n+1; y++)
                {r.push_back(right[y]);}
                l = QuickSortArray(l);
                r = QuickSortArray(r);
                vector<long> ret;
                for(int h = 0; h < l.size(); h++)
                {ret.push_back(l[h]);}
                ret.push_back(pivot);
                for(int k = 0; k < r.size(); k++)
                {ret.push_back(r[k]);}
                return ret;
        }
}

1 个答案:

答案 0 :(得分:1)

我认为你应该在开头检查案例nums.size()== 0而不是nums.size()== 1。

if(nums.size() == 0)
    {return nums;}

另外,我认为你应该写

long left[m];
long right[n];

你为什么不专门使用载体?使用类似C的数组似乎更像是一种在这里寻找麻烦的方法。