You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The issue is on the inconsistency of error message displayed when a large index number is added.
Current Input:
uc 9999999999999999999999999999999999999999999 n/alex
Current Output:
Invalid command format!
uc: Updates the details of the person identified by the index number used in the displayed person list. Existing values will be overwritten by the input values.
Parameters: INDEX (must be a positive integer) [n/NAME] [c/CLASS/GROUP] [e/EMAIL] [p/PHONE] [t/TELEGRAM ID] [g/GITHUB ID]
Example: uc 1 p/91234567 e/johndoe@example.com
Expected Output:
The person index provided is invalid.
Since there is a validation for invalid userID, is it possible to insert an improvement to this validation such that it can check for inputs that exceeds the maximum amount of ID that this application allows?
For example (this should be the expected output):

However (this is the current output):

The text was updated successfully, but these errors were encountered:
The issue is on the inconsistency of error message displayed when a large index number is added.
Current Input:
uc 9999999999999999999999999999999999999999999 n/alex
Current Output:
Invalid command format!
uc: Updates the details of the person identified by the index number used in the displayed person list. Existing values will be overwritten by the input values.
Parameters: INDEX (must be a positive integer) [n/NAME] [c/CLASS/GROUP] [e/EMAIL] [p/PHONE] [t/TELEGRAM ID] [g/GITHUB ID]
Example: uc 1 p/91234567 e/johndoe@example.com
Expected Output:
The person index provided is invalid.
Since there is a validation for invalid userID, is it possible to insert an improvement to this validation such that it can check for inputs that exceeds the maximum amount of ID that this application allows?
For example (this should be the expected output):

However (this is the current output):

The text was updated successfully, but these errors were encountered: