Condense arrays with reduce: Better variable naming

Challenge Condense arrays with reduce has an issue. User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.102 Safari/537.36. Please describe how to reproduce this issue, and include links to screenshots if possible.

This is not quite a issue, but i think that if the variable “previousVal” is changed to “total” or “sum” would help others better understand and avoid confusion. As in this case previousVal represents an accumulator.

My code:

var array = [4,5,6,7,8];
var singleVal = 0;

// Only change code below this line.

singleVal = array.reduce(function(total, currentVal){
  return total + currentVal; // previousVal is the accumulator
});

[erictleung]

@sahilkhurana19 @raisedadead how about this?

… To use reduce you pass in a callback whose arguments are an accumulator (in this case, previousVal) and the current value (currentVal).

The accumulator is like a total that reduce keeps track of after each operation. The current value is just the next element in the array you’re iterating through. …

[sahilkhurana19]

@erictleung So I guess it would be better if I reverted back the changes in my PR and add something like In the following example reduce is being used to subtract all the values of an array, note that while previousVal is the accumulator, currentVal on the other hand iterates over the array and modifies the previousVal as per the return statement.