使用递归的二进制搜索

Question

我不知道为什么它不会返回键，似乎跳过了一步，我觉得逻辑是直截了当的，如果midptr小于键，则在右边搜索，在左边搜索。 但它没有返回键，因此返回-1。 救命？ 这是代码和功能

#include<iostream>
using namespace std;


int binsrch(int *raw, unsigned int size, int key);




int main()
{
  int raw[] = {1,3,5,7,11,23, 48};
  cout << binsrch(raw, 7, 11) << endl;


  system("pause");
  return 0;
}



int binsrch(int *raw, unsigned int size, int key)
{
    int *begptr, *endptr ,*midptr;
//see if we already have the key

if(*raw == key)
  return key;

begptr = raw;
endptr = raw + (size - 1);
midptr = raw + (size / 2);

cout << "#" <<*midptr << " size:" << size<<  endl;
if(*midptr == key)
{
  return key;
}
else  if( *midptr < key) //Search Right
{
  cout << "#" <<*(midptr+1) << " size:" << size<<  endl;
  binsrch(midptr + 1, size / 2, key);
}
else if(*midptr > key) //Search Left
{
    cout << " #" <<*midptr << " size:" << size<<  endl;
    binsrch(begptr, size / 2, key);
}

return -1;
}

Answer 1

您忘记了return声明。 您应该返回递归调用的结果：

binsrch(midptr + 1, size / 2, key);

应该

return binsrch(midptr + 1, size / 2, key);

否则，除非您在第一次递归之前找到键，否则您的初始调用将执行其余部分，并始终返回-1 。

通过添加return语句，您将中断递归调用的控制流（即，您不返回“未找到”值），并且将最后一个返回值一直传播到调用堆栈中，直到第一个调用，最后返回您想要的值。

Answer 2

一切正常，但是您没有返回正确的值。 在else-if语句中，您将函数称为递归函数，但是返回的值不会传递给初始调用！ 尝试：

return binsrch(midptr + 1, size / 2, key);

和

return binsrch(begptr, size / 2, key);

那应该工作。

编辑：嗯，我想我来的很慢；）

Answer 3

您还应该添加：

if(endptr == midptr) return -1;

在计算endptr之后避免无限循环，以防搜索元素不在数组（例如21）中。