search

imanamirshah / pe

0 stars 0 forks source link

Confusing class diagram #17

Open imanamirshah opened 2 months ago

imanamirshah commented 2 months ago

image.png

It says here that this is a class diagram for a method(?). I assume that this is the class diagram for Parser class as mentioned. Also, no clear arrowhead on the label to show direction its supposed to be read. Are we to assume that it is in the same direction as the navigability arrowhead? Additionally, it seems that its a class diagram for Parser class but the Parser class itself is missing from the diagram.

Overall, the diagram is confusing, I do not know what it is trying to show exactly. Does not fulfil its purpose, hence, high severity.

nus-pe-script commented 2 months ago

Your response not required for this bug as the team has accepted the bug as it is.