nus-cs2113-AY2021S2 / pe-dev-response

0 stars 0 forks source link

mark function #1281

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

able to enter duplicate input values with no error message

unmark.png


[original: nus-cs2113-AY2021S2/pe-interim#1281] [original labels: severity.Medium type.FunctionalityBug]

8kdesign commented 3 years ago

Team's Response

This is the intended behavior. The correct task was marked, while the duplicates are automatically ignored. This was done to provide users with the best possible experience.

As for why there isn't an error message, we believe it isn't an error in the first place, unlike those with index out of bounds. Putting multiple of the same index and putting just one tells us the same thing: You want to select the item corresponding to the index. Since it is a valid index, there is no reason why we need to make it an error.

In fact, putting an error would make things worse as the user may get confused about whether the action was successful or not.

Duplicate status (if any):

--