Skip to content

Binary-Search-1#2458

Open
Pranjal0211 wants to merge 3 commits intosuper30admin:masterfrom
Pranjal0211:master
Open

Binary-Search-1#2458
Pranjal0211 wants to merge 3 commits intosuper30admin:masterfrom
Pranjal0211:master

Conversation

@Pranjal0211
Copy link

No description provided.

@super30admin
Copy link
Owner

  • The student has demonstrated a strong understanding of binary search and its applications to different problem scenarios.
  • The code is clean, well-commented, and follows good practices.
  • One minor suggestion: In the "Search-in-Sorted-array-unknown-size" solution, the initial expansion of the search window could be made slightly more efficient by checking if the target is less than the current high value before doubling the high index, though this is a minor optimization.
  • The file naming for "Search-in-Rotated-sorted-array" is inconsistent (missing .java extension), which could be improved for better organization.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants