search

ChongXern / pe

0 stars 0 forks source link

Command head does not work with extra space character #5

Open ChongXern opened 6 months ago

ChongXern commented 6 months ago

Stage of Game: Beginning Step to Reproduce: head Expected: One of the three coin outcomes to be printed Actual:

Screenshot 2024-04-19 at 4.52.40 PM.png

Might affect usability for some users. Even though it is not major, it can still be helpful to do a trim on the command.

nus-pe-script commented 6 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]

SyntaxAnalayser does not account for whitespaces

When i enter ' tail' or ' tail ' it does not register as command input 'tail' I think it would be helpful for users if the program trimmed inputs for whitespaces.

Also, the error message returned is not very intuitive - an user unfamiliar with string processing would not pick up that the issue was because of a trailing whitespace.

image.png

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

Their Response to the 'Original' Bug

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

We accept that it is a bug, but we think that the severity should be low, because if user follows UG, he rarely enters commands with extra spaces. Also, it only causes a minor inconvenience because the user just needs to enter the right command again.

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]