Lucky-Yuan00 / pe

0 stars 0 forks source link

The class diagram is too complex and does not show the ‘+’ and ‘-’ symbols. #7

Open Lucky-Yuan00 opened 1 week ago

Lucky-Yuan00 commented 1 week ago

Description The class diagram is too complex and does not show the ‘+’ and ‘-’ symbols. Perhaps using the correct correspondence symbols would be more convincing?

Evidence image.png

soc-pe-bot commented 4 days ago

Team's Response

Thank you for highlighting the problem of the symbol. This problem arose from our unfamiliarity of using plantUML and not understanding the possible commands to change this. Through more through research, we realised adding "skinparam classAttributeIconSize 0" could remove this unusual notation differently. We could have added a table to show the conversion in hindsight. However, we disagree that this issue should be a medium severity as it should be under the category of notation variations and the accuracy of the diagram remains. Complexity of the diagram has already been simplified by emptying other less important classes as said in the notes and we feel any further simplification would severely decrease reader's understanding of our class diagram like removing methods.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]