在字符串中查找最长的单词 javascript

Question

我的代码可以很好地处理像“The quick brown fox jumped over the lazy dog”这样的字符串。

但不适用于像“谷歌做一个桶滚”这样的字符串。

它说问题是“未定义的 TypeError 不是对象（评估 'Astr[i].length'）”。

 function findLongestWord(str) { var Astr = str.split(" "); var t = Astr[0].length; var Al = Astr.length; var j = 0; for(var i =1; i < t;i++) { if(t < Astr[i].length) { t = Astr[i].length; j = i; } } str = Astr[j]; return str.length; } findLongestWord("Google do a barrel roll");

Answer 1

您的“for”循环中的变量有问题。

如您所见，您拆分数组并获得数组中第一个成员的长度 所以基本上您获得的是第一个字长而不是字数

var Astr = str.split(" ");
var t = Astr[0].length;

在这里您可以看到您使用“t”（第一个字长）作为循环边界。

for(var i =1; i < t;i++)

保持你的代码简单易读，这样它就可以维护了。

function findLongestWord(str) {
   var words = str.split(" ");
   var words_count = words.length;

   var longest_word_length = 0;

    for(var i = 0; i < words_count; i++){
       if(longest_word_length < words[i].length){
           longest_word_length = words[i].length;
       }
    }
    return longest_word_length;
}

findLongestWord("Google do a barrel roll");

请注意，您始终可以为此使用速记函数

function findLongestWord(str) {
  return str.split(' ').reduce(function(longest, cur) {
     return (cur.length > longest.length) ? cur : longest;
  }, '').length;
}

findLongestWord("Google do a barrel roll");

Answer 2

这是改进功能的一种方法：

var str = 'Google do a barrel roll';

function findLongestWord(str) {
  var Astr = str.split(' ');

  if (!Astr.length) {
    throw new Error('findLongestWord(): no words in str');
  }

  var t = Astr[0].length;
  var Al = Astr.length;
  var j = 0;

  for(var i = 0; i < Al; i++)
  {
      if(t < Astr[i].length)
      {
        t = Astr[i].length;
        j = i;
      }
  }
  str = Astr[j];
  return str.length;
}

findLongestWord(str);
//=> 6

你也可以做这样的事情（这更容易理解）：

str.split(' ').reduce(function(longest, cur) {
  return (cur.length > longest.length) ? cur : longest;
}, '');
//=> Google

Answer 3

function findLongestWord(str)
 {var arr=[];
 arr=str.split(' ');
 arr=arr.sort(function(a,b)
      {
 return b.length-a.length;   /*sorting the array in decending order of 
                                        lengths of each word*/
      });
 var st=arr[0];           /* obviously the first element of the array will 
                             have longest length.*/
 return st.length;
    }
findLongestWord("Google do a barrel roll");