search

nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Wrong multiplicity used in Group Storage UML #548

Open nus-pe-bot opened 7 months ago

nus-pe-bot commented 7 months ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

Since there is no association or references between the Group Storage class to the Group class, there should not be a multiplicity given for the arrows pointing to group.

image.png

[original: nus-cs2113-AY2324S2/pe-interim#623] [original labels: type.DocumentationBug severity.VeryLow]

mukund1403 commented 6 months ago

Team's Response

This is a duplicate of 547 since the issue is caused by the use of association instead of dependency. Therefore, fixing that issue will automatically solve this also (see screenshot part b).

image.png

Moreover, it does not clearly state whether it is incorrect to have multiplicity for dependency in the CS2113 textbook, just that it is redundant.

Duplicate status (if any):

Duplicate of #547