HeapSort的问题

Question

我的任务是根据伪代码将代码写入堆。它应该输出数组（4 3 2 5 6 7 8 9 12 1），然后使用printHeap方法打印它。我知道printHeap的工作原理，因为我已经使用了一个名为buildHeap的方法（用于构建最大堆二进制树，但你们都已经知道:)）并且在那里它完美无缺，所以我的问题在于heapSort

它正确排序并按照它应该的方式打印（父 - 子1，父 - 2等），唯一的问题是，最大值和最后值，即12，突然变成了24，我不知道为什么。

代码如下：

void heapSort(int a[], int n){
int x = n+1;
int i;
int temp;
buildMaxHeap(a, n);
for (i = n; i >= 1; i--){
    temp = a[i];
    a[i] = a [0];
    a [0] = temp;
    x--;
    heapify(a, 0, x);
}

void printHeap(int a[], int n){

int i;
printf("graph g { \n");
for (i = 0; i < n/2; i++){
    printf("%d -- %d\n", a[i], a[left(i)]);
    if (right(i) < n){
        printf("%d -- %d\n", a[i], a[right(i)]);
    }
}
printf("}\n");

输出如下：

1 2 3 4 5 6 7 8 9 24
graph g {
1 -- 2
1 -- 3
2 -- 4
2 -- 5
3 -- 6
3 -- 7
4 -- 8
4 -- 9
5 -- 24
}

只是让你知道我到底做了什么，我会在这里附上while .c文件： https://onedrive.live.com/redir?resid=8BC629F201D2BC63!26268&authkey=!AFqVlm9AptiZ_xM&ithint=file%2cc

非常感谢你的帮助！

干杯 阿里克

Answer 1

嗯，你观察到一个未定义的行为。 （我个人在线IDE获得了0而不是12（24）。）

尝试：

void heapSort(int a[], int n)
{
    int x = n; /* changed from n+1 */
    int i;
    int temp;

    buildMaxHeap(a, n);
    for (i = n-1; i >= 0; i--){ /*<-- changed*/
        temp = a[i];
        a[i] = a[0];
        a[0] = temp;
        x--;
        heapify(a, 0, x);
    }
}

今天几乎所有通用语言中的数组都以索引0开头[有关详细信息，请参阅wiki。]错误地循环数组for (i = n; i >= 1; i--)，因为堆是最大值，不要处理第一个元素并超出最后一个元素。尽管在标准中定义了具有nth元素的算术，但并不意味着< n和一些指针工作。

另外，您可以对#define，left等函数使用宏（right s）来提高性能并简化阅读。

我希望节省一天和 AlgoDat 练习。