二进制搜索实现java算法

Question

我需要帮助实现二进制搜索算法，有人可以告诉我我的代码有什么问题：

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

Answer 1

我认为您在更新lowerBoundary和upperBoundary时犯了一个错误。

可能是：

    if(myStore.get(mid).compareTo(idToSearch) < 0){
        lowerBoundary = mid + 1;
    } else {
        upperBoundary = mid - 1;
    }

如果你在mid找到元素，为什么不打破循环？

Answer 2

我会

• 当下限和上限相同时停止。
• 找到匹配时停止
• 使用mid = (hi + lo) >>> 1;来避免导致错误的溢出。
• 阅读JDK中的代码，因为它正常工作。

Answer 3

第一个问题是边界问题，你也应该停下来以防他找到价值，但他并没有导致可能被忽视。

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;
        }
}