anqi20 / pe

0 stars 0 forks source link

Lack of Sequence Diagram #17

Open anqi20 opened 3 years ago

anqi20 commented 3 years ago

image.png

image.png

I feel like there is a lack of sequence diagrams here.

It feels like an excessive usage of code as well. Moreover, the code snippet was also used repeatedly.

nus-pe-bot commented 3 years ago

Team's Response

The interactions between the Ui and the parser as described are shown in the sequence diagram at the top of section 3.1. As the diagram that would be used is repeated, we did not post the same diagram twice. The paragraph highlighted is a more in depth explanation on the processes and datatypes involved from the perspective of the class Ui & Command while the explanation in the Parser section covers the first half of the diagram (conversion to userInput)

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]