我需要帮助来实现二进制搜索算法,谁能告诉我我的代码出了什么问题:
public int bsearch(Item idToSearch) {
int lowerBoundary = 0;
int upperBoundary = myStore.size() - 1;
int mid = -1;
while(upperBoundary >= lowerBoundary) {
mid = (lowerBoundary + upperBoundary) / 2;
//if element at middle is less than item to be searched, than set new lower boundary to mid
if(myStore.get(mid).compareTo(idToSearch) < 0) {
lowerBoundary = mid - 1;
} else {
upperBoundary = mid + 1;
}
} //end while loop
if(myStore.get(mid).equals(idToSearch)) {
return mid;
} else {
return -1; // item not found
}
} // end method发布于 2014-03-31 09:47:38
我认为你在更新lowerBoundary和upperBoundary时犯了一个错误。
可能是:
if(myStore.get(mid).compareTo(idToSearch) < 0){
lowerBoundary = mid + 1;
} else {
upperBoundary = mid - 1;
}如果您在mid中找到元素,为什么不中断循环呢
发布于 2014-03-31 12:21:13
我会的
mid = (hi + lo) >>> 1;以避免溢出导致错误。发布于 2014-03-31 14:37:06
第一个问题是边界,你也应该停下来,以防他发现了价值,但他没有导致可能的忽略。
while(upperBoundary >= lowerBoundary)
{
mid = (lowerBoundary + upperBoundary) / 2;
if (myStore.get(mid).equals(idToSearch)) break; // goal
if(myStore.get(mid).compareTo(idToSearch) < 0)
{
lowerBoundary = mid + 1;
}
else
{
upperBoundary = mid - 1;
}
}https://stackoverflow.com/questions/22752363
复制相似问题