London | 25-ITP-Sep| Sophia Mohamed | Sprint 1 | coursework/sprint-1

[saff-coder]

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

In 1-key exercise, I completed short JavaScript programs and tasks.
2- mandatory -errors exercises are complete. I ran the code in Node.js and wrote the reason for each error
3-mandatory -interpret exercises are complete
4-Completed all stretch challenge requirements

Questions

Thank you, there is no question

[LonMcGregor]

Good start on this sprint's tasks, I have spotted one file where you could improve answers further

[LonMcGregor]

can you think of how to write this? e.g. let ? = ...

[saff-coder]

I have updated the code, changed the variable name and made it clearer

[LonMcGregor]

Do you think the new variable name "movieDuration" is different enough to the first "movieLength" variable? If you were looking at this code for the first time, would you know just by looking at the name what the variable "movieDuration" actually is for?

(Hint: Your original answer, identifying that it comes in hours minutes and seconds is a good observation)

[saff-coder]

That's true, they feel too similar. Maybe I would change it to formattedDuration, it is clear, and this is the final, formatted time string (hh:mm:ss)

[LonMcGregor]

Great, you're finished with this sprint now