我试图在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个最小元素?
正如@Dietmar Kühl所提到的,代码中的<
和>
在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;
}