Skip to content

Commit 8b2dbc2

Browse files
committed
fix the error by putting , in the replaceall() and answer the questions
1 parent ceffa2d commit 8b2dbc2

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

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

Lines changed: 6 additions & 3 deletions
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,14 @@ 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+
// There are two function calls. line 4 and 5
1616
// 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-
17+
//The error iscoming from the line 5, because there was no comma in the replaceAll method .
1818
// c) Identify all the lines that are variable reassignment statements
19+
// line 4 and 5 are the variable reassignment statments.
1920

2021
// d) Identify all the lines that are variable declarations
22+
// line 1 ,2 ,7 and 8 are lines where variable declarions happened
2123

2224
// e) Describe what the expression Number(carPrice.replaceAll(",","")) is doing - what is the purpose of this expression?
25+
// the method .replaceAll is replacing the , in the 10,000 and throws it 10000, the number function will convert it into a number.

0 commit comments

Comments
 (0)