我的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 ...
不幸的是,我无法远程访问此生产环境或其数据或任何其他信息。由于某些原因,目前我无法在那里部署日志记录系统或崩溃转储收集。
查看代码,并尝试不同的参数组合,我无法想象这种异常是如何发生的。
你能帮我一些想法吗?
发布于 2013-07-21 12:12:53
RemoveSubstring("A", "A\uFFFD"); // throws ArgumentOutOfRangeException
RemoveSubstring("A", "A\u0640"); // throws ArgumentOutOfRangeException
默认情况下,.NET中用于字符串操作的许多函数(包括IndexOf
)都是特定于文化的(通常会有重载,您可以在其中传递StringComparison.Ordinal
或StringComparer.Ordinal
以切换到逐位比较)。就我个人而言,我对被选为默认行为不是很满意,但现在对它做任何事情都为时已晚,除非可能,设置明确的开发指南和FxCop规则。
但有时特定于文化的操作正是您所需要的。不幸的是,它们的语义可能是棘手的和反直觉的,可能会违反一些正常假设的不变量,并且需要处理许多转角情况。负责在应用程序中实现区域性敏感逻辑的开发人员应该非常精通这一领域,并且始终准确地了解他们在做什么。我建议为这一领域设定高于正常水平的审查和测试标准。
发布于 2013-07-21 11:58:00
在注释之后,我发现子字符串在无效的索引上不会失败。真正的问题是
@string.IndexOf(substring);
正如Pierre-Luc Pineault所指出的
更正:
@string.IndexOf(substring, StringComparison.Ordinal);
https://stackoverflow.com/questions/17768854
复制相似问题