0

我正在根据 CLRS 在 C 中实现堆排序算法。但是我无法获得排序的输出。你能看看我的代码有什么问题吗？功能maxheap和buildmaxheap工作。我无法弄清楚代码有什么问题。

该代码应该对数组元素进行堆排序。heapsort()我觉得函数中存在错误，因为maxheapandbuildmaxheap工作得很好。

我得到的最终输出是

1 1 1 1 2 1 2 2 1 1

但预期的输出应该是

1 2 3 4 7 8 9 10 14 16

编码：

#include<stdlib.h>
#include<stdio.h>

#define maxn 11
int n=10;

int parent(int i)
{
    return i/2;
}

int left(int i)
{
    return 2*i+0;
}

int right(int i)
{
    return 2*i+1+0;
}

void  max_heap(int x[],int i,int heapsize)
{
    int largest;
    int l=left(i);
    int r=right(i);

    if (l<=heapsize &&  x[l]>x[i]){
        largest=l;
    }
    else
    {
        largest=i;
    }
    if (r<=heapsize && x[r]>x[largest]){
        largest=r;
    }
    if (largest!=i)
    {
        int s=x[i];x[i]=x[largest];x[largest]=s;
        max_heap(x,largest,heapsize);
    }
}

void buildmaxheap(int x[],int heapsize)
{

    int i;
    for(i=5;i>=1;i--)
        max_heap(x,i,heapsize);

}

void heapsort(int x[])
{
    buildmaxheap(x,10);
    int i,t,heapsize=10;
    for(i=10;i>=2;i--)
    {
        int s=x[i];x[1]=x[i];x[i]=s;

        heapsize--;
        /*
         printf("%d",heapsize);
         */
        max_heap(x,i,heapsize);
    }
    for(i=1;i<=10;i++)
        printf("%d\t",x[i]);

}

int main()
{
    int x[maxn],i;
    x[1]=16;
    x[2]=4;
    x[3]=10;
    x[4]=14;
    x[5]=7;
    x[6]=9;
    x[7]=3;
    x[8]=2;
    x[9]=8;
    x[10]=1;
    heapsort(x);
    /*
     for(i=1;i<=10;i++)
     printf("%d\t",x[i]);
     */
}
4

2 回答 2

0

堆排序缺乏逻辑。任何排序算法都必须比较 2 个值，并为小于做一件事，另一件事为更大，如果相同则别管它。目前，您会多次自动交换索引为 1 的比较器。

我无法清楚地看到为什么它会丢失数字导致随机的 1 和 2，但它看起来很糟糕，我不会再给你时间，直到你再试一次。

在 c 中，数组索引从 0 开始，在这个 10 节点的小困境中不要避免它，这没什么。但是，如果您不开始自动将零视为第一，您将付出大量时间。

于 2015-06-17T23:06:25.273 回答
0

这一行：

int s=x[i];x[1]=x[i];x[i]=s;

看起来它正在尝试进行交换，但它混淆了。查看索引并考虑顺序。

在你修复它之后，还有另一个错误。我没有这本书，所以我不知道它到底告诉你做什么，但我相信你需要在删除一个元素后从根开始修复堆属性，而你的代码没有这样做.

于 2015-06-17T23:35:29.587 回答