Open nus-se-script opened 2 months ago
This is not explicitly mentioned that you are not allowed to have words next to the multiplicity. Furthermore, this would not lead to confusion in what it represents since this notation is not being re-used anywhere else for other purposes. Below is a screenshot showing how our notation is valid since it does not contradict other notations.
/var/folders/tg/yrcwyrrj2_qc1tf0swypgctw0000gn/T/ru.keepcoder.Telegram/telegram-cloud-photo-size-5-6138804929951809285-y.jpg
Furthermore, in Week 4 Quiz, our UML diagram is highly similar to the one shown in the quiz, which has the note next to the multiplicity.
/var/folders/tg/yrcwyrrj2_qc1tf0swypgctw0000gn/T/ru.keepcoder.Telegram/telegram-cloud-photo-size-5-6138804929951809398-y.jpg
In our case, the note enhances our diagram as it makes our diagram clearer in terms of explaining how the uniquePersonLists have all the persons while the modelManager contains the filtered contacts list.
/var/folders/tg/yrcwyrrj2_qc1tf0swypgctw0000gn/T/ru.keepcoder.Telegram/telegram-cloud-photo-size-5-6138804929951809364-y.jpg
--
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.
Page 11 of the DG.
Expected: Multiplicity should be shown using only a single
*
symbol.all
andfiltered
are additional notes that should not be shown in the multiplicity field. Use UML notes instead.[original: nus-cs2103-AY2324S2/pe-interim#2789] [original labels: severity.Low type.DocumentationBug]