>>9239378>>9239387From a more general standpoint, this code is way more complicated than it needs to be. You don't need a for loop and two variables here; you could get by with just a while loop and one variable at most. Honestly, you may be better served by starting over and keeping in mind exactly what you want the code to do:
1) Search for the first space in the sentence
2) If there is a space, see if the word at the start of the sentence is longer than the current longest word, and process accordingly
3) If there is not a space, you have reached the last word in the sentence, process it accordingly and terminate.
But, if you want to stick this code, a few issues stand out:
First of all, you have j = sl.indexOf(" ",i) and i = sl.indexOf(" ", i). This works fine on the first iteration, but once you set i = sl.indexOf(" ",i), it is no longer going to do what you intend it to (i.e., find the first space in the remainder of the sentence). On the next loop, it will start from the ith letter in the sentence and search for the next space in the sentence, which is not necessarily the first space in the sentence.
Second, think about what temp = sl.substring(i,j) is doing, particularly in the context of what sl is during later iterations (i.e., once you have run sl = sl.substring((j+1)) ). Once you have removed words that you have already checked, the next word is at the start of sl.
Once this error is corrected, your code doesn't properly handle the last word in the sentence. When you go through the loop and there are two words left, you process the first word, then set i = sl.indexOf(" ", i). But when there is only one word left, i is then going to become -1, unless there is a space at the very end of the sentence. So you don't do anything for the last word.
