limjiajing / pe Goto Github PK

A large white space before the next page

From the user guide, I inferred that there could be two possible ways to input the data

However, it is not the case in the software

Perhaps the error message being "Missing Module name" would be better

Overall, the UserGuide is quite unfriendly and I had a hard time understanding what I can do with the product. Based on individual commands, I knew what to do, however, I took a long time to understand what the software offers. For example, I was wondering why do I need to list the semesters. Then I realized that this program offers me a feature to edit modules based on semesters. Although it is good that the features offered are a wide range, it got confusing as I was wondering what do I need to add or how do I fill in a certain part of the content. e.g. what is BREAKDOWN etc. in the image attached below, since after I added a module, there were many empty fields. But what followed in the user guide was how to delete a module, mark modules task as done, list modules etc. Although I could eventually find it in the user guide, it was because I purposely went to find it (and even that took some time). Overall, the order in which the features are presented in the UserGuide can be improved and a nice overview of the features offered by the application would be nice.

Rather than saying Semester successfully switched again when user was already in Y1S1 beforehand, it could tell the user that he/she is already accessing Y1S1. This message could cause the user to have some misunderstanding on which semester he/she was previously accessing

I was allowed to add a class at 1800, but the timetable does not reflect it

No checking done on credit

UserGuide uses "module name" and "module code" , and "module" ambiguously. At times, "module name" should be more appropriate, but "module code" is used. Attached are some instances where the terms could confuse the user.

In the above two pictures, the command referred i assume it is the same, but the command format given in the UserGuide differs i.e. module name vs module. As a result, i tried to do "add Software Engineering and OOP", took away the space to "add_Software_Enginnering_and_OOP" and hit another error. Until I realised I had to put in a module code instead.

constructor shown wrongly

timetable format have problems when the window isn't opened wide enough. In view of the purpose of this software, this bug cannot be considered cosmetic. For example, the below scenario could cause some problems for users. Could cause potential misunderstanding if the user did not realize 1700 overlapped with 0800 column.

Perhaps some checking could be done location input

I would expect that instead of adding a module, then adding the credits, lessons etc. I could input it all at once. Since a module will definitely have credits and lessons.

Looks as if we are to type in two module name. Maybe a comma or a fullstop between the two "module name" will be better.

The restriction on the timeframe of a lesson to 1h greatly restricts the user as many lessons occur in 2 or 3-hour blocks. In view of the purpose of this software, such a restriction is unexpected and majorly compromises the value of this software.

-1 and 10 are are different in terms of why it isn't accepted, hence, maybe the error message could reflect that.
e.g. -1 --> Only positive integers accepted
10 --> You currently only have 3 tasks

Error message does not tell me where exactly in the input is invalid once the command is invalid. Perhaps could use "invalid command" instead so that the user knows its the command that is the user.

Two different type of arrows used for return

Perhaps some check can be done on time where a class can occur

Maybe the error message could be "invalid time" instead.

dummy report edit