我的QuickSort有问题

时间:2013-02-21 18:00:29

标签: c++ quicksort

我已经按照快速排序代码:

typedef struct tagDataPair {
    int c_value;
    float error;
} DataPair;

void SortByErrorQS(std::vector<DataPair>& points, int left, int right)
{
    std::vector<int> stack;
    stack.push_back(left);
    stack.push_back(right);
    while(stack.size() > 0)
    {
        right = stack.back();
        stack.pop_back();
        left = stack.back();
        stack.pop_back();

        float pivot = (points.at(left).error + points.at(right).error + points.at((left + right)>>1).error)/3;
        int i = left, j = right;
        DataPair temp;
        while(i < j)
        {
            while(points.at(i).error <= pivot && (i <= right))
                ++i;
            while(points.at(j).error > pivot && (j > left))
                --j;
            if(i <= j)
            {
                temp = points[i];
                points[i] = points[j];
                points[j] = temp;
                i++; j--;
            }
        }

        if(left < j)
        {
            stack.push_back(left);
            stack.push_back(j);
        }
        if(i < right)
        {
            stack.push_back(i);
            stack.push_back(right);
        }
    }
}

出于某种原因,这种情况陷入了无限循环,我无法弄清楚出了什么问题,或者说是为什么。有人可以帮我指点这里发生了什么吗?

1 个答案:

答案 0 :(得分:3)

要将std::sortDataPair结构一起使用,您可以提供自定义比较器。在C ++ 11中,这可以使用lambda函数完成:

std::sort(points.begin(), points.end(), [](const DataPair& a, const DataPair& b) {
  return a.error < b.error;
});

这将按DataPair的递增顺序对error进行排序。

C ++ 03方法是提供比较函数:

bool compare(const DataPair& a, const DataPair& b)
{
  return a.error < b.error;
}

std:sort(points.begin(), points.end(), compare);

std::sort的复杂性保证为O(NlogN)。常见的实现使用quicksort或introsort。