我不明白你为什么要在这里使用正则表达式。一个简单的单行就可以,例如。即使是这样的扩展方法:
static int NumbersCount(this string str)
{
return str.ToCharArray().Where(c => Char.IsNumber(c)).Count();
}
在我看来,它更清晰,更易于维护。
您可能可以尝试使用组匹配和类似的东西
"(?<client>[0-9]{5,9}?)|(?<serial>[0-9]{10}?)|(?<mobile>[0-9]{13,}?)"
然后你会检查你是否匹配“client”、“serial”、“mobile”,并在此基础上解释字符串输入。但是更容易理解吗?
对于以后阅读您的代码的人来说,它是否更清楚地表达了您的意图?
如果要求这些数字必须是连续的(正如@Corak 指出的那样)......我仍然会迭代地写,就像这样:
/// <summary>
/// returns lengths of all the numeric sequences encountered in the string
/// </summary>
static IEnumerable<int> Lengths(string str)
{
var count = 0;
for (var i = 0; i < str.Length; i++)
{
if (Char.IsNumber(str[i]))
{
count++;
}
if ((!Char.IsNumber(str[i]) || i == str.Length - 1) && count > 0)
{
yield return count;
count = 0;
}
}
}
然后你可以简单地:
bool IsClientID(string str)
{
var lenghts = Lengths(str);
return lenghts.Count() == 1 && lenghts.Single() == 5;
}
它更冗长吗?是的,但是如果每次验证规则发生变化或需要进行一些调试时,人们仍然会更喜欢你,而不是让他们摆弄正则表达式:) 这包括你未来的自己。