updated exercise-2 and exercise-3

Author: TzeMingHo

Sprint-1/destructuring/exercise-2/exercise.js

@@ -73,16 +73,18 @@ let hogwarts = [
 
 
 function gryffindorHouse(hogwarts) {
-  return hogwarts.filter(({house}) => house === 'Gryffindor').map(({firstName, lastName}) => `${firstName} ${lastName}`).join("\n")
+  return hogwarts
+    .filter(({house}) => house === 'Gryffindor')
+    .map(({firstName, lastName}) => `${firstName} ${lastName}`)
+    .join("\n")
 }
 
 function teachersWithPets(hogwarts) {
-  return hogwarts.reduce((nameList, {firstName, lastName, pet, occupation}) => {
-    if (occupation === "Teacher" && pet !== null) {
-      nameList.push(`${firstName} ${lastName}`)
-    }
-    return nameList;
-  }, []).join("\n")
+  return hogwarts
+    .filter(({occupation, pet}) => occupation === "Teacher" && pet !== null)
+    .map(({firstName, lastName}) => `${firstName} ${lastName}`)
+    .join("\n")
 }
 
 console.log(gryffindorHouse(hogwarts));
+console.log(teachersWithPets(hogwarts))

Sprint-1/destructuring/exercise-3/exercise.js

@@ -14,6 +14,9 @@ function receiptAndCosts(order) {
   let receipt = []
 
   function orderLineFormatting(...args) {
+    if (args.length !== 3) {
+      throw new Error("expecting 3 arguments in order of quantity, itemName, and subTotal")
+    }
     const orderLineSpacing = [8, 20, 5]
     let formattedOrderLine = ''
     for (let index = 0; index < args.length; index++) {