Skip to content

Commit 73113c5

Browse files
committed
Revert "Delete Sprint-2/3-mandatory-interpret/1-percentage-change.js"
This reverts commit add8d2e.
1 parent add8d2e commit 73113c5

1 file changed

Lines changed: 32 additions & 0 deletions

File tree

Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,32 @@
1+
let carPrice = "10,000";
2+
let priceAfterOneYear = "8,543";
3+
4+
carPrice = Number(carPrice.replaceAll(",", ""));
5+
priceAfterOneYear = Number(priceAfterOneYear.replaceAll("," ""));
6+
7+
const priceDifference = carPrice - priceAfterOneYear;
8+
const percentageChange = (priceDifference / carPrice) * 100;
9+
10+
console.log(`The percentage change is ${percentageChange}`);
11+
12+
// Read the code and then answer the questions below
13+
14+
// a) How many function calls are there in this file? Write down all the lines where a function call is made
15+
4 function calls
16+
-Number(carPrice.replaceAll(",", ""))
17+
-replaceAll(",", "")
18+
-Number(priceAfterOneYear.replaceAll("," ""))
19+
-replaceAll("," "")
20+
// 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?
21+
-Line 5
22+
-missing ) after argument list
23+
-why: replaceAll() needs two arguments separated by a comma.
24+
-fix: by adding comma replaceAll(",", "")
25+
// c) Identify all the lines that are variable reassignment statements
26+
line 4 and 5
27+
28+
// d) Identify all the lines that are variable declarations
29+
line 1,2,7 and 8
30+
// e) Describe what the expression Number(carPrice.replaceAll(",","")) is doing - what is the purpose of this expression?
31+
-First removing the comma "10,000"
32+
-Second convert string into number 10000

0 commit comments

Comments
 (0)