迭代数组时未定义“长度”属性

Question

迭代一个数组以找到最长的字符串。每次我收到错误Cannot read property length of undefined。 Console.log 告诉我正在读取和理解数组的长度以及字符串的长度，所以我不明白哪里有未定义的属性。事实上，我刚刚将程序的原始长度增加了三倍，试图确保每个变量都已定义，但仍然没有好处。任何帮助将不胜感激。

function findLongestWord(str) {
  var longest = 0;
  var array = str.split(" ");
  var arrayL = array.length;
  for (i=0; i<=arrayL; i++) {
    var currentWord = array[i];
    var currentL = currentWord.length;
    if (currentL > longest) {
      currentL = longest;
    };
  };
  return longest;
};

findLongestWord("The quick brown fox jumped over the lazy dog");

编辑：虽然下面的答案确实解决了这个问题，但我也只是想提醒那些以后可能会在谷歌上搜索这个帖子的人，我还必须将我的最终 if statement 从 currentL = longest; 交换为 longest = currentL，因为 @987654327 @ 是我最终返回的内容。

Answer 1

问题在于迭代的界限

for (i=0; i<=arrayL; i++) {

请注意，您将要查找 array[array.length] 执行此操作，并且始终未定义，因为 10 项数组中的第 10 项是 [9]，[10] 未定义。

另外，请在此处使用 var

for (var i=0; i < arrayL; i++) {

Answer 2

您读错了 :-) 长度不是 未定义，而是您尝试获取此属性的对象 - 未定义。
所以 currentWord 是 undefined
因为你有错误的循环条件：i<=arrayL,
当i == arrayL - var currentWord = array[i] 是未定义并且你得到错误。

只需修复它：i < arrayL

Answer 3

与其使用容易出错的手写循环来迭代数组，不如利用 JS 为您提供的函数 - 在本例中为 Array.prototype.map：

function findLongestWord(s) {
    return Math.max.apply(this, s.split(/\s+/).map(function(w) {
        return w.length;
    }));
}

Math.max.apply(this, some_array) 是使用给定数组调用Math.max(a, b, c, d, ...) 的标准 JS 方式，而不是传递未知长度的参数列表。

在 ES2015 中你可以使用更短的：

function findLongestWord(s) {
    return Math.max(...s.split(/\s+/).map((w) => w.length));
}