nus-cs2113-AY2021S1 / pe-dev-response

0 stars 0 forks source link

Wrong commands accepted for random bunny, save bunny, list bunny and reset bunny #2450

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

For these features, as long as the first few characters match, the system will deem them as valid command. So when I type random bunny12345, or save bunny nidaidwnindiwnd i or list bunnyyyyyyyyy or reset bunny8384ddjdjjdjdj, the system accepts them as valid command. Screenshot 2020-11-13 at 17.09.09.png Screenshot 2020-11-13 at 17.10.47.png Screenshot 2020-11-13 at 17.09.26.png Screenshot 2020-11-13 at 17.11.07.png Screenshot 2020-11-13 at 17.11.14.png


[original: nus-cs2113-AY2021S1/pe-interim#2366]

AmeliaTYR commented 3 years ago

Team's Response

Rejected as it is by design for the app to accept commands with typos as long as the key components required by the command are present. The reason is to prevent frustration of the user when they make typos while fast typing.

Again, by typing the key component of the command and having spelled it correctly shows intent that the user wanted to use the command (it is highly unlikely for the user to keysmash the command by accident), especially for simple commands like list bunny with no argument. Thus it makes sense for the app to proceed with carrying out the comand despite the typo of adding extra characters.

The app is still usable despite accepting additonal/ excessively typed commands, and can still perform the necessary commands as expected.

Duplicate status (if any):

--