search

jyztintan / pe

0 stars 0 forks source link

Non-specific error message for index out of bounds input #16

Open jyztintan opened 5 months ago

jyztintan commented 5 months ago

image.png

This could inconvenience users as they have to check which is the applicable error.

soc-pe-bot commented 5 months ago

Team's Response

This is a very simple check which does not inconvenience the user, since with this error message, as long as users key in a positive number, the issue would be the latter. If the user key in a negative number, it should be clear to the user that the issue is the former.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: The current error messaging within the application lacks specificity, which can lead to user confusion and inefficient troubleshooting. The command takes in an internship index, and a task index. However, the same error message is returned during the 4 following scenarios:

  1. Negative internship index, positive task index

  2. Outofbounds internship index, positive task index

  3. Valid internship index, Negative task index

  4. Valid internship index, outofbounds task index

  5. Negative internship index, Negative task index

  6. Outofbounds internship index, outofbounds task index

This ambiguity forces users to verify two separate inputs—the internship index and the task index, and check for two separate conditions for its validity in each input-- positive number and not out of bounds.