Как это возможно в этом коде: "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.
Но иногда специфические для конкретной культуры операции - это именно то, что вам нужно. К сожалению, их семантика может быть сложной и противоречащей интуиции, может нарушать некоторые обычно предполагаемые инварианты и иметь множество угловых случаев, о которых нужно позаботиться. Разработчики, которые отвечают за внедрение чувствительной к культуре логики в приложении, должны быть очень квалифицированными в этой области и всегда точно понимать, что они делают. Я бы рекомендовал установить стандарты проверки и тестирования для этой области выше нормального.
Ответ 2
После комментариев, я выяснил, что подстрока не терпит неудачу по недопустимому индексу. Реальная проблема заключается в
@string.IndexOf(substring);
как указал Пьер-Люк Пинео
Исправление:
@string.IndexOf(substring, StringComparison.Ordinal);