search

nus-cs2113-AY2021S2 / pe-dev-response

0 stars 0 forks source link

Invalid price range rounding for reco #1263

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

image.png

Price range inputted by me was 8.155-10.055. According to the UG, if the input is more than 2 d.p. it will be rounded up to the nearest 2 decimal places. However, when typing the list command, the price range shows 8.15-10.05 which is invalid when rounding up 8.155-10.055. Correct rounding up should be 8.16-10.06.

[original: nus-cs2113-AY2021S2/pe-interim#1263] [original labels: severity.Medium type.FunctionalityBug]

huachen24 commented 3 years ago

Team's Response

We should have said that the prices are truncated if more than 2 decimal places. However, prices are usually given to the nearest cent, so it is a minor issue.

Duplicate status (if any):

--