search

nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Alt frame in UML for calorie #2093

Open nus-se-script opened 2 months ago

nus-se-script commented 2 months 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.

Use of alt frame but there is no "else". I think you meant "opt"

Screenshot 2024-04-19 at 5.24.18 PM.png

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

rexyyong commented 2 months ago

Team's Response

Hi there, thank you for the bug report!

Our team recognizes that opt can be used here also. However, we feel that we are not violating the rules by using alt here as it is not clearly mentioned that when only one condition is present, alt cannot be used.

Semantically, when there is only one condition, alt and opt mean the same thing.

As such, we have chosen not in scope label for this issue as we believe that it is a valid concern but we should not be penalized for this.

image.png

Thank you!

Duplicate status (if any):

--