mitchmalvin1 / pe

0 stars 0 forks source link

Arrow is Class Diagram #18

Open mitchmalvin1 opened 1 year ago

mitchmalvin1 commented 1 year ago

Screenshot 2023-04-14 at 5.29.33 PM.png

The association between the classes should be modelled with a solid arrow rather than a dotted one. Even if this is intended, there is almost no explanation to this class diagram in the DG.

nus-pe-script commented 1 year ago

Team's Response

The usage of arrows with dotted line can be used to indicate dependency between 2 classes. For Parser - -> AddCommand, it follows the usage showed during lecture 8, since it is similar to the case where "ActionFactory creates ActionX". The other 2 uses of arrows with dotted line indicate that Ui or FlashcardList is used inside a function under AddCommand, but is not stored inside AddCommand, which is shown by the parameters listed inside execute(...).

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]