I wrote such code and i can’t understand why my loop doesn’t work (it shows check mark only for example “Google do a barrel roll” where the the first word is longest. i would be glad any advice before i will go to see someone’s code.
Your code so far
function findLongestWord(str) {
var arr = str.split(' ');
for (var n = 0;n <=arr.length; n++) {
var max = 0;
if (arr[n].length >= arr[n+1].length) {
if (arr[n].length > max)
max = arr[n].length;
}
else {
if (arr[n+1].length > max)
max = arr[n+1].length;
}
return max;
}
}
findLongestWord("Google do a barrel roll");
1 Like
Thanks randelldawson. I’m glad that i needn’t to see finished works
I did it with yout remark though still not absolutely sure why it worked =) Last thing i changed was < instead of <= and all worked
function findLongestWord(str) {
var arr = str.split(' ');
var max = 0;
for (var n = 0; n <arr.length; n++) {
if (arr[n].length > max) {
max = arr[n].length;
}
}
return max;
}
findLongestWord("Google do a barrel roll");
Other than the ‘return’ problem, I noticed :
In the first block of code you had placed the ‘var max = 0’ inside the for loop where it would be set back to ‘0’ for each iteration.
In the code modifications where it worked, you have moved ‘var max = 0;’ out of the loop so it can keep track of the longest word encountered without being reset each time.
With max being reset on each iteration I think the effect would have been to return the length of the last word tested every time?
1 Like
Yes, exactly, i played with this variable also
I’ve edited your post for readability. When you enter a code block into the forum, remember to precede it with a line of three backticks and follow it with a line of three backticks to make easier to read. See this post to find the backtick on your keyboard. The “preformatted text” tool in the editor (</>) will also add backticks around text.
