Skip to content

Commit 8cc6b17

Browse files
committed
Convert price strings to numbers before calculating
1 parent 46da3bd commit 8cc6b17

1 file changed

Lines changed: 13 additions & 3 deletions

File tree

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

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,11 +12,21 @@ 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-
15+
4 function calls
16+
-Number(carPrice.replaceAll(",", ""))
17+
-replaceAll(",", "")
18+
-Number(priceAfterOneYear.replaceAll("," ""))
19+
-replaceAll("," "")
1620
// 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?
17-
21+
-Line 5
22+
-missing ) after argument list
23+
-why: replaceAll() needs two arguments separated by a comma.
24+
-fix: by adding comma replaceAll(",", "")
1825
// c) Identify all the lines that are variable reassignment statements
26+
line 4 and 5
1927

2028
// d) Identify all the lines that are variable declarations
21-
29+
line 1,2,7 and 8
2230
// 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)