Как это возможно в этом коде: "ArgumentOutOfRangeException: startIndex не может быть больше длины строки"?

У меня есть следующий метод в коде С#:

/// <summary>
/// Removes the first (leftmost) occurence of a <paramref name="substring"/> from a <paramref name="string"/>.
/// </summary>
/// <param name="string">The string to remove the <paramref name="substring"/> from. Cannot be <c>null</c>.</param>
/// <param name="substring">The substring to look for and remove from the <paramref name="string"/>. Cannot be <c>null</c>.</param>
/// <returns>
/// The rest of the <paramref name="string"/>, after the first (leftmost) occurence of the <paramref name="substring"/> in it (if any) has been removed.
/// </returns>
/// <remarks>
/// <list type="bullet">
/// <item>If the <paramref name="substring"/> does not occur within the <paramref name="string"/>, the <paramref name="string"/> is returned intact.</item>
/// <item>If the <paramref name="substring"/> has exactly one occurence within the <paramref name="string"/>, that occurence is removed, and the rest of the <paramref name="string"/> is returned.</item>
/// <item>If the <paramref name="substring"/> has several occurences within the <paramref name="substring"/>, the first (leftmost) occurence is removed, and the rest of the <paramref name="string"/> is returned.</item>
/// </list>
/// </remarks>
/// <exception cref="ArgumentNullException">
/// The <paramref name="string"/> is <c>null</c>. -or- The <paramref name="substring"/> is <c>null</c>.
/// </exception>
public static string RemoveSubstring(string @string, string substring)
{
    if (@string == null)
        throw new ArgumentNullException("string");

    if (substring == null)
        throw new ArgumentNullException("substring");

    var index = @string.IndexOf(substring);
    return index == -1
        ? @string
        : @string.Substring(0, index) + @string.Substring(index + substring.Length);
}

Реализация выглядит очень простой и понятной и имеет превосходный охват модульными тестами. Никаких неожиданных результатов на моей машине, построения серверов или других машин, к которым у меня нет доступа, или в большинстве производственных средах, не было.

За исключением того, что только один удаленный клиент изредка сообщает об аварии приложения при этом методе со следующей трассировкой стека:

System.ArgumentOutOfRangeException: startIndex cannot be larger than length of string.  
Parameter name: startIndex   
   at System.String.InternalSubStringWithChecks(Int32 startIndex, Int32 length, Boolean fAlwaysCopy)  
   at System.String.Substring(Int32 startIndex)
   at MyNamespace.StringUtils.RemoveSubstring(String string, String substring)
   at ...

К сожалению, у меня нет удаленного доступа к этой производственной среде или ее данным или к любой дополнительной информации. По некоторым причинам, в настоящее время я не могу развернуть систему ведения журнала или сборку дампов сбоев.

Если посмотреть на код и попробовать разные комбинации аргументов, я не могу представить, как могло произойти это исключение.

Не могли бы вы помочь мне с некоторыми идеями?

Ответ 1

RemoveSubstring("A", "A\uFFFD"); // throws ArgumentOutOfRangeException
RemoveSubstring("A", "A\u0640"); // throws ArgumentOutOfRangeException

Многие функции для строковых манипуляций в .NET, включая IndexOf, по умолчанию зависят от культуры (обычно есть перегрузки, где вы можете передать StringComparison.Ordinal или StringComparer.Ordinal, чтобы переключиться на побитовые сравнения). Лично я не очень доволен тем, что было выбрано как поведение по умолчанию, но слишком поздно что-то делать с этим, за исключением, возможно, установления явных директив разработки и правил FxCop.

Но иногда специфические для конкретной культуры операции - это именно то, что вам нужно. К сожалению, их семантика может быть сложной и противоречащей интуиции, может нарушать некоторые обычно предполагаемые инварианты и иметь множество угловых случаев, о которых нужно позаботиться. Разработчики, которые отвечают за внедрение чувствительной к культуре логики в приложении, должны быть очень квалифицированными в этой области и всегда точно понимать, что они делают. Я бы рекомендовал установить стандарты проверки и тестирования для этой области выше нормального.

Ответ 1

Ответ 2