LJ-37 / pe

0 stars 0 forks source link

Class diagram mistake #13

Open LJ-37 opened 3 years ago

LJ-37 commented 3 years ago

Class diagram separates attributes into different compartments. They should be in the same compartment. image.png

nus-pe-bot commented 3 years ago

Team's Response

Screenshot 2021-04-20 at 12.30.44 PM.png

I understand that attributes should not be split into compartments. However, i did not mean to split the attributes. This is evident of the absence of black lines between them. The white lines are just a graphical glitch when editing the diagram.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: This should not be a rejected bug as this poor choice of graphics can be easily misread by many to think that these class diagrams does not follow the standards in this module. There is indeed clear white line between the attributes which is a clear sign of splitting into compartments. More considerations should have been done when drawing the class diagram.


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]