C ++ Quicksort算法崩溃

时间:2012-05-22 20:04:04

标签: c++ algorithm sorting quicksort

我正在研究一种c ++快速排序算法,该算法计算排序期间的重要操作数。它将正确排序100个值,但之后它会陷入无限循环。任何帮助将不胜感激!

// QuickSort.cpp

#include "QuickSort.h"
#include <algorithm>

using namespace std;

QuickSort::QuickSort(int max) : SortData(max)
{
    /*
    maxSize = max;
    theData = new long[max];
    */
    SortData::randomize(12);

}

QuickSort::~QuickSort(void)
{

}

_int64 QuickSort:: sort()
{
    numops = 0;
    quicksort(theData, 0, (size()-1));
    return(numops);
}

//Include counting number of operations
void QuickSort::quicksort(long theData[], int left, int right)
{
    //Default variables

    if (size() <= 1 || left >= size() || right >= size()) return; //Bounds checking

    if (left < right)
    {
        long pivot = partition(theData, left, right);

        quicksort(theData, left, pivot-1);
        quicksort(theData, pivot+1, right);
    }
}

int QuickSort::partition(long theData[], int left, int right)
{
    long pivot = theData[left];
    while (true)
    {
        while (theData[left] < pivot) left++;
        numops ++;

        while (theData[right] > pivot) right--;
        numops ++; 

        if (left < right)
        {
            swap(theData[left], theData[right]);
            numops+=3;
        }

        else
        {
            return right;
        }
    }
}

// QuickSort.h

#pragma once
#include "SortData.h"

class QuickSort : public SortData
{

public:
    QuickSort(int max = 100);
    ~QuickSort(void);
    _int64 sort();
private:
    _int64 numops;
    void QuickSort::quicksort(long theData[], int left, int right);
    int partition(long theData[], int left, int right);
}; 

2 个答案:

答案 0 :(得分:4)

在你的分区功能中

 if (left < right)

总是如此。因此,您将获得无限的(真)循环。

您在SortData.h中的size()函数可能存在问题,我们还看不到它。

由于数据是随机的,因此您可能会不时地在某些输入集上看到问题。

轻微修改必须有所帮助:

if (left <= right) {
         if (left < right) {
             swap(theData[left], theData[right]);
             numops += 3;
         }
         left++;
         right--;
}
else {
    return right;
}

对不起,仔细检查:)

答案 1 :(得分:1)

如果partitionleft < right

theData[left] == theData[right]会被卡住