search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Commands require the n/NAME parameter, and if there are duplicates, the usage of "sn". This can be improved. #87

Open nus-se-script opened 2 weeks ago

nus-se-script commented 2 weeks ago

There are several commands where if duplicate student names exist, the user has to input the name and the student number.

This is an inconvenience to users since if name was not useful as an identifier, why not simply use student number from the start (which is guaranteed to be unique)?

Some examples include markat or adda.

Severity low as the odds of same name is low.

Examples include the below, where John Doe is repeated twice.

image.png

[original: nus-cs2103-AY2425S1/pe-interim#120] [original labels: type.FeatureFlaw severity.Low]

Edsel-Tan commented 2 weeks ago

Team's Response

We have two interpretations of your suggestion.

  1. We use student number only - there is no way to delete by name. We believe this is objectively a downgrade to our current model since names are more intuitive for tutors. The student number is simply a way to disambiguate in the rare case of duplicate student names.

  2. We add another option to delete by student number only. This is a possible improvement in the future.

Duplicate status (if any):

--