search

ziyingli / pe

0 stars 0 forks source link

Use of aggregation symbols #11

Open ziyingli opened 4 years ago

ziyingli commented 4 years ago

Syllabus says to avoid the use of aggregation symbols

Screenshot 2020-04-17 at 3.12.40 PM.png

Screenshot 2020-04-17 at 3.12.49 PM.png

nus-pe-bot commented 4 years ago

Team's Response

The textbook states this line about using aggregation symbols: "Martin Fowler’s famous book UML Distilled advocates omitting the aggregation symbol altogether because using it adds more confusion than clarity." However, it does not state that using the aggregation symbol is not allowed, simply that one piece of advice is to use the symbol with caution. The use of aggregation symbol here shows that each of the components are standalone and are not deleted when the Manager is deleted. This can actually help other developers when they are deciding how they want to extend the each of the subcomponents. Will not fix this issue.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]