0

我正在尝试在 C++ 中使用 quickselect 来执行此操作,但它不断返回我第 k 个最小的元素而不是第 k 个最大的元素。我的逻辑哪里错了?

int partition(int* input, int p, int r)
{
    int pivot = input[r];

    while ( p < r )
    {
        while ( input[p] < pivot )
            p++;

        while ( input[r] > pivot )
            r--;

        if ( input[p] == input[r] )
            p++;
        else if ( p < r ) {
            int tmp = input[p];
            input[p] = input[r];
            input[r] = tmp;
        }
    }

    return r;
}

int quick_select(int* input, int p, int r, int k)
{
    if ( p == r ) return input[p];
    int j = partition(input, p, r);
    int length = j - p + 1;
    if ( length == k ) return input[j];
    else if ( k < length ) return quick_select(input, p, j - 1, k);
    else  return quick_select(input, j + 1, r, k - length);
}

我应该改变什么来使这个第 k 个最大而不是第 k 个最小的元素?

4

1 回答 1

1

正如@Dietmar Kühl 提到的那样,您的代码中的 and 是相反的,<通过更改它们,它可以正常工作。>partition()

此外,我的建议是使用partition()如下的快速排序法线,其中两个索引沿同一方向移动,其中一个永远不会超过另一个。让任何人感到困惑并不容易。

int partition(int *input, int p, int r) {
    int pivot,i,j,tmp;
    pivot = input[r];
    i = p-1;
    for (j=p;j<=r-1;j++) {
        if (input[j]>= pivot) {
            i++;
        tmp = input[i];
        input[i] = input[j];
        input[j] = tmp;
        }
    }
    tmp = input[i+1];   
    input[i+1] = input[r];
    input[r] = tmp;
    return i+1;
}
于 2013-08-26T07:17:59.817 回答