powzx / ped Goto Github PK

In this case, I think the error comes from the order index being out of range instead of the person being invalid. A suggestion would be to consider different error handling for these separate cases?

Similar to quantity out of range, maybe can print a more specific error message instead of a generic one?

Similar to the edit command, maybe can consider to handle invalid person and index out of range errors separately.

A suggestion is to consider names as case-insensitive strings?

An overall suggestion is for a storage function to save user data locally so that they can come back to the program to check back on the list?

As a user, I might get confused for this because it seems to be a valid command. A suggestion to consider can be to print what the user should do? Or maybe can print the correct format for the user?

As a user reading the user guide, it was not mentioned that the menu command cannot take in inputs, a suggestion would be to inform users in the user guide not to enter inputs after the menu command?

I understand that in the UG it is stated that the maximum quantity is 999, maybe a suggestion to consider is to also inform the user that the quantity is out of range as an error message instead of a generic invalid add command?

The list contains "john" and "John". As a user who can distinguish between "john" and "John", I might not be able to after using the find command since they both return "john" to me. Maybe can consider making names case-insensitive when adding to list so this bug may be indirectly eliminated too?

The message suggests "delete 1/a" but the user guide says that the food index is an integer.