二进制最小堆上的 BubbleDown 操作不起作用

Question

我试图从二进制堆中提取最小值，但它不起作用。这是我的 BubbleDown 代码：

void heapBubbleDown(Heap * const heap, int idx) {
    int min;

    while(RIGHT(idx) < heap->count) {
        min = LEFT(idx);

        if(RIGHT(idx) < heap->count) {
            if(heap->items[LEFT(idx)] > heap->items[RIGHT(idx)]) {
                min = RIGHT(idx);
            }
        }

        heapSwapValue(&(heap->items[idx]), &(heap->items[min]));

        idx = min;
    }
}

看起来它只交换了一些数字，但不是全部，我不明白为什么。我已经多次尝试以不同的方式重新编码......

我做错了什么？

Answer 1

我不认为问题在于它交换了几个元素。当最小的子项 >= 当前项时，您必须停止。

我会将最后两行重写为：

    if (heap->items[idx] > heap->items[min]) {
       heapSwapValue(&(heap->items[idx]), &(heap->items[min]));
       idx = min;
    } 
    else
       break;
}

Answer 2

目前条件还不够。可能会出现没有右孩子但需要与左孩子进行交换的情况。此外，正如亨克建议的那样，您需要检查计算出的最小值实际上是否小于您当前的值。