search

matthewgani / pe

0 stars 0 forks source link

Small error in page 19 sequence diagram. #9

Open matthewgani opened 3 years ago

matthewgani commented 3 years ago

If I remember properly, I think the return for isfalse shouldnt be in the midway of the bar. image.png

nus-pe-bot commented 3 years ago

Team's Response

returning the self invocation if isfound is false

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: According to this image.png, the method shouldnt be active after being returned. To me the isFound == false return makes the diagram look confusing especially because of the return of dining options when it shouldnt be returned if its false (if I understood properly).