PDHung1104 / pe

0 stars 0 forks source link

Output of `list` command is not consistent #8

Open PDHung1104 opened 2 months ago

PDHung1104 commented 2 months ago

I played a few lucky rolls for player Linh### and Giang:1000 in group bcd.

After a few rolls, this is the last 4 elements of list:

image.png

After exiting and running the program again, I only entered list and this was the result:

image.png

nus-se-script commented 2 months ago

Team's Response

The bug is caused by the use of : and # in the username Giang : 1000 and Linh###. There is a : and # in the username string that is used as a delimiter for our database. Therefore, upon exit and reentry of program, the expenses were loaded in incorrectly which caused the wrong value to be shown for expenses upon entering the list command.

image.png

This error is not high as it is only produced when fields entered contain : or # in them. Most users are unlikely to enter these characters on a general basis. As such, we think that the severity should be medium because this is a flaw that causes occasional inconvenience to some users that use : or # in their usernames. Furthermore, they can continue to use the product because their data is still saved, and they can still use other features. Hence, this will not make the product unusable for most users.

This is a duplicate of issue #87 as the user entered the : and # characters in one of the parameters which caused the bug.

The 'Original' Bug

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

application crashes upon use of balance command for decription with #s in expenses

make sure alice and bob are members of the group expense list used

image.png

command to add expense: expense stuff#8 /amount 9.00 /paid alice /user bob

upon exiting and entering the application, enter the group with the added expense and type balance alice

image.png


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

Their Response to the 'Original' Bug

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

The bug is caused by the expense command: expense stuff#8 /amount 9.00 /paid alice /user bob. There is a # in the expense description that is used as a delimiter for our database. Therefore, upon exit and reentry of program, the moneyList parameter in the Group class was not initialised which caused an illegal access error when entering the balance alice command again.

image.png

This error is not high as it is only produced when fields entered contain # in them. Most users are unlikely to enter these characters on a general basis. As such, we think that the severity should be medium because this is a flaw that causes occasional inconvenience to some users that use the # in the description, payee_name and payer_name. Furthermore, they can continue to use the product because their data is still saved, and they can still use other features. Hence, this will not make the product unusable for most users.

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.Medium`] Originally [`severity.High`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]