search

nigel27022001 / pe

0 stars 0 forks source link

Multiplicity of Model Diagram is not shown/incorrect #11

Open nigel27022001 opened 5 months ago

nigel27022001 commented 5 months ago

image.png

As seen in the user guide, the email , role , course are non-optional, which means these classes should have the multiplicity of one as every person should have an address, email, course.

nus-pe-bot commented 5 months ago

Team's Response

Unspecified means unspecified https://github.com/nus-cs2103-AY2324S2/forum/issues/173 . It does not mean optional.

As such, the current notation is not incorrect.

Nevertheless, we agree that adding 1 could improve the documentation.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: Prof said that "When multiplicity is not given, it means unknown/unspecified (i.e., there is no 'default')"

Which means that this is when the multiplicity is not known, however in this case, the developing team knows that the email is non-optional, which has a multiplicity of "1...N", which is not the same as the multiplicity being unknown.