Inconsistent prefixes for INDEX and TASK_INDEX

[tanjiaxian99]

INDEX does not require a prefix, while TASK_INDEX requires a -ti prefix. This could be quite confusing for new users who are just starting to use the app.

[nus-pe-bot]

Team's Response

First off, we did mention clearly in all our command instructions what INDEX refers to, and what TASK_INDEX refers to. So before the user would already be clear of what the command inputs mean.

Since INDEX and TASK_INDEX are used together in the same command, having a -ti prefix makes it more user friendly as it is easier to see which number is for the index of person and which number is for the index of tasks.

Lets take the rm command for example to demonstrate what you deemed as 'consistent':

• If we omit any flags, the command would be something like rm 1 5 which is even more confusing to the user as there is no distinction.

• If we also make INDEX have flags. It would look something like rm -pi 1 -ti 5 which is also very confusing/misleading since it could potentially mean removing both person and task together.

The flags also implicitly serve a purpose to let users know that it is an attribute associated with the person. For instance, -ti refers to the task of a person, -p refers to the phone number of a person, -a refers to the address of a person etc. Commands except deleting a person and adding a person actually manipulate the attributes related to the person and not the person itself. Going back to your point, giving TASK_INDEX a -ti flag would actually make it clear to the users that they are manipulating the task of the specified person and not the person itself.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

• [ ] I disagree

Reason for disagreement: [replace this with your explanation]

---