Javascript Last Letter 被截断

Question

好的，所以基本上这段代码将一个句子传递给函数，函数需要找出哪个单词最长返回。一切都很好，除了最后一个字母不断被切断。那么什么是解决这个问题的好方法呢？

    function LongestWord(sen) { 
sen = sen.toLowerCase();
  var build = "";
  var arr = [];
  var longest = 0;
for(var i = 0; i < sen.length;i++){
  var cur = sen.charCodeAt(i);
  console.log(sen.charCodeAt(i))
  if(i == sen.length - 1){

       arr.push(build);
    }
    if(sen.charAt(i) === " "){
    arr.push(build);
     build = "";
    }
    if(cur >= 97 && cur <= 122){
    build += String.fromCharCode(cur);
    }
  }
  console.log(arr);
  for(var e = 0; e < arr.length - 1;e++){
    if(arr[e].length > arr[e + 1].length){

   longest = arr[e];
    }
    else{
    longest = arr[e + 1];
    }
  }
  return longest; 

}

// keep this function call here 
// to see how to enter arguments in JavaScript scroll down
console.log(LongestWord("Johnny ErsoL"));  

它返回“Johnny”，这是正确的，但这是数组最后的样子。

[ 'johnny', 'erso' ]

Answer 1

这是我的建议？

function LongestWord(sen) {
    return sen.split(/\b/).filter(function(item) {
        return item.trim().length;
    }).sort(function(a,b) {
        return b.length - a.length;
    });
}

在单词边界分割句子，然后剪掉空格，最后按每个单词的长度排序，返回排序后的数组。

Answer 2

尝试替换

for(var i = 0; i < sen.length;i++){
  var cur = sen.charCodeAt(i);
  console.log(sen.charCodeAt(i))
  if(i == sen.length - 1){

       arr.push(build);
    }
    if(sen.charAt(i) === " "){
    arr.push(build);
     build = "";
    }
    if(cur >= 97 && cur <= 122){
    build += String.fromCharCode(cur);
    }
  }

与

arr=sen.split();

Answer 3

您有一个带有e <arr.length -1 的for 循环。我认为你不需要 -1。

Answer 4

我知道您已经找到了答案，但我只是想向您展示您正在做的事情的替代方法。

为了简化您的代码并使其更易于理解且更灵活，通常最好确保每个函数只执行一件事或具有单一职责。

在您的代码中，您的LongestWord 函数负责识别什么是单词并找出哪个单词最长。

你可以做的是创建一个知道如何将句子标记为单词的函数：

function forEachWordsIn(str, callback) {
    var rx = /\b(\w+?)\b/g,
        match;

    while (match = rx.exec(str)) callback(match[0]);
}

然后使用longestWord函数中的那个词迭代器函数，这使得这个算法现在变得非常简单：

function longestWord(sen) {
    var longestWord = '';

    forEachWordsIn(sen, function (word) {
        if (word.length > longestWord.length) longestWord = word;
    });

    return longestWord;
}

注意：我将LongestWord 重命名为longestWord，因为使函数以大写字母开头是识别构造函数的众所周知的标准。