search

bchenghi / pe

0 stars 0 forks source link

Command format should include slashes. #10

Open bchenghi opened 3 years ago

bchenghi commented 3 years ago

I feel that all commands should have a slash in front of them, for consistency. This makes commands easier to remember as well, as now the user has to remember which commands has slashes and which ones don't.

nus-se-bot commented 3 years ago

Team's Response

Hi,

The reason why the stop command is /stop is because when the user is playing a flashcard game and the translation is stop. Typing stop without the slash will confuse the application into stopping the game. Because /play and /stop are in the same category of commands (PlayMode commands) they have a / included.

The / is used to differentiate between the PlayMode commands and the CommandMode commands.

As you can tell from how we implemented the PlayMode, this application is designed to be very light weight i.e. as little commands as possible AND as simple commands as possible.

We put in extra effort to reduce the amount of commands the user AND the complexity of those commands and hence, we do not consider this as a feature flaw.

Bug is rejected.

Regards, Gabriel Sim

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: The first paragraph by the dev team is rejected as the original issue was about placing slashes in front of all commands. While the dev team responded with why only /stop and /play commands has slashes. For the second paragraph in the response, the dev team stated that the slashes is to differentiate between PlayMode, and CommandMode. The original issue was requesting the dev team to place slashes in front of all commands, so that the user no longer has to remember which commands has slashes and which ones don't. The issue that it might be easier for the user if all commands had slashes and why exactly it was not implemented this way, is still unanswered. The third and fourth paragraph stated that this would reduce the number commands. This is rejected as adding slashes to the front of all commands would not increase the number of commands.