二进制搜索超出了时间限制

Question

我一直在研究使用二进制搜索来搜索元素的程序，但是每次运行该程序时，都会出现超过时间限制的错误。 虽然我搜索了此错误，但是我无法在此程序中找到错误，请帮助。这是代码

#include<stdio.h>
int BinarySearch(int low,int high,int *a,int item);
int main(){
  int a[50] , i , low , high , n , item , position;

  printf("Enter the number of Elements");
  scanf("%i",&n);
  for(i=0;i<n;i++){
    scanf("%i",&a[i]);
  }

  printf("Enter the Element to be searched");
  scanf("%i",&item);
  low=0;
  high=n-1;
  position=BinarySearch(low,high,a,item);

  if(position==-1)
    printf("Item not found");
  else
    printf("item was found on the %i position",position);

  return 0;
}
int BinarySearch(int low,int high,int *a,int item){
  int mid,loc,flag=0;
  while (item!= a[mid] && low <= high)
  {
    mid = (low + high) / 2;

    if(a[mid]==item){
      loc=mid;
      flag=flag+1;
    }
    else if (item < a[mid])
      high = mid - 1;
    else
      low = mid + 1;
  }
  if (flag>=1)
    return loc;
  else
    return -1;
}

Answer 1

您的方法似乎存在多个问题。

首先，删除您的while旁边的分号，就像现在一样，每次搜索的项目不在位置0时，while都是无限循环 ：

这个：

while (item!= a[mid] && low <= high);

应该变成这个：

while (item!= a[mid] && low <= high)

完成之后，请考虑以下事项：

二进制搜索只能应用于已排序的集合。

阅读所有数字后，您应该首先对它们进行排序，然后应用算法。

编辑 ：

另外，如注释中所述，您正在尝试在函数开始时将一些变量初始化为0（零），但这并不完全正确：

这个：

int mid,loc,flag=0;

只会将标志初始化为零， 中和loc会被初始化，可能会造成垃圾。

相反，请执行以下操作：

int mid= (low + high) /2, loc=-1, flag=0;

通过将loc设置为-1，您甚至可以通过消除标志来改善代码：

int BinarySearch(int low,int high,int *a,int item)
{
  int mid= (low + high) /2, loc=-1;
  while (item!= a[mid] && low <= high)
  {
    if(a[mid]==item){
      loc=mid;
    }
    else if (item < a[mid])
      high = mid - 1; 
    else
      low = mid + 1;
  }
  return loc;
}

如您所见， 标志已被完全消除，您无需再次设置中间位置 。

Answer 2

while (item!= a[mid] && low <= high);

最后删除半冒号:)

而且您的数组必须经过排序才能进行二进制搜索