我试图在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个最小元素?
答案 0 :(得分:1)
代码中的<
和>
在partition()
中是反向的,正如@DietmarKühl所提到的那样,通过更改它们,它可以正常工作。
此外,我的建议是使用正常的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;
}