我正在使用指针和动态内存分配来实现二进制搜索。我首先对元素列表进行排序,然后执行搜索。排序部分工作正常,但我的搜索工作不正常。我知道算法是正确的,但我不知道如何去解决语法问题。有人可以帮助我吗? PS这是关于堆栈溢出的第一个问题,BTW。 :)
#include <stdio.h>
#include <stdlib.h>
int binary_search(int x,int start,int end, int *array)
{
int q;
q=(start+end)/2;
if(x==*(array+q))
{
return q;
}
else if (x>*(array+q))
{
binary_search(x,q+1,end,array);
}
else if (x<*(array+q))
{
binary_search(x,start,q-1,array);
}
else if(start>=end)
return -1;
}
int main()
{
int n,*a,tmp,i,j,search,pos;
printf("Enter n");
scanf("%d",&n);
a=malloc(sizeof(int)*n);
for(i=0;i<n;i++)
{
scanf("%d",&tmp);
*(a+i)=tmp;
}
for(i=0;i<n-1;i++)
{
for(j=0;j<n-1;j++)
{
if(*(a+j)>*(a+j+1))
{
tmp=*(a+j);
*(a+j)=*(a+j+1);
*(a+j+1)=tmp;
}
}
}
printf("Sorted elements are\n");
for(i=0;i<n;i++)
{
printf(" %d",*(a+i));
}
printf("Enter element to search\n");
scanf("%d",&search);
pos=binary_search(search,0,n-1,&a);
printf("\nThe element is located at position %d",pos);
return 0;
}
答案 0 :(得分:2)
除上述答案外 -
int binary_search(int x,int start,int end, int *array)
^integer pointer
但在main
中,您传递int **
类型 -
pos=binary_search(search,0,n-1,&a);
^a is already an integer pointer
编译器会发出警告。
只需通过a
即可。此外,您忘记了free
a
,因为您已使用malloc
为其分配内存。
答案 1 :(得分:1)
首先,*(array+q)
与array[q]
相同。
其次,在发布问题之前,您应该做一些研究。 http://www.cquestions.com/2008/01/c-program-for-binary-search-through.html
如果要搜索的号码不存在,则会进入无限循环。 这应该解决它。
int binary_search(int x,int start,int end, int *a)
{
int q;
if(start>end)
return -1;
q=(start+end)/2;
if(x==*(a+q))
{
return q;
}
else if (x>*(a+q))
{
return binary_search(x,q+1,end,a);
}
else if (x<*(a+q))
{
return binary_search(x,start,q-1,a);
}
}
答案 2 :(得分:-1)
您忘记在递归调用中添加return
。因此,如果没有适当的返回值,函数就会失效。
此外,需要在最顶层执行检查if(start>=end)
。否则,当x
不在array
时,您会看到无限递归。
更新:
int binary_search(int x,int start,int end, int *array)
{
int q;
if(start>=end)
return -1;
q=(start+end)/2;
if(x==*(array+q))
{
return q;
}
else if (x>*(array+q))
{
// Add return
return binary_search(x,q+1,end,array);
}
// There is no need to do any further checks.
return binary_search(x,start,q-1,array);
}