search

Tanishq4331 / pe

0 stars 0 forks source link

Confusing command format in addtoclass, removefromclass #5

Open Tanishq4331 opened 2 years ago

Tanishq4331 commented 2 years ago

The absence of parameters in some commands makes it confusing when entering different types of information.

For instance:

image.png

In the above example there is nothing to distinguish class indices and student indices. This might be confusing for the user and lead to mistakes.

If parameters were defined, a command format like the one below could be used, which would be clearer :

addtoclass c/CLASS_INDEX s/STUDENT_INDEX...

nus-pe-bot commented 2 years ago

Team's Response

Our product is catered towards fast typist, and by removing unnecessary flags, the user would be able to input the command faster. Thus this is not a bug. Additionally, the order of the inputs are very intuitive because there is only one class we can add to for each command, and the first number has to be the class index. This is stated in the UG as well. Therefore we feel that it only makes it harder and confusing for the user if we add the c/ and s/ fields.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]