West-Midlands |26-May-ITP |Maryam Janjua |Sprint 1 |Complete Sprint 1 Course Work by maryam-devio · Pull Request #1409 · CodeYourFuture/Module-Structuring-and-Testing-Data

maryam-devio · 2026-06-26T19:23:33Z

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 this task we need to solve JavaScript tasks.
I deeply understand the concept read documentation, solve tasks and tested changes.
Theses exercises helped me to understand java-script syntax, built-in methods and data types briefly.

LonMcGregor

Well done doing the stretch tasks.

There's just one file I have comments about

LonMcGregor · 2026-07-01T12:48:57Z

+*/
 // e) What do you think the variable result represents? Can you think of a better name for this variable?
-
+// The result represents the how long the movie is, in hours, minutes and seconds. It can named as MovieDuration.


Does"MovieDuration" make enough of a distinction to the other variable name called "movieLength"?

Thanks for the feedback. I agree that MovieDuration was too similar to movieLength, so I've renamed it to duration to make the distinction clearer.

LonMcGregor · 2026-07-01T12:49:25Z

+// The result represents the how long the movie is, in hours, minutes and seconds. It can named as MovieDuration.
 // f) Try experimenting with different values of movieLength. Will this code work for all values of movieLength? Explain your answer
+// Yes, I changed the value of movieLength and it worked. The movieLength is the variable used in calculating other 
+// values and to evaluate the total length of movie.


It may work without errors, but are there any possible inputs where the output doesn't look quite right?

Yes, I found that although the code worked without errors, the output was not always formatted correctly. For example, when the hours, minutes, or seconds were less than 10 (e.g. 7 or 5), they were displayed as single digits instead of two digits. I fixed this by adding a helper function that adds a leading zero to single-digit values before formatting the duration.
Thank you for your feedback.

maryam-devio added 5 commits June 26, 2026 10:57

print desired Initials

2d865ae

Solved Exercise Math.floor and Math.random

5e52076

Erros Identified for all file of Sprint-1

9dcebe9

Did Mandatory Intepret tasks on 3 files of sprint 1

aef0c07

Did chrome tasks

d89f407

maryam-devio added 📅 Sprint 2 Assigned during Sprint 2 of this module Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Jun 26, 2026