我正在尝试使用以下代码在java中创建Max-Heap:
public class Heapify {
// 16 14 10 8 7 9 3 2 4 1
public static int[] Arr = {4, 1, 3, 2, 16, 9, 10, 14, 8, 7};
public static int counter = 0;
public static void main(String[] args) {
int kk;
for (kk = 0; kk <= Arr.length - 1; kk++) {
heapM(Arr, kk);
}
for (int krk = 0; krk < Arr.length; krk++) {
System.out.println(Arr[krk]);
}
}
public static void heapM(int[] Arr, int i) {
int largest;
int left = i * 2;
int right = i * 2 + 1;
if (((left < Arr.length) && (Arr[left] > Arr[i]))) {
largest = left;
} else {
largest = i;
}
if (((right < Arr.length) && (Arr[right] > Arr[largest]))) {
largest = right;
}
if (largest != i) {
swap(i, largest);
heapM(Arr, largest);
}
}
private static void swap(int i, int largest) {
int t = Arr[i];
Arr[i] = Arr[largest];
Arr[largest] = t;
}
}
所需的输出应为:
16 14 10 8 7 9 3 2 4 1
我正在接受
4 3 16 14 8 9 10 2 1 7
有人可以帮助解决为什么没有正确构建堆?
由于
答案 0 :(得分:5)
我运行了你的代码,除了Jodaka所说的,我还发现了一个错误
for (kk = 0; kk <= Arr.length - 1; kk++) {
heapM(Arr, kk);
}
应该是
for (kk = Arr.length -1; kk >= 0; kk--) {
heapM(Arr, kk);
}
因为在执行MaxHepify时,从最后开始,然后向后退。 和
int left = i * 2;
int right = i * 2 + 1;
应该像Jodaka所说的那样
int left = 2*i+1;
int right = 2*i + 2;
我在这里运行它,输出现在正确
答案 1 :(得分:4)
我认为你的问题在这里:
int left = i * 2;
int right = i * 2 + 1;
由于java数组从零开始,你说左边的0是0 * 2 = 0!修复你的逻辑,看看是否能解决你的问题。
答案 2 :(得分:2)
for (kk = Arr.length -1; kk >= 0; kk--) {
heapM(Arr, kk);
}
应该是
for (kk = (Arr.length -1)/2; kk >= 0; kk--) {
heapM(Arr, kk);
}
效率更高。
编辑这使您可以在O(n)时间而不是O(n * lg(n))时间内构建堆,并在此处进行说明:http://en.wikipedia.org/wiki/Binary_heap#Building%5Fa%5Fheap
答案 3 :(得分:1)
如果你想制作一个最大堆,你应该从n / 2开始下降到0而不是0到n。由于0到n所发生的事情就像在一个数组中那样说2 0 1 12 13.执行最大堆后的输出数组将是2 13 1 12 0,就好像你从n / 2调用到0那样它将是13 12 1 2 0.
答案 4 :(得分:1)
这是作业吗? 如果没有,你可以使用JAVA的PriorityQueue