search

ChongXern / pe

0 stars 0 forks source link

Inconsistent exception message for save command #4

Open ChongXern opened 6 months ago

ChongXern commented 6 months ago

Stage of Game: During game Steps to Reproduce: save -1 Expected: 

SyntaxAnalyser: SAVE expects the 1st argument to be integer between 0 to 8
Ui: Bad Token Error, please check your arguments
Ui: Command could not be executed
Try again

Actual: Screenshot 2024-04-19 at 4.38.56 PM.png

The exception message shown here is completely understandable and will not affect the flow of the game, but it might be helpful to standardise how the messages are shown, such as by printing "between 0 to 8" instead.

nus-se-script commented 6 months ago

Your response not required for this bug as the team has accepted the bug as it is.