search

charkty / pe

0 stars 0 forks source link

Should not combine Association Multiplicity with the expanded class diagram form #12

Open charkty opened 9 months ago

charkty commented 9 months ago

The "1" should not be there as it is one or the other. Not supposed to use 2 different types of formatting together.

image.png

nus-pe-script commented 9 months ago

Team's Response

I think it is allowed

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: As seen in the attributes of class NUSDegs, #ui:Ui was used. However, there is a line with association multiplicity pointing from NUSDegs to Ui too. Only one implementation should have been chosen where either

(1) the association multiplicity implementation should be removed with the line

(2) the #ui:Ui attribute should have been removed

image.png