search

nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

DG: Parser class's UML diagram seems cluttered #2466

Open nus-se-script opened 4 months ago

nus-se-script commented 4 months ago

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.

Not necessary to include all methods of the parser class in the UML diagram. It is more important to show the interactions with different classes. (But you did that, so alls good, but just take note for the methods in parser class!)

Screenshot 2024-04-19 at 5.25.41 PM.png

[original: nus-cs2113-AY2324S2/pe-interim#2394] [original labels: severity.Low type.DocumentationBug]

ttasnim5 commented 4 months ago

Team's Response

We wanted to show the full functionality of the Parser class with this class diagram, specifically to see exactly how Parser interacts with the other classes through its methods.

Duplicate status (if any):

--