search

papataco14 / pe

0 stars 0 forks source link

Interfaces shown in sequence diagram #14

Open papataco14 opened 1 year ago

papataco14 commented 1 year ago

This is part of the add sequence diagram in the DG.

image.png

It seems from your codebase, Model is an interface. However, including it in the sequence diagram as :Model adds more confusion, as it is convention to show only the objects involved. Perhaps it would be better to show :ModelManager here instead?

nus-pe-bot commented 1 year ago

Team's Response

Valid Point. However, Model Manager is also a Model. Thus, our diagram is not wrong.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]