1

出于培训目的，我在 javascript 中实现了一组排序算法。我的实现已经成功地对整数和单字符字符串数组进行排序，除了堆排序。

我的实现对数组进行了正确的排序，除了在返回排序后的数组时将最小的数字放在末尾。

我不是 CS 学生/研究生，也没有很多编程背景。我正在通过读取/尝试/失败方法进行学习，但我无法使其正常工作。

我会将代码及其结果放在本段下方。最后，我想补充一点，我将这篇 Wikipedia 文章中的伪代码作为我实现的参考。

function heapSort(intl)
{
    var l   = intl.length,
        end = l - 1,
        swap;
    intl = _heapify(intl,l);

    while(end>0)
    {
        swap      = intl[0];
        intl[0]   = intl[end];
        intl[end] = swap;
        --end;
        intl = _siftDown(intl, 0, end);
    }

    return intl;
}


function _heapify(intl, l)
{
    var start = (l - 2) / 2;
    while(start >= 0)
    {
        intl = _siftDown(intl, start, l-1);
        --start;
    }
    return intl;
}

function _siftDown(intl, start, end)
{
    var root = start,
        child, swap, swapr;
    while(root*2+1 <= end)
    {
        child = root * 2 + 1;
        swap = root;
        if(intl[swap] < intl[child])
            swap = child;
        if(child+1 <= end && intl[swap]<intl[child+1])
            swap = child + 1;
        if(swap!=root)
        {
            swap       = intl[root];
            intl[root] = intl[swap];
            intl[swap] = swap;
            root       = swap;
        }else
        {
            return intl;
        }
    }
    return intl;
}


x =
[24,5,896,624,437,5,6,4,37,45,654];
y =
["a","b","r","s","t","e","q","u","q"];

console.log(heapSort(x),"\n",heapSort(y));

通过nodejs运行代码：

$ node sort.js
[ 5, 5, 6, 24, 37, 45, 437, 624, 654, 896, 4 ]
[ 'b', 'e', 'q', 'q', 'r', 's', 't', 'u', 'a' ]

我试图通过更改代码来找到错误所在，但我找不到它。谁能告诉问题出在哪里？提前致谢。

4

1 回答 1

1

我看到了 2 个错误：

  1. 在_heapify函数中，如果不是奇数，则start变量不是整数：l

    var start = Math.floor( ( l - 2 ) / 2 );

  2. 在_siftDown函数中，您使用swap而不是swapr有效交换数组的 2 个元素：

    swapr      = intl[root];  // <-- Here swapr instead of swap
intl[root] = intl[swap];
intl[swap] = swapr;       // <-- Here swapr instead of the 1st occurrence of swap
root       = swapr;       // <-- Here swapr instead of swap
于 2012-11-28T09:34:44.417 回答