nus-cs2113-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Wrong response when addf command parameters are wrong #710

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

Screen Shot 2020-04-15 at 12.53.51.png

I believe the addf command does not have anything to do with weight. Not sure why it says 'Invalid index of weight! Please check and try again.' when the command format is not followed.

This causes occasional inconvenience to users when they enter the command wrongly (which could be rather common as it is easy to mistake -- for - in the command) but they can continue to use the product.

_{[original: nus-cs2113-AY1920S2/pe-interim#715]}

yuxianglim commented 4 years ago

Team's Response

The usability is not affected in this case. This error might be caused when there is already an error in the user's weight such as empty weight after user accidentally deleted all weight from the record. Thus the system might prompt the user to check the weight prior to using anymore functions in the program.

We can rectify this small UI issue by making the statement or intuitive for the user based on the command input if required.

Duplicate status (if any):

Duplicate of #694