search

nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Unclear exception message for updating cat of grocery #506

Open nus-se-script opened 4 months ago

nus-se-script commented 4 months ago

Input typed in

cat kailan d/

expected output

error message telling me to change d/ to c/ instead it tells me that grocerky kailan d/ does not exist

actual output

The grocery (kailan d/) does not exist!

Screenshots

image.png

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

64-1 commented 4 months ago

Team's Response

The User Guide states the correct format which is c/ not d/.

image.png

If the format entered is wrong, our app prints an error message outlining which part of the user input is wrong (The grocery (kailan d/) does not exist!). In your example, the error message printed out this intended.

For this stage of our product, we believe our current error message is clear enough to inform the user which part of their input is wrong i.e. d/ appears to be the wrong parameter, allowing the user to adjust their input accordingly.

However, we do agree with your suggestion to make the error message clearer.

Duplicate status (if any):

--