Reputation: 145
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
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