search

djleong01 / pe

0 stars 0 forks source link

Did not handle decimal point max well #3

Open djleong01 opened 2 months ago

djleong01 commented 2 months ago

after entering max decimal point (1.11111111111111111), the table doesnt format or round up properly

image.png

soc-pe-bot commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

incorrect unit price in reciept

when user previously inputs a item cost with large number of decimal places, the item is not the same as reflected in the receipt when ordering

image.png

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

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

For the receipt, the price is intentionally rounded off to two decimal places, since that is the convention when expressing money. Moreover, that is the receipt that is for the customer and it is hence formatted as such.

We however do accept that we should have done rounding off for decimal places for the menu to improve the consistency of the program.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]

## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.VeryLow`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]