search

btaanish / pe

0 stars 0 forks source link

Parameters shown as their contents #13

Open btaanish opened 11 months ago

btaanish commented 11 months ago

image.png

All parameters to methods here shown as theri contents, perhaps would be more intuitive to show the paramters with the type that each of these methods are passed.

nus-pe-bot commented 11 months ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

Thanks for your suggestion! However, it is acceptable to both pass the parameter of functions in a sequence diagram as its contents or as its type. This is not a bug as we simply pass parameters to methods as its contents as a simplification. We had also clarified on prof regarding the representation of method parameters in the issue below. Hence we reject this as a bug.

https://github.com/nus-cs2103-AY2324S1/forum/issues/455

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]