search

Decaxical / pe

0 stars 0 forks source link

Student Referenced using Index is not updated after using List #1

Open Decaxical opened 2 years ago

Decaxical commented 2 years ago

While it has been stated in the User Guide, given the scenario where the TA wishes to enrol a student after using the find command, having to list student again in order to find the index of the student in order to enrol them is too troublesome and diminishes the usefulness of the find command.

nus-pe-bot commented 2 years ago

Team's Response

The intention of the find command is to let the TA easily find a particular student that they may be teaching this semester. As they can be teaching across different modules and classes, this find command will help them easily locate a particular student given the name of the student. Based on our use case stated in our DG, finding students by name would benefit the TA by allowing them to keep in contact with the students.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Sorry if I caused any misunderstandings, while I understand that the intention of the find command is only for locating students. I believe that it is a FeatureFlaw because the Student Index that all commands reference is the index of the original student index as displayed by list student rather than the currently displayed list of students as shown by commands such as find. This can make it confusing for the users and also makes it more inefficient for the users as they would have to scroll through the list to find the index of the student they wish to run operations on. While the users have other means of identifying the student other than index like student ID. It would be a lot faster and more convenient if they were allowed to use the index of the currently displayed list.

I stated that it "diminishes the usefulness of the find command" because it had the potential to be more useful.