search

lokidoki102 / pe

0 stars 0 forks source link

Repetitive sequence diagram #12

Open lokidoki102 opened 2 months ago

lokidoki102 commented 2 months ago

Both Add command sequence diagram and Edit command sequence diagram are very identical to each other, with only the difference of the command input.

Edit: image.png

Add: image.png

However, there are some difference in the implementation of add command and edit command in the codes.

For example, edit command would call 

        model.setPerson(personToEdit, editedPerson);
        model.updateFilteredPersonList(PREDICATE_SHOW_ALL_PERSONS);

in the EditCommand.java.

I think this is what sets a part edit command from add command and should be reflected in the sequence diagram.

nus-pe-bot commented 2 months ago

Team's Response

We are not showing the difference between AddCommand and EditCommand, but rather we are just showing the sequence diagram for edit and add individually and separately

And the two lines you included: model.setPerson(personToEdit, editedPerson); model.updateFilteredPersonList(PREDICATE_SHOW_ALL_PERSONS);

are not meant to be in the sequence diagram.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]