search

nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Find-s displays all tutors #5270

Open nus-se-script opened 11 months ago

nus-se-script commented 11 months ago

Similar to find-t, find-s also displays all the tutors with Lee when I am looking for only Lee Jun Hong

image.png

In this case, there are only 8 schedules. However, in a larger set of data, there may be thousands of schedules to sieve through although I only want to see Lee Jun Hong's schedule

It would also be inefficient to use list-s [INDEX] to find that tutor's schedule if I have thousands of tutors to look through before being able to find his index.

[original: nus-cs2103-AY2324S1/pe-interim#4800] [original labels: type.FeatureFlaw severity.Medium]

saltedfishxx commented 11 months ago

Team's Response

While it's true that searching for a common surname might yield a large number of results, it's essential to understand that the current feature was designed to allow for flexibility in case a user doesn't have the exact spelling, or wants to see a broader list of tutors with a similar surname. This flexibility can be beneficial in scenarios where users might not be sure of the complete name or want to explore options.

Additionally, the app was created with the intention of serving a tution coordinator handling 20-50 tutors. As such, the likelihood of encountering 1000 tutors with the same surname or name keyword will be very low.

Duplicate status (if any):

Duplicate of #3122