使用java创建max堆

我尝试使用以下代码在java中创建一个最大堆：

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;

    }
}

而我正在

4 3 16 14 8 9 10 2 1 7

有人能帮忙解释一下为什么堆没有被正确构建吗

---

谢谢

我想你的问题在这里：

int left = i * 2;
int right = i * 2 + 1;

---

因为java数组是基于零的，所以您说0的左子元素是0*2=0！修正你的逻辑，看看这是否能解决你的问题。

我运行了你的代码，除了Jodaka所说的，我还发现了一个错误

for (kk = 0; kk <= Arr.length - 1; kk++) {
        heapM(Arr, kk);
}

因为当你做MaxHepify的时候，你要从头开始，然后向后走。 及

应该是，正如乔达卡所说

int left = 2*i+1;
int right = 2*i + 2;

---

我在这里运行了它，现在输出是正确的

因为您想要创建一个最大堆，所以应该从n/2到0开始，而不是从0到n。由于0到n，发生的情况是，在一个数组中，比如说2 0 1 12 13。执行max heap后的输出数组将为2 13 1 12 0，其中，如果从n/2向下调用为0，则输出数组将为13 12 1 2 0

这是家庭作业吗？ 如果没有，您可以使用JAVA的PriorityQueue

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））时间内构建堆，并在此处解释：

请以可读的方式发布缩进的代码片段。感谢Sujay提供的格式，Millimoose很抱歉，原始格式输出是相同的，即使我将循环更改为从1开始，我的观点是，您没有正确计算任何节点的子节点。我只是用0作为最明显的例子。啊，完全忽略了第一部分。抢手货很高兴我能帮上忙：）好卢卡纳，我只是想重新整理一下我的知识，给读者增加了参考，因为这是一个很好的答案，但解释得不够。

int left = 2*i+1;
int right = 2*i + 2;

for (kk = Arr.length -1; kk >= 0; kk--) {
    heapM(Arr, kk);
}

for (kk = (Arr.length -1)/2; kk >= 0; kk--) {
    heapM(Arr, kk);
}