Birmingham | 26-ITP-May | Toluwalase Tiamiyu | Sprint 1 | Coursework by TTiamiyu · Pull Request #1380 · CodeYourFuture/Module-Structuring-and-Testing-Data

TTiamiyu · 2026-06-22T11:11:43Z

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 PR the coursework to test the basics of structuring and testing data was done and committed regularly

…g out the initials

… values in a loop

…thods

…gits

…or identification in percentage change calculation

…, and calculations in time formatting

…e pence to pounds conversion process

…s in Chrome

Gopi-2001 · 2026-06-25T10:08:10Z

 // Line 1 is a variable declaration, creating the count variable with an initial value of 0
 // Describe what line 3 is doing, in particular focus on what = is doing
+
+The = served as the executer of the assignment operation.


Suggestion: In JavaScript a single-line comment starts with // and is ignored by the JavaScript engine. Can you look where all can you use it.

Thank you for your suggestion, i have effected the comment tag to my response.

Gopi-2001 · 2026-06-25T10:10:07Z

 // Line 1 is a variable declaration, creating the count variable with an initial value of 0
 // Describe what line 3 is doing, in particular focus on what = is doing
+
+The = served as the executer of the assignment operation.


What will be the new value of count after statement at line 3 is executed ?

The new count value would would be '0 + 1' = 1.

Gopi-2001 · 2026-06-25T11:03:53Z

+
+// Export the initials for testing instead of returning at top-level
+module.exports = getInitials;
+


You have properly use function and the output is the expected string "CKJ", but instead of using a function, can you try to get the same result by Declare a variable called initials

There are a couple of ways to declare initials using the 'let' statement, i have used one of them in my recent commit.

Gopi-2001 · 2026-06-25T11:17:04Z

-const dir = ;
-const ext = ;
+const dir = ; /Users/mitch/cyf/Module-JS1/week-1/interpret/
+const ext = ; file.txt;


Is the semicolon position correct while declaring variable dir and ext ?

I have taking the semicolon out as its not required when '=' is in use.

Gopi-2001 · 2026-06-25T11:17:42Z

 // Create a variable to store the ext part of the variable

-const dir = ;
-const ext = ;


Think of ways to use slice method extract and store directory part and extension part in the variable dir and ext respectively.

Thank you for your review, i have infused the .slice method and i have used the console.log to recall the answer to 'dir' and 'ext', rather than manually inputting it.

Gopi-2001 · 2026-06-25T11:29:38Z

+//This is just an instruction for the first activity - but it is just for human consumption
+//We don't want the computer to run these 2 lines - how can we solve this problem?
+
+To prevent the computer from executing these lines of code, you can comment them out, 


How will you use multi-line comment for line 4 and 5 ?

i made use of the " /* " to comment out line 4 and 5.

Gopi-2001 · 2026-06-25T11:37:13Z

 // Then try updating the expression last4Digits is assigned to, in order to get the correct value
+
+PREDICTION:
+//the code wont work because the card number isnt in ("") and the computer wont be able to apply the .slice due to this.


It is good practice to use single line comment or multiline comment whenever required.

try looking for lines which needs to be ignored by computer in the code.

I see what you mean and i have commented out the headings i created earlier.

Gopi-2001 · 2026-06-25T11:37:54Z

+FIX:
+const cardnumber = "4533787178994213";
+const last4Digits = cardnumber.slice(-4);
+console.log(last4Digits); 


Is there any function that you can use to convert number to string in JS ?

I can use String(num) or num.toString() after i have used a "let" declare statement preceding the use of the stated function.

Gopi-2001 · 2026-06-25T11:43:54Z


 // b) Run the code and identify the line where the error is coming from - why is this error occurring? How can you fix this problem?
+    //The error is in line 5, the error is occurring because there was a comma missing here; ("," "").



Instead of writing comma is missing here, we can put it like "Comma is missing "," and "" for priceAfterOneYear.replaceAll("," "") function call.

Your suggestion is better and i have incorporated it, thank you.

Gopi-2001 · 2026-06-25T11:52:42Z

+//It will do the math perfectly for most normal positive numbers, but there are a few situations where the code will act wierdly.
+//1. Visual problems
+//2. Negative numbers
+//3.Decimal numbers


I am not sure about Visual Problems, can you describe what you mean by it.

We can also consider any Non-Integer numbers .

What i mean with this is The "Single Digit" Visual Bug where the output would not look like a normal digital clock.

I was hoping the non integer numbers would be captured under the decimal numbers response i provided

…ation

…lues

…eAfterOneYear.replaceAll function call

TTiamiyu added 14 commits June 13, 2026 02:42

added a comment explaining the role of the = .

704f58f

Implemented the function to get the initials as output without writin…

dc17501

…g out the initials

Add dir and ext variable assignments in 3-paths.js

dce53b5

added a "for" statement, for random number generation to log multiple…

cecb6ba

… values in a loop

Added comments to explain code execution prevention and commenting me…

bbfef51

…thods

Change age variable declaration from const to let for reassignment

274fe16

Fix console log placement to correctly print city of birth

63265cb

Fix card number assignment to string for correct slicing of last 4 di…

e6db536

…gits

Fix variable naming for twelve and twenty-four hour clock time

1c5023d

Add comments to clarify function calls, variable assignments, and err…

b06aae3

…or identification in percentage change calculation

Enhance comments for clarity on variable declarations, function calls…

496d1b8

…, and calculations in time formatting

Enhance comments to provide a detailed step-by-step explanation of th…

32d7119

…e pence to pounds conversion process

Add results section to document outcomes of alert and prompt function…

5e06e8c

…s in Chrome

Add outputs and answers for console exploration activity

7503415

TTiamiyu added 📅 Sprint 1 Assigned during Sprint 1 of this module Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Jun 22, 2026

Gopi-2001 added the Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. label Jun 25, 2026

Gopi-2001 self-assigned this Jun 25, 2026

Gopi-2001 reviewed Jun 25, 2026

View reviewed changes

Gopi-2001 removed the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Jun 25, 2026

TTiamiyu added 7 commits June 26, 2026 08:02

Added '//' to commenting out my response.

ebaef49

Declared initials with 'let' statement inside function.

6afaba2

Fix dir and ext variable assignments in file path extraction

ebbbb71

Update comment formatting to use multi-line comment style for clarity

23af249

Comment out original code and predictions for clarity in error explan…

ca8a1e2

…ation

Update comments to clarify potential issues with movieLength input va…

8977bb2

…lues

Clarify error explanation in comments regarding missing comma in pric…

aa69c9f

…eAfterOneYear.replaceAll function call

TTiamiyu added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Jun 26, 2026


		// Export the initials for testing instead of returning at top-level
		module.exports = getInitials;


		// b) Run the code and identify the line where the error is coming from - why is this error occurring? How can you fix this problem?
		//The error is in line 5, the error is occurring because there was a comma missing here; ("," "").

Uh oh!

Uh oh!

Conversation

TTiamiyu commented Jun 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Learners, PR Template

Changelist

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

TTiamiyu Jun 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

TTiamiyu commented Jun 22, 2026 •

edited

Loading

TTiamiyu Jun 26, 2026 •

edited

Loading