search

rismm / pe

0 stars 0 forks source link

Class diagram of Command #12

Open rismm opened 2 months ago

rismm commented 2 months ago

It can be seen that the Command classes inherit from the abstract class Command, and some classes inherit from the abstract classes ListActionCommand and TransactionActionCommand. I think it would be beneficial if there could be more in depth explanation on the details of the implementation, as it is unclear why there are two separate abstract command classes in addition to the main abstract Command class. There could be additional details on the abstract methods that subclasses would inherit.   image.png

nus-pe-bot commented 2 months ago

Team's Response

Hi there, thank you for the report.

Indeed you are right that we can add in abstract methods for the abstract classes. However, our DG has been revised with the teaching team that it was okay to omit these abstract classes in order to keep the diagrams simple. In addition, as can be seen from the image of AB3 class diagram below, they have also omitted all kinds of methods or variables from their classes, therefore we found it unecessary to include the methods and variables in the class diagrams.

image.png

Thank you again for the report!

Best Regards, StockPal Team

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]