WM71811 / pe

0 stars 0 forks source link

Macro logic diagram problem #18

Open WM71811 opened 3 years ago

WM71811 commented 3 years ago

I think the two loops should be combined? Since only after first loop the command object parsed is then used for second loop Also the condition being "for each macro line" may be could be rephrased Screenshot 2020-11-13 at 5.37.28 PM.png

nus-se-bot commented 3 years ago

Team's Response

The two different loops are intentional here. In the code there are actually 2 different loops, the first executes the parsing all at once, then the second executes the commands one by one. This is so we can catch any parse errors in the first loop, before running the second loop.

The paragraph right above describes these two seperate steps:

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]