回答:
間違いなく正規表現:
string CleanPhone(string phone)
{
Regex digitsOnly = new Regex(@"[^\d]");
return digitsOnly.Replace(phone, "");
}
またはクラス内で常に正規表現を再作成しないようにします。
private static Regex digitsOnly = new Regex(@"[^\d]");
public static string CleanPhone(string phone)
{
return digitsOnly.Replace(phone, "");
}
実際の入力に応じて、先頭の1を取り除く(長距離の場合)か、xまたはXの後ろにあるもの(延長の場合)などを実行するための追加のロジックが必要になる場合があります。
[^\d]
は次のように簡略化できます\D
あなたは正規表現でそれを簡単に行うことができます:
string subject = "(913)-444-5555";
string result = Regex.Replace(subject, "[^0-9]", ""); // result = "9134445555"
Regexを使用する必要はありません。
phone = new String(phone.Where(c => char.IsDigit(c)).ToArray())
system.linq;
最高のパフォーマンスとより少ないメモリ消費量のために、これを試してください:
using System;
using System.Diagnostics;
using System.Text;
using System.Text.RegularExpressions;
public class Program
{
private static Regex digitsOnly = new Regex(@"[^\d]");
public static void Main()
{
Console.WriteLine("Init...");
string phone = "001-12-34-56-78-90";
var sw = new Stopwatch();
sw.Start();
for (int i = 0; i < 1000000; i++)
{
DigitsOnly(phone);
}
sw.Stop();
Console.WriteLine("Time: " + sw.ElapsedMilliseconds);
var sw2 = new Stopwatch();
sw2.Start();
for (int i = 0; i < 1000000; i++)
{
DigitsOnlyRegex(phone);
}
sw2.Stop();
Console.WriteLine("Time: " + sw2.ElapsedMilliseconds);
Console.ReadLine();
}
public static string DigitsOnly(string phone, string replace = null)
{
if (replace == null) replace = "";
if (phone == null) return null;
var result = new StringBuilder(phone.Length);
foreach (char c in phone)
if (c >= '0' && c <= '9')
result.Append(c);
else
{
result.Append(replace);
}
return result.ToString();
}
public static string DigitsOnlyRegex(string phone)
{
return digitsOnly.Replace(phone, "");
}
}
私のコンピューターでの結果は:
Init ...
時間:307
時間:2178
それを行うにはより効率的な方法があると確信していますが、おそらくこれを行うでしょう:
string getTenDigitNumber(string input)
{
StringBuilder sb = new StringBuilder();
for(int i - 0; i < input.Length; i++)
{
int junk;
if(int.TryParse(input[i], ref junk))
sb.Append(input[i]);
}
return sb.ToString();
}
これを試して
public static string cleanPhone(string inVal)
{
char[] newPhon = new char[inVal.Length];
int i = 0;
foreach (char c in inVal)
if (c.CompareTo('0') > 0 && c.CompareTo('9') < 0)
newPhon[i++] = c;
return newPhon.ToString();
}
return newPhone.ToString();
「System.Char []」を返します。私はあなたが意味したと思うがreturn new string(newPhone);
、これはまた、>
andの<
代わりに>=
andのために、0と9の数を除外している<=
。ただし、その場合でも、newPhon
配列が必要以上に長いため、stringには末尾のスペースが含まれます。