堆排序算法 CLRS
Heapsort algorithm CLRS
我正在根据 CLRS 在 C 中实现堆排序算法。但是我无法获得排序的输出。你能看看我的代码有什么问题吗?函数 maxheap
和 buildmaxheap
有效。我无法弄清楚代码有什么问题。
代码应该对数组元素进行堆排序。我觉得 heapsort()
函数有错误,因为 maxheap
和 buildmaxheap
工作得很好。
我得到的最终输出是
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]);
*/
}
堆排序缺乏逻辑。任何排序算法都必须比较 2 个值,如果小于则做一件事,大于则做另一件事,如果相同则不要理会。目前,您多次自动交换索引为 1 的比较器。
我无法清楚地看到为什么它丢失数字导致随机 1 和 2,但它看起来很破烂,我不会再给你时间,直到你再试一次。
在c中数组索引从0开始,不要在这个10个节点的小困境中避免它,这没什么大不了的。但是,如果您不开始自动将零视为第一位,您将付出巨大的代价。
这一行:
int s=x[i];x[1]=x[i];x[i]=s;
看起来它正在尝试进行交换,但它混淆了。看索引,想顺序。
修复后,还有一个错误。我没有这本书,所以我不知道它告诉你做什么,但我相信你需要修复堆 属性 从根 开始 在您删除一个元素后,您的代码不会那样做。
我正在根据 CLRS 在 C 中实现堆排序算法。但是我无法获得排序的输出。你能看看我的代码有什么问题吗?函数 maxheap
和 buildmaxheap
有效。我无法弄清楚代码有什么问题。
代码应该对数组元素进行堆排序。我觉得 heapsort()
函数有错误,因为 maxheap
和 buildmaxheap
工作得很好。
我得到的最终输出是
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]);
*/
}
堆排序缺乏逻辑。任何排序算法都必须比较 2 个值,如果小于则做一件事,大于则做另一件事,如果相同则不要理会。目前,您多次自动交换索引为 1 的比较器。
我无法清楚地看到为什么它丢失数字导致随机 1 和 2,但它看起来很破烂,我不会再给你时间,直到你再试一次。
在c中数组索引从0开始,不要在这个10个节点的小困境中避免它,这没什么大不了的。但是,如果您不开始自动将零视为第一位,您将付出巨大的代价。
这一行:
int s=x[i];x[1]=x[i];x[i]=s;
看起来它正在尝试进行交换,但它混淆了。看索引,想顺序。
修复后,还有一个错误。我没有这本书,所以我不知道它告诉你做什么,但我相信你需要修复堆 属性 从根 开始 在您删除一个元素后,您的代码不会那样做。