Funktion schlägt 2 von 3 Tests fehl

Question

Ich helfe einem Freund, eine Funktion zu schreiben, die eine Zeichenfolge von Leerzeichen-getrennten Wörtern nimmt und die längste zurückgibt, aber mein Code fehlschlägt 2 aus den 3 Tests, die es zu überprüfen.

Dies sind die Tests:

describe("test for longest function functionality", function() { 
    it("should return correct output for normal strings", function() { 
    expect(longest("A")).toEqual("A"); 
    expect(longest("I love Avatar")).toEqual("Avatar"); 
    expect(longest("The stupidities of youth")).toEqual("stupidities"); 
    }); 

    it("should return correct output for gibberish", function() { 
    expect(longest("hgdydrxtfEq Rradsc tstsa taeWwwec fgdd")).toEqual("hgdydrxtfEq"); 
    }); 

    it("should work for sentences with numbers", function() { 
    expect(longest("This is a sentence with a number 7685838788")).toEqual("7685838788"); 
    }); 
}); 

Dies ist meine Lösung so weit:

function longest(str) { 
    var longestStart = 0; 
    var longestEnd = 0; 
    var current = 0; 
    var currentLength = 0; 
    while (current < str.length) { 
    if (str.charAt(current) == ' ') { 
     if (currentLength > (longestEnd - longestStart)) { 
     longestStart = (current - currentLength); 
     longestEnd = current; 
     } 
     currentLength = 0; 
    } else { 
     currentLength++; 
    } 
    current++; 
    } 
    return str.substring(longestStart, longestEnd); 
} 

Answer 1

Sie Verarbeitung nicht nach der Zeichenfolge endet. Versuchen Sie, diesen Code auch am Ende hinzuzufügen:

if (currentLength > (longestEnd - longestStart)) { 
    longestStart = (current - currentLength); 
    longestEnd = current; 
} 

Ihr Code ist nicht der einfachste; Sie sollten String.prototype.split() verwenden, um den Code zu erreichen, der am einfachsten zu verstehen ist.

Answer 2

Persönlich würde ich dies tun, indem eine Funktion wie so mit:

function longestSubstring(stringToSplit) { 
 
    var arrayOfStrings = stringToSplit.split(" "); 
 

 
    var longestString = arrayOfStrings[0]; 
 

 
    for (var i = 0; i < arrayOfStrings.length; i++) { 
 
    if (longestString.length < arrayOfStrings[i].length) { 
 
     longestString = arrayOfStrings[i]; 
 
    } 
 
    } 
 

 
    console.log(longestString); 
 
} 
 

 
longestSubstring("Hello my friend"); 
 
longestSubstring("1 12 123 1234 12345"); 
 
longestSubstring("aaaaaaah hello yell");