Mnsd05 / pe

0 stars 0 forks source link

Missing methods in class Diagram #15

Open Mnsd05 opened 1 year ago

Mnsd05 commented 1 year ago

There are some methods missing in the class diagram and the authors noted that "Not all methods are included here". I am not sure if it is allowed or not but in my opinion, it is more clear to give all properties and methods

nus-pe-script commented 1 year ago

Team's Response

Which class diagram is the tester referring to? There are 2 class diagrams that uses "Not all methods are included here".

The team believes that class diagram are subjective as taught and that not every information is required to reduce clutter. This is especially for the Ui class where alot of the methods are similar where they are prints for the user.

Ui Class Diagram is done by shawntangy. The other Diagram is done by pyongjie.

Items for the Tester to Verify

:question: Issue response

Team chose [response.IssueUnclear]

Reason for disagreement: [replace this with your explanation]