这是有条件的问题。现在,我搜索并看到了另一种(正确的)方法。但是我只是不明白为什么我的方法不起作用。
问题:编写一个名为“ getLongestOfThreeWords”的函数。
给定3个单词,“ getLongestOfThreeWords”返回三个单词中最长的一个。
笔记:
如果有平局,则应返回平局中的第一个单词。
我理解“正确”的方式,但是我的答案显示“未定义”,似乎无法弄清楚原因。
"My way"
function getLongestOfThreeWords(word1, word2, word3) {
if(word1.length > word2.length && word1.length > word3.length){
return word1;
} else if(word2.length > word1.length && word2.length > word3.length){
return word2;
} else if (word3.length > word1.length && word3.length > word2.length){
return word3;
}
}
与
"correct way"
function getLongestOfThreeWords(word1, word2, word3) {
let wordsArr = [word1, word2, word3];
let longestWord = '';
for (var i = 0; i < wordsArr.length; i++) {
if (wordsArr[i].length > longestWord.length) {
longestWord = wordsArr[i];
}
}
return longestWord;
}
var output = getLongestOfThreeWords('these', 'three', 'words');
console.log(output); // --> 'these'
最佳答案
因为您还没有针对边缘情况进行调整-当两个单词的长度相同且该长度大于另一个单词时。 (还请注意,您的方法效率很低,即使再输入一个字也需要进行大量调整),请尝试以下操作:
if (...) {...}
else if (...) {...}
else if (...) {...}
else if (word1.length == word2.length && word1.length > word3.length) {
return word1;
}
else if (word2.length == word3.length && word2.length > word1.length) {
return word3;
}