louislouis19 / pe Goto Github PK

As a user, if I update my data manually, and set my quantity as a double (1.5), as I in fact have some half-packet of items in the fridge, this value is not filtered out. No warnings mention in the UG and prevents the program from starting as it throws an exception

I edit the quantity from files and set it to minus, this is not handled properly when file is reloaded. Perhaps invalid data as such should be discarded, as this affects the functionality for users.

As seen below, I would like to buy a huge amount and a huge number in the saved shop is not filtered out properly. Program unable to start, exception thrown

I believe that the second and third paragraphs are referring to one point. Could perhaps use indentation to indicate that paragraph 3 is just an example of paragraph 2, instead of being referred to as another separate point

Let's say I want run out of wine, and I want to add wine to the shopping list. Unfortunately, I did not maanage to get it right in the first try. Now, I am unable to add wine to my shopping list... I have to re-add wine to my list and remove it before adding to my shopping list, such inconvenience for users.

Perhaps a better feature design could be possible or give an extra attempt for users to add a valid number

When I change the name of my food to "|", an exception is thrown at the start of the program. Preventing the program to start. Perhaps a filter could be implemented for this.

When referring to a method inside a class, I believe that the proper notation is "Class#method()", instead of "Class.method()"

As a disgusting person, I want to keep expired items for future use. I am unable to add manually, but I can add expired items directly from the save file. Perhaps consistency is needed, either filter out the expired items, or allow expired items all together

When I changed the food name burger into an empty string in the file, the values in the file is never filtered. Causing an exception to be thrown and program unable to start

Shopping list is a feature, but only stated as an extra note. Perhaps, having a separate seection or moving it under the remove section (as it is integrated) could help.

For self invocations like in step 7 and 11, shouldn't the dotted arrows be on the right hand side, instead of left?

Shouldn't it be something like this?

Error message states that "number can not be less than 0". This implies that 0 is allowed, which makes no sense(?)

In the architecture section, you styled the methods, like execute() here.

However, in the implementation section, methods are not styled in the same way. Perhaps having a consistent styling would help

As a user, if I edit manually from the file and insert an extra space, in this case, before "Qty", this saved data will not be filtered out and prevents the program from running, as an exception is thrown.

My notifications are on, and I expect a health reminder notice at the start of the program. It never appears though.

Random values are added, as a result of my confusion on how to add items to the shoplist

Unable to list expiring item. I have an expiring item in 7 days, which is 19 November, today is 12 November. It is not displayed on the list

Perhaps could give user another chance to put in quantity after an invalid attempt, instead of having to re insert commands

Two screenshots to show the difference in format.

Trial

As a super rich user, I would like to add billions of billions of food to my fridge. I set the quantity to a very huge quantity from the save file. And this value is read as a string as it is above INT_MAX, and not filtered out properly.
Preventing the program from starting all together, as it throws an exception

Although it's just the steps to indicate the end of the method, you included it in other implementation explanations. So, I expect it to be in this explanation as well

As fridget list and shoplist are not synchronized, how do I add to the shoplist?

With the diagram being cut half this way, users might not be able to view it clearly. Below are 2 examples