Skip to content

Commit 91804a8

Browse files
author
kohanman
committed
Sprint 1 coursework 3.1: change const to let
1 parent 52c1bf8 commit 91804a8

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

Sprint-1/3-mandatory-interpret/1-percentage-change.js

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ let carPrice = "10,000";
22
let priceAfterOneYear = "8,543";
33

44
carPrice = Number(carPrice.replaceAll(",", ""));
5-
priceAfterOneYear = Number(priceAfterOneYear.replaceAll("," ""));
5+
priceAfterOneYear = Number(priceAfterOneYear.replaceAll(",", ""));
66

77
const priceDifference = carPrice - priceAfterOneYear;
88
const percentageChange = (priceDifference / carPrice) * 100;
@@ -12,11 +12,17 @@ console.log(`The percentage change is ${percentageChange}`);
1212
// Read the code and then answer the questions below
1313

1414
// a) How many function calls are there in this file? Write down all the lines where a function call is made
15+
// lines 4,5 replaceAll() function is called , and then Number() function is called, and then console.log once on line 10
16+
// so total 5 function calls
1517

1618
// 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?
19+
// error is in line 5, because of a missing , in replaceAll("," ""), which I will add to fix the problem.
1720

1821
// c) Identify all the lines that are variable reassignment statements
22+
//lines 4 and 5 are variable reassignment statements
1923

2024
// d) Identify all the lines that are variable declarations
25+
//lines 1 and 2, as well as 7 and 8 are variable declarations
2126

2227
// e) Describe what the expression Number(carPrice.replaceAll(",","")) is doing - what is the purpose of this expression?
28+
//.replaceAll() is removing the , in carPrice and Number() is turning it from a string into a number

0 commit comments

Comments
 (0)