JavaScript 使用三元运算符减少

Question

I have a piece of code that just works. This code looks for the largest value in the array of numbers. Could someone translate this to simple JavaScript(without ternary) so a novice programmer can understand it?

  const mostVotes = votes.reduce((bestIndex, v, i, arr) => v > arr[bestIndex] ? i : bestIndex, 0);

At first, I was trying to implement Math.max, but I needed the index of the largest value in the array, so I went after reduce and this is what I was trying to do with it.

const mostVotes = votes.reduce((acc, value, i, arr) => {
  if(value > acc) {
    return i
  }
}, 0)

Thanks for the answers, much appreciated! I'm starting to understand this and it's much clearer now. Javascript reduce and ternary together is a nice fit.

Answer 1

Essentially, the code you provided is looping through each element in votes and checking whether it is greater than an element stored at a particular index. This index is stored in the variable bestIndex an is used to mark/keep track of the index which holds the largest element from all elements seen while looping.

In your example, your ternary is checking if a given element is larger than the currently marked biggest element (by doing v > arr[bestIndex] ). If this is the case we then set the index of the current element to be the new position of the largest element (by implicitly returning i ). If this is not the case, we leave the index of the largest element as it is by implicitly returning bestIndex .

You can translate this into a more procedural style of programming by using for loops and if-statements like so:

 let votes = [-4, 10, 100, -3, 40]; let positionOfMax = 0; for(let i = 0; i < votes.length; i++) { if(votes[i] > votes[positionOfMax]) { // v > arr[bestIndex] positionOfMax = i; // ? i (from ternary) } /* Not needed else {posittionOfMax = positionOfMax} // : bestIndex (from ternary) */ } console.log(positionOfMax);

I encourage you to take a look at .reduce() and the documentation on the conditional (ternary) operator . They're both useful and powerful tools which can help speed up your development.

Answer 2

What might be confusing about the original code was the lack of brackets {} .

() => 'test' is the same as () => { return 'test' }

In your case:

(bestIndex, v, i, arr) => v > arr[bestIndex] ? i : bestIndex

(bestIndex, v, i, arr) => {
  return (v > arr[bestIndex] ? i : bestIndex)
}

(bestIndex, v, i, arr) => {
  if(v > arr[bestIndex])
    return i
  else 
    return bestIndex
}

const mostVotes = votes.reduce((bestIndex, v, i, arr) => {
  if(v > arr[bestIndex])
    return i
  else 
    return bestIndex
}, 0);

Answer 3

The below if/else should get you to where you want to be.

if (v > arr[bestIndex]) {
  return i
} else {
  return bestIndex
}