我正在研究一种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);
};
答案 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)
partition
和left < right
, theData[left] == theData[right]
会被卡住