search

daryltay415 / pe

0 stars 0 forks source link

Print message seems to be different from quantity removed for removing flowers #3

Open daryltay415 opened 7 months ago

daryltay415 commented 7 months ago

image.png

I tried removing 70 red roses from a bouquet with only 20 red roses. This caused the program to remove only 20 roses however, it prints the message that I have removed 70 red roses.

nus-se-bot commented 7 months ago

Team's Response

We felt that because of the line "tried to remove more than the quantity available, quantity set to", it makes it clear enough to the user that instead of going to a negative value, the quantity is set to 0. Thus, the user was able to "successfully" remove more than the quantity, albeit with a warning.

However we understand that the "You have successfully removed the following: ... " might sound misleading to the user and it is an easy fix we can implement in the future.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]