Fixed and updated files in Sprint-2

Author: Konvaly

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

@@ -12,8 +12,7 @@
 // So, the error sounds "SyntaxError: Identifier 'str' has already been declared", it means that we can't declare the sane variable name twice
 // =============> write your new code here
 function capitalise(str) {
-  let capitalisedStr = `${str[0].toUpperCase()}${str.slice(1)}`;
-  return capitalisedStr;
+  return `${str[0].toUpperCase()}${str.slice(1)}`;
 }
 
 console.log(capitalise("greetings")); // Output: "Greetings"

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

@@ -24,9 +24,7 @@
 // Finally, correct the code to fix the problem
 // =============> write your new code here
 function convertToPercentage(decimalNumber) {
-  const percentage = `${decimalNumber * 100}%`;
-
-  return percentage;
+  return `${decimalNumber * 100}%`;
 }
 
 console.log(convertToPercentage(0.5));

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

@@ -2,16 +2,17 @@
 
 // =============> I think the result will be undefined because in the function there is no return statement.
 
-function multiply(a, b) {
-  console.log(a * b);
-}
+// function multiply(a, b) {
+//   console.log(a * b);
+// }
 
 // console.log(`The result of multiplying 10 and 32 is ${multiply(10, 32)}`);
 
 // =============> Its interesting - function log the result  but doesn't return it. So when we see the output 320 and "undefined"
 
 // Finally, correct the code to fix the problem
-//  =============> function multiply(a, b) {
-//                   return a * b;
-//                 }
-//                 console.log(`The result of multiplying 10 and 32 is ${multiply(10, 32)}`);
+function multiply(a, b) {
+  return a * b;
+}
+
+console.log(`The result of multiplying 10 and 32 is ${multiply(10, 32)}`);

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

@@ -1,15 +1,17 @@
 // Predict and explain first...
 //  =============> I guess that the output will be undefined because there is a semicolon after the "return" and this breaks the line of code, and after return statement nothing can be returned
 
-function sum(a, b) {
-  return;
-  a + b;
-}
+// function sum(a, b) {
+//   return;
+//   a + b;
+// }
 
-console.log(`The sum of 10 and 32 is ${sum(10, 32)}`);
+// console.log(`The sum of 10 and 32 is ${sum(10, 32)}`);
 
 // =============> The output is "The sum of 10 and 32 is undefined" as I predicted above
 // Finally, correct the code to fix the problem
-//  =============> function sum(a, b) {
-//                   return a + b;
-//                 }
+function sum(a, b) {
+  return a + b;
+}
+
+console.log(`The sum of 10 and 32 is ${sum(10, 32)}`);

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

@@ -9,13 +9,13 @@
 
 // const num = 103;
 
-function getLastDigit() {
-  return num.toString().slice(-1);
-}
+// function getLastDigit() {
+//   return num.toString().slice(-1);
+// }
 
-console.log(`The last digit of 42 is ${getLastDigit(42)}`);
-console.log(`The last digit of 105 is ${getLastDigit(105)}`);
-console.log(`The last digit of 806 is ${getLastDigit(806)}`);
+// console.log(`The last digit of 42 is ${getLastDigit(42)}`);
+// console.log(`The last digit of 105 is ${getLastDigit(105)}`);
+// console.log(`The last digit of 806 is ${getLastDigit(806)}`);
 
 // Now run the code and compare the output to your prediction
 // =============> The last digit of 42 is 3
@@ -24,13 +24,13 @@ console.log(`The last digit of 806 is ${getLastDigit(806)}`);
 // Explain why the output is the way it is
 // =============> Because we use the global variable "num" in the function instead of passing the parameter to the function.
 // Finally, correct the code to fix the problem
-//=============> const num = 103;
-//               function getLastDigit(num) {
-//                  return num.toString().slice(-1);
-//               }
-//               console.log(`The last digit of 42 is ${getLastDigit(42)}`);
-//               console.log(`The last digit of 105 is ${getLastDigit(105)}`);
-//               console.log(`The last digit of 806 is ${getLastDigit(806)}`);
+
+function getLastDigit(num) {
+  return num.toString().slice(-1);
+}
+console.log(`The last digit of 42 is ${getLastDigit(42)}`);
+console.log(`The last digit of 105 is ${getLastDigit(105)}`);
+console.log(`The last digit of 806 is ${getLastDigit(806)}`);
 //
 // This program should tell the user the last digit of each number.
 //

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

@@ -15,11 +15,11 @@
 // 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 toUpperCaseSnakeString(str) {
+function toUpperSnakeCase(str) {
   transformedStr = str.toUpperCase().replaceAll(" ", "_");
   return transformedStr;
 }
 
-console.log(toUpperCaseSnakeString("hello there")); // "HELLO_THERE"
-console.log(toUpperCaseSnakeString("lord of the rings")); // "LORD_OF_THE_RINGS"
-console.log(toUpperCaseSnakeString("code your future is great")); // "CODE_YOUR_FUTURE_IS_GREAT"
+console.log(toUpperSnakeCase("hello there")); // "HELLO_THERE"
+console.log(toUpperSnakeCase("lord of the rings")); // "LORD_OF_THE_RINGS"
+console.log(toUpperSnakeCase("code your future is great")); // "CODE_YOUR_FUTURE_IS_GREAT"

Sprint-2/5-stretch-extend/format-time.js

@@ -3,11 +3,16 @@
 // Your task is to write tests for as many different groups of input data or edge cases as you can, and fix any bugs you find.
 
 function formatAs12HourClock(time) {
-  const hours = Number(time.slice(0, 2));
-  if (hours > 12) {
-    return `${hours - 12}:00 pm`;
-  }
-  return `${time} am`;
+  const hours24 = Number(time.slice(0, 2));
+  const minutes = time.slice(3).padStart(2, "0");
+  const timeSuffix = hours24 >= 12 ? "pm" : "am";
+
+  let hours12 = hours24 % 12;
+  if (hours12 === 0) hours12 = 12;
+
+  const formattedHours12 = String(hours12).padStart(2, "0");
+
+  return `${formattedHours12}:${minutes} ${timeSuffix}`;
 }
 
 const currentOutput = formatAs12HourClock("08:00");
@@ -23,3 +28,26 @@ console.assert(
   currentOutput2 === targetOutput2,
   `current output: ${currentOutput2}, target output: ${targetOutput2}`
 );
+
+const currentOutput3 = formatAs12HourClock("00:00");
+const targetOutput3 = "12:00 am";
+console.assert(
+  currentOutput3 === targetOutput3,
+  `current output: ${currentOutput3}, target output: ${targetOutput3}`
+);
+
+const currentOutput4 = formatAs12HourClock("12:00");
+const targetOutput4 = "12:00 pm";
+console.assert(
+  currentOutput4 === targetOutput4,
+  `current output: ${currentOutput4}, target output: ${targetOutput4}`
+);
+
+const currentOutput5 = formatAs12HourClock("19:12");
+const targetOutput5 = "07:12 pm";
+console.assert(
+  currentOutput5 === targetOutput5,
+  `current output: ${currentOutput5}, target output: ${targetOutput5}`
+);
+
+// I found that when I added minutes, the output became incorrect because the original function only changed the hours. I fixed this bug by introducing a new variable called minutes and using it as the second part of the output after the hours. I also added some formatting for the minutes and defined the period of time (am/pm) in a variable called timeSuffix to make the output display correctly.