search

Zhengwinter / pe

0 stars 0 forks source link

No link between enumeration and other classes [DG] #15

Open Zhengwinter opened 4 months ago

Zhengwinter commented 4 months ago

image.png

There should be some dependencies between the enumeration for task type and task priority level and the other classes, should they be isolated?

nus-pe-script commented 4 months ago

Team's Response

This is the correct way of showing enums.

https://nus-cs2113-ay2324s2.github.io/website/se-book-adapted/chapters/uml.html#enumerations

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: It would be better if you showed the dependencies between the class "Task" and the enumerations "Task Type" and "TaskPriorityLevel", otherwise the reader may not understand how the enumerations are linked to your classes and where you make use of them.

image.png

We can see an example of enumeration being properly linked in the image above.