aweijun / pe

0 stars 0 forks source link

Inconsistency in Model.diagram [DG] #10

Open aweijun opened 2 years ago

aweijun commented 2 years ago

Current

image.png

Proposed

In some cases multiplicity seem to be included, yet in others it is not. You could standardize it to reduce user confusion.

nus-se-script commented 2 years ago

Team's Response

We intentionally place multiplicity as certain points to emphasise to the reader that there is only one item associated with it (e.g. AddressBook and TaskList), as their multiplicities would not be obvious to a person trying to understand the implementation of our app.

For the rest, we do not use multiplicities because it is obvious that a person would only have one name, for example.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: While the name example meant make sense, it is also possible for people to have multiple multiple username. I still believe this could have been made clearer.