search

AY2425S1-CS2103T-W13-2 / tp

MIT License
0 stars 4 forks source link

Allow partial string matching in search #72

Closed SuhailLoya closed 2 weeks ago

SuhailLoya commented 2 weeks ago

Fixes #54

Changes made

1) Added partial string searching for find command. So now find "john yeo" will find "johnny", "john doe", and "alex yeoh". 2) Added new tests for the enhancement 3) Updated the UG accordingly

Possible further enhancements

1) Possibly enforce a minimum length for the search string so not too many matches are returned 2) Possibly implement some sort of sorting in the filtered list displayed so more closely matching names are shown first

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Files with missing lines Coverage Δ Complexity Δ
...in/java/seedu/address/commons/util/StringUtil.java 96.15% <100.00%> (+1.41%) 10.00 <3.00> (+3.00)
...ss/model/person/NameContainsKeywordsPredicate.java 100.00% <100.00%> (ø) 7.00 <1.00> (ø)