larrywang0701/ped

ped's Issues

Command examples for "add" in UG gives "Invalid command format" error in the app

The command examples in UG does not work in the app

add n/John Doe p/98765432 e/[email protected] a/John street, block 123, #01-01 sb/Primary 4 Math d/wed b/1500 e/1600 pr/20

add n/Betsy Crowe p/92939402 e/[email protected] a/Newgate Prison p/1234567 s/Secondary 3 Physics d/mon b/1900 e/1930 pr/35

Command examples for "edit" in UG gives "Invalid command format" error in the app

The command examples in UG does not work in the app

edit p/91234567 d/Sun

edit n/Betsy Crower a/Betsy street, block 110, #03-02

Getting wrong error message for some commands

When I execute the command

find n/A abc/B

The app give me the following error message:

Names should only contain alphanumeric characters and spaces, and it should not be blank

But actually the error message should be something like invalid command format

Several command accepts redundant content after the command itself

Some command does not check whether that are redundant input contents. For example, the following commands can execute without any errors:

help abcdefg

unpaidAll abcdefg

undo abcdefg

redo abcdefg

rev abcdefg

exit abcdefg

In UG: `find` command does not have correct optional parameter description

In UG, the find command has format find n/NAME sb/SUBJECT, which means that both "n/" and "sb/" must present (without indicating optional parameters). But from the real behavior of the app, parameter "n/" and "sb/" can both exist or only exist one (but at least must exist one). For example:

find n/A

find sb/A

Better handling the situation of failed to save data for the app

When the app can not save data due to some reason, it will give the error message Could not save data due to the following error: data\tuitionconnect.json. But maybe you need to specify why the app can not save data, or the users may confused.

And also, the program continue runs after failure to write to file. The information that the user make will store in RAM for this session, but when you restart the app, the data will lost. Maybe add a pop up box to notify the user to select another location to store the data file when the user is closing the program but the data failed to write to the default location?

Note: I use Windows file permission to set tuitionconnect.json to totally denied permission to test the situation of can not read/save data