search

nus-cs2113-AY2021S2 / pe-dev-response

0 stars 0 forks source link

Wrong "Sequence diagram for findNearest" #1047

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

image.png

For the above diagram, there are a few problems.

  1. The first arrow should use solid line instead of dotted line
  2. The return dotted line is missing from findNearest to Map, and from Facility to findNearest
  3. Activation bar should only start after the incoming arrow, and terminate immediately after the return arrow. I have circled out instances which this is violated
  4. Return of control to the user at the end of Map is also missing.

[original: nus-cs2113-AY2021S2/pe-interim#1047] [original labels: severity.Medium type.DocumentationBug]

geezzzyyy commented 3 years ago

Team's Response

No response provided.

Duplicate status (if any):

--