Question

因此，在遵循Cormen的quicksort和hoares分区算法之后，这就是我能够生成的代码。阵列部分排序与未初始化的元素/垃圾元素，我不能为我的生活找出原因...我认为我完全按照书写的那样遵循算法。

以下是本书中的伪代码：

HOARE-PARTITION(A, p, r)
1 x = A[p]
2 i = p - 1
3 j = r + 1
4 while TRUE
5     repeat
6        j = j - 1
7     until A[j] <= x
8     repeat
9        i = i + 1
10    until A[i] >= x
11    if i < j
12        exchange A[i] with A[j]
13    else return j

以下是我将其翻译成的C ++代码：

void quickSort(int arr[], int p, int r){
    if(p < r){
        int q = hoare_partition(arr, p, r);
        quickSort(arr, p, q-1);
        quickSort(arr, q+1, r);
    }
}

int hoare_partition(int arr[], int p, int r){
    int x = arr[p];
    int i = p - 1;
    int j = r + 1;
    while(true){
        do{
            j--;
        }while(arr[j] > x);
        do{
            i++;
        }while(arr[i] < x);
        if(i < j){
            int temp = arr[i];
            arr[i] = arr[j];
            arr[j] = temp;
        }
        else 
            return j;
    }
}

我使用以下内容进行测试

cout << endl << endl << "Testing quicksort" << endl;
int tarr[10] = {2, 30, 1, 99, 46, 33, 48, 67, 23, 76};
quickSort(tarr, 0, 10);
cout << "arr after quicksort: ";
for(int i = 0; i < 10; i++){
    cout <<  tarr[i] << " ";
}
cout << endl;

输出

arr after quicksort: -2146162183 1 2 23 30 33 46 48 67 76

提前感谢任何帮助...谢谢

修改

将测试用例调用更改为quickSort（arr，0,9），修复了这种情况。

但是，使用反向排序数组作为输入，这是输出：

arr2 is:
30 29 28 27 26 25 24 23 22 21 20 19 18 17 16 15 14 13 12 11 10 9 8 7 6 5 4 3 2 1
arr2 after quicksort :
1 3 5 7 9 11 13 15 17 19 20 21 18 22 16 23 14 24 12 25 10 26 8 27 6 28 4 29 2 30

使用此测试代码：

int arr2[30];
fillArrayReverse(arr2, 30);
cout << "arr2 is :" << endl;
for(int i = 0; i < 30; i++){
    cout << arr2[i] << " ";
}
cout << endl;
quickSort(arr2, 0, 29);
cout << "arr2 after quicksort: " << endl;
for(int i = 0; i < 30; i++){
    cout << arr2[i] << " ";
}
cout << endl;

Answer 1

书中的伪代码是正确的。问题不在于返回j（这是正确的），而是在quickSort函数本身。它实际上应该如下：

void quickSort( int arr[], int p, int r )
{
    if( p < r )
    {
        int q = hoare_partition( arr, p, r );
        quickSort( arr, p, q );
        quickSort( arr, q + 1, r );
    }
}

请注意，我们会在第一次调用q - 1时删除quickSort，并将其替换为q。事先我们基本上是跳过枢轴点，只对它周围的物品进行排序。我在wikipedia快速排序后看了一下Hoard Partition伪代码后发现了这种差异。

Answer 2

这是您的测试代码的问题。第三个参数应该是最后一个元素的索引，而不是元素的数量：

quickSort(tarr, 0, 9);

你正在阅读数组的末尾。

有关Hoare分区，Quicksort和Cormen等的更多信息，请参阅this question。

您遇到的其他问题的简单修复是将hoare_partition（）更改为return i而不是j。请参阅链接了解更多详情（据报道，这是书中的错误）。

Hoare的分区无法正常工作（快速排序）

修改

2 个答案: