Skip to content

Implements iterative binary search in Search.binary(). - #14

Open
Kimmicorn-glitch wants to merge 1 commit into
lindelwa122:mainfrom
Kimmicorn-glitch:patch-1
Open

Implements iterative binary search in Search.binary().#14
Kimmicorn-glitch wants to merge 1 commit into
lindelwa122:mainfrom
Kimmicorn-glitch:patch-1

Conversation

@Kimmicorn-glitch

Copy link
Copy Markdown
  • Returns index if found, NotFoundError object if not.
  • Handles edge cases: empty list, single element, first/last element.
  • Optimized for large datasets.

Implemented binary search algorithm in the binary method.
@lindelwa122

Copy link
Copy Markdown
Owner

@Kimmicorn-glitch, your implementation works, and I have marked it as hacktoberfest-accepted. But I am just curious, why did you remove the docstrings?

@Kimmicorn-glitch

Kimmicorn-glitch commented Oct 20, 2025 via email

Copy link
Copy Markdown
Author

@lindelwa122

lindelwa122 commented Oct 20, 2025

Copy link
Copy Markdown
Owner

@Kimmicorn-glitch, no it's okay, I just wanted to find out your reasoning.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants