corrected issues raised from run 1

Author: Mugs3

Sprint-3/1-key-errors/0.js

@@ -10,7 +10,10 @@ function capitalise(str) {
 }
 console.log(capitalise("beautiful"));
 
-// function is assigned 2 arguments/values=============> write your explanation here
+// write your explanation here:
+// Syntax Error: Identifier 'Str' has been already been declared/assigned
+// The error means Str had already been identified in the function capitalise(str). Let str is assigning a second value which is causing that error message
+
 // =============> write your new code here
 
 function capitalise(str) {

Sprint-3/1-key-errors/1.js

@@ -1,7 +1,8 @@
 // Predict and explain first...
 
 // Why will an error occur when this program runs?
-// decimalNumber is called twice=============> write your prediction here
+// write your prediction here
+// This is because the variable name for the parameter and the cont is the same
 
 // Try playing computer with the example to work out what is going on
 
@@ -14,8 +15,9 @@ function convertToPercentage(decimalNumber) {
 
 console.log(decimalNumber);
 
-// decimalNumber has been declared as a function and a variable within function scope=============> write your explanation here
-//Removing const decimalNumber still returns Syntax error if still within the function scope so it's not restricted to the function
+//=============> write your explanation here
+//The decimalNumber is declared twice as a result, calling convertToPercentage function and passing a value gets assigned to parameter decimalNumber
+//So i had to move it outside of the function scope to remove restrictions
 
 // Finally, correct the code to fix the problem.
 // =============> write your new code here

Sprint-3/1-key-errors/2.js

@@ -11,6 +11,9 @@
 // Syntax errorError Unexpected number.=============> write the error message here
 
 // Means Function identifier should be a name not a number =============> explain this error message here.
+// Whats the difference between function identifier and parameter?
+// I believe function and identifier and parameter can be used interchangeably. So Parsing number 3 into the parameter would throw an error.
+// Changing number 3 to 'num' as a placeholder fixes the error
 
 // Finally, correct the code to fix the problem
 

Sprint-3/2-mandatory-debug/0.js

@@ -10,7 +10,9 @@ console.log(`The result of multiplying 10 and 32 is ${multiply(10, 32)}`);
 
 // =============> write your explanation here
 //The 2nd console log returned value value is undefined. I have added a return function to output the multiply value.
-// Finally, correct the code to fix the problem
+//The value returned by the function was undefined because the value hasn't been stored anywhere, and there's no return to return the value to the function call.
+
+//Finally, correct the code to fix the problem
 //  =============> write your new code here
 function multiply(a, b) {
   console.log(a * b);

Sprint-3/2-mandatory-debug/1.js

@@ -9,7 +9,9 @@ function sum(a, b) {
 
 console.log(`The sum of 10 and 32 is ${sum(10, 32)}`);
 
-// Maybe that unassigned return; may cause the code not to run fully =============> write your explanation here
+// =============> write your explanation here
+// The return statement was placed before the expression hence the undefined outcome. The parsed argument is not receiving output.
+
 // Finally, correct the code to fix the problem
 //  =============> write your new code here
 function sum(a, b) {

Sprint-3/3-mandatory-implement/1-bmi.js

@@ -19,7 +19,9 @@ function calculateBMI(weight, height) {
 }
 
 function calculateBMI(weight, height) {
-  return (weight / height).toFixed(1);
+  // return (weight / height).toFixed(1); - 1st attempt
+  return (weight / (height * height)).toString().slice(0, 4);
 }
+
 console.log(`the bmi of 70 and 2.99 is ${calculateBMI(70, 2.99)}`);
 console.log(`the bmi of 90 and 1.96 is ${calculateBMI(90, 1.96)}`);

Sprint-3/3-mandatory-implement/2-cases.js

@@ -14,10 +14,15 @@
 // You will need to come up with an appropriate name for the function
 // Use the MDN string documentation to help you find a solution
 // This might help https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/toUpperCase
+//
 
 function toUpperSnakeCase(text) {
-  return text.trim().replace(/\s+/g, "_").toUpperCase();
+  // return text.trim().replace(/\s+/g, "_").toUpperCase(); - first attempt that inserted underscore
+  return text.toUpperCase().replaceAll(" ", "_");
 }
 
-const result = toUpperSnakeCase("Hello there");
-console.log(result);
+const text = toUpperSnakeCase("Hello there");
+console.log(text);
+
+// Trim and regex pattern was the only saving grace I came across to resolve the underscore issue I had spent hours researching.
+// I have however now modified and replaced the pattern for easy understanding with AI's help

Sprint-3/3-mandatory-implement/3-to-pounds.js

@@ -25,7 +25,7 @@ const pence = paddedPenceNumberString
 console.log(`£${pounds}.${pence}`);*/
 
 // Revised code:
-function poundsToPence(penceString) {
+function toPounds(penceString) {
   const penceStringWithoutTrailingP = penceString.substring(
     0,
     penceString.length - 1,
@@ -43,6 +43,8 @@ function poundsToPence(penceString) {
 
   return `£${pounds}.${pence}`;
 }
-console.log(poundsToPence("499p"));
-console.log(poundsToPence("1009p"));
-console.log(poundsToPence("2999p"));
+console.log(toPounds("499p"));
+console.log(toPounds("1009p"));
+console.log(toPounds("2999p"));
+
+// updated function name

Sprint-3/4-mandatory-interpret/time-format.js

@@ -32,7 +32,7 @@ function formatTimeDisplay(seconds) {
 
 // c) What is the return value of pad when it is called for the first time?
 // =============> write your answer here
-0;
+//00;
 
 // d) What is the value assigned to num when pad is called for the last time in this program?  Explain your answer
 // =============> write your answer here