myrilla12 / pe

0 stars 0 forks source link

Assign error message can be made clearer #5

Closed myrilla12 closed 1 week ago

myrilla12 commented 1 week ago

Details

I initially assumed that trailing whitespaces were the cause of the error, before realising it was due to existing pid and employee id pair in assignments. Below is the bug report made related to this false assumption: Upon executing assign aid/3 pid/10 id/3 the following error message is received despite the fact that the assignment does not exist in the addressbook: Screenshot 2024-11-15 at 5.03.03 PM.png

This error message is inaccurate. Application is unable to handle trailing whitespaces after id parameter. Similar issues for inputs like assign aid/3 pid/10 id/3. Screenshot 2024-11-15 at 5.06.12 PM.png

The only conditions for Numeric inputs is stated in the UG as follows: Screenshot 2024-11-15 at 5.14.38 PM.png

No mention of not including whitespaces found.

Suggested improvement

From the above misunderstanding, I believe that the error message can be improved to avoid similar confusion with other users. I think pid and employee id pairing should be further clarified and a line break could be added so that all the important sections of the error message would be visible on majority of screen sizes.