London | 26-ITP-May | Yonatan Teklemariam | Sprint 3 | Dead-code

Sprint-3/3-dead-code/exercise-1.js

@@ -5,13 +5,10 @@ let testName = "Jerry";
 const greeting = "hello";
 
 function sayHello(greeting, name) {
-  const greetingStr = greeting + ", " + name + "!";
+  // const greetingStr = greeting + ", " + name + "!"; this line will also be removed because the we're trying to return something else, not the greetingStr and apparently it is a redundant code
   return `${greeting}, ${name}!`;
-  console.log(greetingStr);
+  // console.log(greetingStr); this line will obviously be omitted as it is unreachable code because it is written after return. In fact, anything after return is not reachable and should be get ridden of
 }
 
-testName = "Aman";
-
 const greetingMessage = sayHello(greeting, testName);
-
-console.log(greetingMessage); // 'hello, Aman!'
+console.log(greetingMessage);

Sprint-3/3-dead-code/exercise-2.js

@@ -2,12 +2,13 @@
 // The countAndCapitalisePets function should continue to work for any reasonable input it's given, and you shouldn't modify the pets variable.
 
 const pets = ["parrot", "hamster", "horse", "dog", "hamster", "cat", "hamster"];
-const capitalisedPets = pets.map((pet) => pet.toUpperCase());
+// const capitalisedPets = pets.map((pet) => pet.toUpperCase()); -- is obviously redundant code as it also writtend inside the function countAndCapitalisePets and should be removed
 const petsStartingWithH = pets.filter((pet) => pet[0] === "h");
 
-function logPets(petsArr) {
-  petsArr.forEach((pet) => console.log(pet));
+/* function logPets(petsArr) {
+  petsArr.forEach((pet) => console.log(pet)); -- is also unnecessarily unused code and should also be removed
 }
+  */
 
 function countAndCapitalisePets(petsArr) {
   const petCount = {};