London | 26-ITP-Jan | Carlos Abreu | Sprint 1 | Coursework

[carlosyabreu]

Learners, PR Template

Self checklist

• I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
• My changes meet the requirements of the task
• I have tested my changes
• My changes follow the style guide

Changelist

PR for Sprint 1 of Structuring and Testing Data

[cjyuan]

Very detailed explanations and well formatted code.

I only have some questions to challenge your understanding and some suggestions.

[cjyuan]

Operation like count = count + 1 is very common in programming, and there is a programming term describing such operation.

Can you find out what one-word programming term describes the operation on line 3?

[carlosyabreu]

Evening @cjyuan

Absolutely sure.
The operator = is called the assignment operator.
On the right hand side of =, the expression count + 1 is evaluated first.
Since count currently has the value 0, count + 1 results in 1.
Then the result 1 is assigned to the variable count on the left hand side of the operator =.
At this stage the value stored in count becomes 1.
To wrap up the line 3 does is to take the current value of count, add 1 to it, and store the result back into count.

[cjyuan]

I was asking for the programming term for operation like "count = count + 1" or "count++".

[cjyuan]

@JaypeeLan Sorry I didn't add the "Review in progress" label when I started reviewing this PR.

[carlosyabreu]

Hi @cjyuan

After answering your queries in comment box I clicked the button 'resolution' or something similar.
That's the first time I encounter that button in PR and I'm not sure after answering I need to press the button.
After that the comment is hidden.
And 'All checks have failed' still active.
Don't know what section of the code I need to change for the bot to give 'All checks successful'.
Thanks

[carlosyabreu]

Sorry @cjyuan after a few minutes the bot run showing 'All checks have passed' message.
Last message was unnecessary.
Thanks again.

[cjyuan]

All good.