search

Jane-Ng / pe

0 stars 0 forks source link

DG: Overall class diagram and Model Component #17

Open Jane-Ng opened 4 years ago

Jane-Ng commented 4 years ago

image.png

From the overall class diagram, it does not show that Parser inherits from the Command classes

image.png

However, in the model component, the Parser inherits the Command classes

nus-pe-bot commented 4 years ago

Team's Response

This was not intended to be a class diagram but instead it was an architecture diagram to show the relations between the data transfer between different classes. In that sense, the Parser sends data to the commands, the commands then utilise the data from the teamMembers and projects list and the projects list utilises the data from the tasks.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: All the arrows and lines used are what will be used in a class diagram. So it can be very misleading as it will come across as a class diagram. And I don't think this is something that is not in scope. Using a sequence diagram might be better to show the data transfer between different classes.

:question: Issue severity

Team chose [severity.High] Originally [severity.Medium]

Reason for disagreement: [replace this with your explanation]