search

Halpfrog / pe

0 stars 0 forks source link

Missing step under implemenation #13

Open Halpfrog opened 2 years ago

Halpfrog commented 2 years ago

image.png

Implementation step 2 jumps from logicmanager straight to editRoleCommandParser and ignores CommandListParser. Implementation is meant to make the reader understand how your software deals with this specific user input. By skipping steps but showing them in the UML diagram it adds additional confusion to the reader who now has to go through your code to figure out what exactly you meant.

nus-pe-script commented 2 years ago

Team's Response

The text above shows an overview of the implementation of the command. The sequence diagram shows a more detailed version. There is no value in describing every aspect of the implementation in words when it can be more concisely shown ("comprehensible and not comprehensive").

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: [replace this with your explanation]