tyuyang / ped Goto Github PK

This bug applies to the following commands:

1. remove-pet
2. remove-stat

Will this count as 2 bug? :)

When the user enters a NAME that is not part of the existing pet list, the program will sometimes crash, and sometimes show the ERROR: Pet not found error message. I could not find a pattern for those crashes though.

I entered a valid command though? It has missing arguments, but still a valid command right?

When add-stat command with NAME = hi is used, an error is thrown, but the success message "Updated age to abcd for hi" is still shown. Another example below:

In the viewing pet list section, the format and example of usage is inconsistent. One is list-task and the other is list-tasks. It is unclear which one is the correct format, or even both.

In the adding a stat to a pet section of the user guide, there are no examples of what are considered valid stats, so the user can assume that any random string can be entered for the STAT parameter.

I typed a space right after add-pet.

Example above is when I used the add-pet command for the NAME " hihi" rather than "hihi"

Doing the commands: ("" added to show the spaces, commands are performed without the "")

1. add-pet "hihi "
2. remove-pet "hihi "

Will crash the program, which further shows that it is space sensitive.

As seen above, trying to mark task 2 when there is only 1 task results in the program crashing.

Exactly what the title says. Messages are shown below.

To be more helpful, shown below are the savefiles.

The program should check if input is valid number for the stats age and weight since only numbers should be expected as input for those stats.

In the user guide, there is no exit feature. The only way to exit the program is to close the whole terminal? I assume that this is just missing documentation since typing exit in the terminal actually exits the application.

For the adding a pet feature, the NAME parameter is ambiguous since it could also refer to the name of the owner rather than the name of the pet. Recommended to specify in the user guide that the name refers to the name of the pet.

Logging messages are shown when using the application. Example when I typed the command exit below.

As a user, one should expect that trivial logging messages like those should be tagged under FINE to FINEST rather than INFO.

In the removing a pet section of the user guide, the description of that section has grammatical errors. "Removes a pet to the Pet List" should be "Removes a pet from the Pet List" instead.

From removing a task section to marking a task as not done section, the parameter NUMBER was never clarified to mean task number, which can be unclear for the user. None of the expected output examples showed any relationship between the NUMBER and the task number either.

In addition, the user will not be able to tell if the NUMBER parameter refers to the number as seen in the list-task or schedule command.

In the viewing pet list section, the first word view is in plural form, which is grammatically inconsistent with the rest of the user guide. It should be views instead.

In quick start of the user guide, step 3 is missing the instruction to cd into the directory that the pettracker.jar file is in. Running the stated command without changing the directory results in this:

Omitting the part where I type the command out for privacy reasons. Just trust that I did what I said.

Assume that I only did 2 add-pet commands before the commands shown above.