nus-cs2113-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Incorrect error message thrown when entering different formats other than numbers #1097

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

image.png

this is a NFE, rather than a not found


[original: nus-cs2113-AY2122S1/pe-interim#1071] [original labels: type.FeatureFlaw severity.VeryLow]

richwill28 commented 2 years ago

Team's Response

While it is true that a might not be a valid module from NUSMods, however, our current implementation is more efficient. Consider the two scenarios:

1. User input module -> check user list -> exist or not
2. User input module -> check NUSMods -> check list -> exist or not

Note: Checking NUSMods is a relatively expensive operation

We choose to only check the module in the user list to improve our program efficiency (i.e., to avoid scenario 2). Also, our current error message is technically correct as well.

Duplicate status (if any):

--