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

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

/// <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 ...

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

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

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

17
.net c# debugging exception substring

2 ответов

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

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

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

10

после комментариев я обнаружил, что подстрока не терпит неудачу при недопустимом индексе. Реальная проблема заключается в 

@string.IndexOf(substring);

Как указал Пьер-Люк Пиньо

устранение:

@string.IndexOf(substring, StringComparison.Ordinal);
-1