Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Diagram too complicated #13

Open
Darren159 opened this issue Nov 17, 2023 · 1 comment
Open

Diagram too complicated #13

Darren159 opened this issue Nov 17, 2023 · 1 comment

Comments

@Darren159
Copy link
Owner

Darren159 commented Nov 17, 2023

image.png

@nus-se-bot
Copy link

nus-se-bot commented Nov 21, 2023

Team's Response

Since you did not give any elaboration, we are unsure which details you think are not necessary here. We feel that all the details in this diagram are necessary. We are trying show the internal operations of the logic package, in this case, for the edit command.

  • :Logic object is responsible for taking in user command, and control the overall logic of the programme. This is an important participant.
  • :NetworkBookParser object is responsible for assigning the parsing duty to the correct parser.
  • :EditCommandParser object is responsible for parsing the edit command itself.
  • :EditAction object represents the action of mutating an editable person. This is where we practice polymorphism so that each subclass represents a different kind of action.
  • :EditCommand object represents the command itself.
  • :EditPersonDescriptor object is the mutable version of the Person class.

Each object plays an important role here in explaining the mechanism of the edit command.

Items for the Tester to Verify

❓ Issue response

Team chose [response.IssueUnclear]

  • I disagree

Reason for disagreement: [replace this with your explanation]


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants