Shubham Gupta - TCH
Shubham Gupta - TCH

Reputation: 145

Why is my Binary Search slower than Linear Search?

I was trying to code a Binary Search and Linear Search and I was shocked by seeing that binary search is slower than Linear Search by sometimes even by 2 times. Please help me. Here is my code.

Binary Search Code:

def binary_search(array, target, n=0):
    l = len(array)-1
    i = l//2
    try:
        ai = array[i]
    except:
        return False
    if ai == target:
        n += i
        return (True, n)

    elif target >= ai:
        array = array[i+1:l+1]
        n += i + 1
        return binary_search(array, target, n)

    elif target <= ai:
        array = array[0: i]
        return binary_search(array, target, n)

Linear Search Code

def linear_search(array, target):
    for i, num in enumerate(array):
        if num == target:
            return True, i

    return False

Test Case Code:

import random 
import time
n = 10000000
num = sorted([random.randint(0, n) for x in range(n)])

start = time.time()
print(linear_search(num, 1000000))
print(f'Linear Search: {time.time() - start}')

start_new = time.time()
print(binary_search(num, 1000000))
print(f'Binary Search: {time.time() - start_new}')

Upvotes: 2

Views: 614

Answers (1)

satojkovic
satojkovic

Reputation: 699

As @khelwood said, your code will be much faster with no slicing.

def binary_search_no_slice(array, target, low, high):
    if low > high:
        return False
    mid = (low + high) // 2
    if array[mid] == target:
        return True
    elif array[mid] > target:
        return binary_search_no_slice(array, target, low, mid - 1)
    else:
        return binary_search_no_slice(array, target, mid + 1, high)

Added below to your test code.

start_new2 = time.time()
print(binary_search_no_slice(num, 1000000, 0, len(num) - 1))
print(f'Binary Search no slice: {time.time() - start_new2}')

Here is the result on my machine(macOS Catalina, 2.8GHz Corei7, 8GB RAM)

False
Linear Search: 2.172485113143921
False
Binary Search: 0.56640625
False
Binary Search no slice: 2.8133392333984375e-05

Upvotes: 1

Related Questions