string.LastIndexOf() Bug?

The string, int32 overload of LastIndexOf says, in the description, "Reports the zero-based index position of the last occurrence of a specified Unicode character within this instance. The search starts at a specified character position and proceeds backward toward the beginning of the string."

Thus if you pass in 0, it will only check the first character, not check the whole string from 0.

The second parameter does not do what you seem to think it does:

LastIndex(char value, int startIndex)

the startIndex is the char to start searching backwards through the string, so if you pass a 0 then only the first char is checked...

To check the whole string from the end you would have to pass the length of the string -1.

see MSDN String.LastIndex

The docs (http://msdn.microsoft.com/en-us/library/bc3z4t9d.aspx#Y0) say:

The search begins at the startIndex character position of this instance and proceeds backward toward the beginning until either value is found or the first character position has been examined. For example, if startIndex is Length - 1, the method searches every character from the last character in the string to the beginning.

(My emphasis)

So this:

"  <exception>".LastIndexOf("<",0)

is beginning at 0 and working backwards, and therefore correctly finding no result and returning -1.

I think the confusion is that LastIndexOf counts backwards, where IndexOf counts forwards.

I think you're missunderstanding the method.

The second parameter is the character where you start to search. If you search backward from the character in the 0 position... the results are right.

It is a mistake, because LastIndexOf ** () ** search backward from the specified location.

Try this code:

"  <exception>".LastIndexOf("<", 5)

Hope it is useful

That's because the second parameter says that it should start at position zero and look from there and towards the beginning of the string.

You would only find the string if it is at the beginning of the string, and only if you are looking for a string that is one character long:

"<exception>".LastIndexOf("<",0)  // returns 0

"<exception>".LastIndexOf("<ex",0) // returns -1

• haystack.LastIndexOf("needle") searches a string for the last occurence of a "needle" -string.

• The method LastIndexOf and IndexOf returns -1 if the value to search for never occurs.

• LastIndexOf seeks from <--right-to-left, whereas

• IndexOf seeks from left-to-right-->

  • The character indices of the string start at 0 and increment from left to right

  • Both methods assume an offset-index of 0 when no second parameter is supplied

• The function also exists in JavaScript as lastIndexOf

  • (Note: the mixedcase or camelBack-casing) and that setting an index outside the bounds of the haystring's length is OK in JavaScript, but in C# it will result in a System.ArgumentOutOfRangeException: < 0 || > this.Length Exception.

Examples:

"  <exception>".LastIndexOf("<", 3)
 //> returns 2
"  <exception>".LastIndexOf("<", "  <exception>".Length)
 //> returns 2
"  <exception>".LastIndexOf("<", "  <exception>".Length+1)
 //> ArgumentOutOfRangeException
"  <exception>".LastIndexOf("<", 2)
 //> returns 2
"  <exception>".LastIndexOf("<", -1)
 //> returns -1
"  <exception>".LastIndexOf("<", -2)
 //> ArgumentOutOfRangeException
"  <exception>".LastIndexOf("<", 1)
 //> returns -1

Since in the last case no string or character < exists within the haystring's character interval of 0-1, -1 is correctly returned.

It's OK the first expression finds the < symbol from the 0 element of string to begin e. g. no such simbol find; the second one search through whole the string and find in position 2 and last one searching from 0 simbol which equal < and return the 0 position.

msdn) link