nus-cs2103-AY1920S1 / pe-dev-response

0 stars 0 forks source link

addExpense does not accept amounts starting with a decimal point #207

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

Summary: When the amount entered into the addExpense command starts with a decimal point, an error appears even though the number meets all the constraints specified in the user guide ( (0-100000), no more than 2 decimal points, only digits).

Steps to Reproduce: Execute the following command: addExpense cat/food n/Mala amt/.1 d/2019 11 09

image.png


[original: nus-cs2103-AY1920S1/pe-interim#207]

jessicax941 commented 4 years ago

Team's Response

We acknowledge that it can be an issue but it causes a very minor inconvenience to the user as most expenses would be at least a dollar. Also, adding an amount of 0.1 works fine:

image.png

We also do not feel that it is a feature flaw as the addExpense feature does not become less useful for normal usage as they could easily key in 0.1 instead, which does not take very long at all to key in, as compared to .1 Instead of that, we felt that it is more of a documentation bug. We could have included a short note in the User Guide that amount parameter has to start with a digit and cannot start with a decimal point.

Duplicate status (if any):

--