Merge branch 'master' of https://github.com/LambdaSchool/JavaScript-I into solution

SunJieMing · SunJieMing · commit 23c8e366b4eb · 2017-10-02T16:55:18.000-07:00
diff --git a/src/arrays.js b/src/arrays.js
@@ -22,12 +22,14 @@ const map = (elements, cb) => {
   return mappedArr;
 };
 
-const reduce = (elements, cb, startingValue = elements.shift()) => {
+const reduce = (elements, cb, startingValue) => {
   // Combine all elements into a single value going from left to right.
-  // Elements will be passed one by one into `cb`.
-  // `memo` is the starting value.  If `memo` is undefined then make `elements[0]` the initial value.
-  let memo = startingValue;
-  each(elements, (item) => {
+  // Elements will be passed one by one into `cb` along with the `startingValue`.
+  // `startingValue` should be the first argument passed to `cb` and the array element should be the second argument.
+  // `startingValue` is the starting value.  If `startingValue` is undefined then make `elements[0]` the initial value.
+  const elementsCopy = elements.slice();
+  let memo = startingValue || elementsCopy.shift();
+  each(elementsCopy, (item) => {
     memo = cb(memo, item);
   });
   return memo;
diff --git a/src/callbacks.js b/src/callbacks.js
@@ -1,6 +1,6 @@
 /* eslint-disable */
 
-/*  For portion of the assignment your job is to write functions
+/*  For a portion of the assignment your job is to write functions
  *  so that each function invocation below works.  You're working backwards.
  *
  *  There are no tests for this file.

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`/* eslint-disable */`
`2`	`2`
`3`		`-/* For portion of the assignment your job is to write functions`
	`3`	`+/* For a portion of the assignment your job is to write functions`
`4`	`4`	`* so that each function invocation below works. You're working backwards.`
`5`	`5`	`*`
`6`	`6`	`* There are no tests for this file.`