Skip to content

Commit

Permalink
Merge pull request #338 from rxchell/index-parsing-enhancement
Browse files Browse the repository at this point in the history
Check for valid index [Enhancement]
  • Loading branch information
zaidansani authored Nov 11, 2024
2 parents bb2dc96 + ed48614 commit feab79b
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions docs/DeveloperGuide.md
Original file line number Diff line number Diff line change
Expand Up @@ -1114,6 +1114,15 @@ Team size: 5
* **Sample Input:** User enters `2023-03-32` for an appointment date.
* **Expected Output:** The result display box shows `Invalid date: The day is out of range for the month`.


4. **Modify index parsing to check for valid indexes**
* **Flaw:** For `edit` commands utilising `INDEX` as a field, when users input an index larger than the
list size of `Person` or `Appointment`, the application does not check for the validity of the index.
* **Enhancement:** Implement a check to ensure that the index provided by the user is within the valid range
of the list size. If the index is invalid, display an error message to the user.
* **Sample Input:** `edit person 100000`
* **Expected Output:** The result display box shows `Invalid index! Please enter a valid index within the list size.`

<br>

--------------------------------------------------------------------------------------------------------------------
Expand Down

0 comments on commit feab79b

Please sign in to comment.