Merge branch 'master' into master

Author: knight7

src/algorithms/sorting/merge-sort/MergeSort.js

@@ -24,36 +24,37 @@ export default class MergeSort extends Sort {
   }
 
   mergeSortedArrays(leftArray, rightArray) {
-    let sortedArray = [];
+    const sortedArray = [];
 
-    // In case if arrays are not of size 1.
-    while (leftArray.length && rightArray.length) {
-      let minimumElement = null;
+    // Use array pointers to exclude old elements after they have been added to the sorted array.
+    let leftIndex = 0;
+    let rightIndex = 0;
 
-      // Find minimum element of two arrays.
-      if (this.comparator.lessThanOrEqual(leftArray[0], rightArray[0])) {
-        minimumElement = leftArray.shift();
+    while (leftIndex < leftArray.length && rightIndex < rightArray.length) {
+      let minElement = null;
+
+      // Find the minimum element between the left and right array.
+      if (this.comparator.lessThanOrEqual(leftArray[leftIndex], rightArray[rightIndex])) {
+        minElement = leftArray[leftIndex];
+        // Increment index pointer to the right
+        leftIndex += 1;
       } else {
-        minimumElement = rightArray.shift();
+        minElement = rightArray[rightIndex];
+        // Increment index pointer to the right
+        rightIndex += 1;
       }
 
-      // Call visiting callback.
-      this.callbacks.visitingCallback(minimumElement);
-
-      // Push the minimum element of two arrays to the sorted array.
-      sortedArray.push(minimumElement);
-    }
+      // Add the minimum element to the sorted array.
+      sortedArray.push(minElement);
 
-    // If one of two array still have elements we need to just concatenate
-    // this element to the sorted array since it is already sorted.
-    if (leftArray.length) {
-      sortedArray = sortedArray.concat(leftArray);
-    }
-
-    if (rightArray.length) {
-      sortedArray = sortedArray.concat(rightArray);
+      // Call visiting callback.
+      this.callbacks.visitingCallback(minElement);
     }
 
-    return sortedArray;
+    // There will be elements remaining from either the left OR the right
+    // Concatenate the remaining elements into the sorted array
+    return sortedArray
+      .concat(leftArray.slice(leftIndex))
+      .concat(rightArray.slice(rightIndex));
   }
 }