在FOR循环中放置0会返回不需要的结果

Question

我开始学习JavaScript，到目前为止享受它。 我有这个循环（目的是罚款数组中的最高数字）：

function max(arr) {
  var max = arr[0]; 
  for (var i = 0; i < arr.length; i++) {
    if (arr[i] > max) {
      max = arr[i];
    }
    return max;
  }
}

但它不起作用，除非我将i改为i = 1.有人能解释原因吗？谢谢！

Answer 1

问题

在您的实施尝试中，您有一个逻辑错误：

function max(arr) {
  var max = arr[0]; 
  for (var i = 0; i < arr.length; i++) {
    if (arr[i] > max) {
      max = arr[i];
    }
    // return max; <-- this always returns arr[0]
  }
  return max; // <-- it should be here to work as expected
}

另一项改进是从var i = 1开始，因为您在开始for循环之前已经将arr[0]的初始值分配给max。

但下面是一个更好，更强大的实现。

解决方案

从技术上讲，您应该这样做是为了说明空数组，具有负值的数组，格式错误的值或可以对有效数字强制执行的值：

function myMax(array) {
  var max = -Infinity

  for (var index = 0, value; index < array.length; index++) {
    // coerce element to number
    value = Number(array[index])
    // short-circuit for malformed value
    if (isNaN(value)) return value
    if (value > max) max = value
  }

  return max
}

匹配Math.max()

的ECMAScript规范

这在功能上等同于所有可能的数组输入的以下实现，空或非空：

function max(array) {
  return Math.max.apply(Math, array)
}

来自documentation for Math.max() on MDN：

  

如果没有给出参数，则结果为-Infinity。

     

如果至少有一个参数无法转换为数字，则结果为NaN。

演示

＆＃13;

＆＃13;

function myMax(array) {
  var max = -Infinity

  for (var index = 0, value; index < array.length; index++) {
    // coerce element to number
    value = Number(array[index])
    // short-circuit for malformed value
    if (isNaN(value)) return value
    if (value > max) max = value
  }

  return max
}

function specMax(array) {
  return Math.max.apply(Math, array)
}

function test(input) {
  var set = new Set()
  console.log('input:', JSON.stringify(input))
  set.add(myMax(input))
  console.log('output:', myMax(input), '[myMax(input)', set.has(specMax(input)) ? 'matches' : 'does not match', 'specMax(input)]')
}

test([-50, 25, 0, 50, -25])
test([-100, -80, -90, -85, -95])
test([])
test([0, 'malformed', 100, 20])
test([1, 2, 3, 4, ' 5.e0  \t\r\n'])
test([{
  toString () { return '200' },
  valueOf () { return 100 },
  toJSON () {
    return {
      "toString()": this.toString(),
      "valueOf()": this.valueOf()
    }
  }
}])

＆＃13;

.as-console-wrapper {
  max-height: 100% !important;
}

＆＃13;

＆＃13;

＆＃13;

Answer 2

function max(arr) {
  var max = 0; 
  for (var i = 0; i < arr.length; i++) {
   if (Math.max(arr[i], max) == arr[i]) {
      max = arr[i];
   }
  }
  return max;

}

如果你在for循环中发出一个返回指令，它就会破坏它。

Answer 3

移动返回是必要的。但是当i = 1时，你永远不会看到数组的第一个元素。

function max(arr) {
   var max; 
   for (var i = 0; i < arr.length; i++) {
      if ((max !== undefined) && (arr[i] > max)) {
         max = arr[i];
      }
   }
   return max;
}

如果arr为空，这甚至会返回未定义但可以说是正确的undefined值。