search

nus-cs2113-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Clumped class diagram #396

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years 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.

Unnecessarily cluntered and hard to read. like what was mentioned in the PE-D report by our prof, things like visibility etc are not needed to be included.

Capture.PNG

[original: nus-cs2113-AY1920S2/pe-interim#386]

btricec commented 4 years ago

Team's Response

Agree that the diagram is a little crowded, however, can visibility really be omitted? Private methods and attributes were already excluded to reduce clutter.

Duplicate status (if any):

--