Fixes for Sprint 1

Author: Ayogit1

Sprint-1/fix/median.js

@@ -7,27 +7,27 @@
 
 
 function calculateMedian(list) {
-  // chekc it is an rray
+  // Check it is an array
   if (!Array.isArray(list)) return null;
 
-  // pull out numbers
+  // Pull out numbers
   const nums = list.filter(n => typeof n === "number" && !Number.isNaN(n));
 
-  // If no  numbers, return null
+  // If no numbers, return null
   if (nums.length === 0) return null;
 
-  // Sort without changing original array
-  const sorted = [...nums].sort((a, b) => a - b);
+  // MODERN FIX: Use .toSorted() to avoid mutating the original array
+  const sorted = nums.toSorted((a, b) => a - b);
 
   const len = sorted.length;
   const mid = Math.floor(len / 2);
 
-  //  If length is odd, take middle element
+  // If length is odd, take middle element
   if (len % 2 === 1) {
     return sorted[mid];
   }
 
-  // If length is even  take average of two middle elements
+  // If length is even, take average of two middle elements
   return (sorted[mid - 1] + sorted[mid]) / 2;
 }
 

Sprint-1/implement/sum.js

@@ -1,11 +1,19 @@
-sum.js
+
 function sum(arr) {
+    if (!Array.isArray(arr)) return 0;
+
     return arr.reduce((accumulator, currentValue) => {
-        if (typeof currentValue === 'number' && !isNaN(currentValue)) {  // investigate  if the current element is a number and not NaN
-            return accumulator + currentValue;
+        // 1. Convert the value to a Number (handles strings like "10")
+        const numericValue = Number(currentValue);
+
+        // 2. Check if the result is a valid number and not NaN
+        // Note: Number.isNaN is stricter/safer than the global isNaN
+        if (!Number.isNaN(numericValue) && typeof currentValue !== 'boolean' && currentValue !== null) {
+            return accumulator + numericValue;
         }
-        return accumulator;  // If not a number, return current sum
-    }, 0); // This line initialize sum at 0 for  empty arrays
+
+        return accumulator;
+    }, 0);
 }
 
-module.exports = sum;
+module.exports = sum;