search

jemehgoh / pe

0 stars 0 forks source link

Redundant "no matches found" case for alt block in sequence diagram for "Find Command" in DG #12

Open jemehgoh opened 2 weeks ago

jemehgoh commented 2 weeks ago

image.png

In the sequence diagram at the top of the "Find command" section of the developer guide, the two "alt" blocks each have a case "no matches" found in which nothing is done (i.e. no methods executed, no return arrow, etc.)

As per the course convention, an "opt" block should have been used, rather than an "alt" block, if there was no else condition.

nus-se-bot commented 1 week ago

Your response not required for this bug as the team has accepted the bug as it is.