search

shxr3f / pe

0 stars 0 forks source link

Missing information in Sequence Diagram and Description #12

Open shxr3f opened 2 years ago

shxr3f commented 2 years ago

In the description of the Add Feature there is a statement "ExpenseManager will add the price of the new record to total expense using addToExpense(price)." However this ExpenseManger class is not shown in the Sequence Diagram neither is the method addToExpense. As the description is not vivid enough, it cannot be inferred on how this ExpenseManager interacts with the other Parser and AddCommand classes. Might have been clearer if the ExpenseManager was included in the diagram.

Similarly the creation of the Record object is also not shown in the diagram.

Overall the description and the Sequence Diagram that come along with it does not complement each other.

image.png

soc-pe-bot commented 2 years ago

Your response not required for this bug as the team has accepted the bug as it is.