Заменить нечисловое значение пустой строкой
-
06-07-2019 - |
Вопрос
Быстрое дополнение в нашем проекте.Поле в нашей базе данных для хранения номера телефона допускает только 10 символов.Итак, если мне передают «(913)-444-5555» или что-то еще, есть ли быстрый способ запустить строку через какую-то специальную функцию замены, которую я могу передать ей набор разрешенных символов?
Регулярное выражение?
Решение
Определенно регулярное выражение:
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, "");
}
В зависимости от ваших реальных входных данных вам может потребоваться дополнительная логика для выполнения таких действий, как удаление ведущих единиц (для больших расстояний) или всего, что заканчивается x или X (для расширений).
Другие советы
Вы можете легко сделать это с помощью регулярного выражения:
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())
Вот способ сделать это методом расширения.
public static class Extensions
{
public static string ToDigitsOnly(this string input)
{
Regex digitsOnly = new Regex(@"[^\d]");
return digitsOnly.Replace(input, "");
}
}
Используя методы Regex в .NET, вы сможете сопоставить любую нечисловую цифру с помощью \D, например:
phoneNumber = Regex.Replace(phoneNumber, "\D", "");
Как насчет метода расширения, который не использует регулярное выражение.
Если вы придерживаетесь одного из вариантов Regex, по крайней мере, используйте RegexOptions.Compiled
в статической переменной.
public static string ToDigitsOnly(this string input)
{
return new String(input.Where(char.IsDigit).ToArray());
}
Это основано на ответе Усмана Зафара, преобразованном в группу методов.
для лучшей производительности и меньшего потребления памяти попробуйте следующее:
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, "");
}
}
Результат на моем компьютере:
В этом...
Время: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();
}