William

[WilliamAlkawaz]

No description provided.

[WilliamAlkawaz]

• Input validation is required because we're not sure about user's intent and also to avoid non-meaningful input. This can be achieved by creating a function for each datatype validation with a prompt to show a meaningful message to users.
• One simple warning in the Order class has been remove.
• It is always a good practice to use try and catch with meaningful error messages.
• Encapsulation issue has been resolved. Local variables should be private such as "Triangle Price" and so on.
• Follow naming convention with variables and methods.
• Consider use decimal type for price.
• Class documentation can help other programmers understand the code easily.
• Finally, unit test has not been found.