search

LapisRaider / pe

0 stars 0 forks source link

Missing multiplicity for Person association to it's fields #9

Open LapisRaider opened 2 years ago

LapisRaider commented 2 years ago

Should explicitly state the multiplicity of person association to email, name, phone and address. Since a person must have email, name, phone and address, the multiplicity of 1 must be stated. If it's not stated it is assumed that those fields can be empty.

image.png

image.png

soc-se-bot commented 2 years ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

Rationale

The textbook says that multiplicities are optional in the class diagram.

Refer to the following excerpt:

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]

:question: Issue severity

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

Reason for disagreement: [replace this with your reason]