Замініть нечислові порожніми рядками

Швидке додавання до вимог у нашому проекті. Поле в нашій БД для розміщення телефонного номера задано лише 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, "");
}

Залежно від ваших даних у реальному світі, вам може знадобитися додаткова логіка для того, щоб робити такі речі, як викреслити провідні позиції 1 (на великі відстані) або що-небудь, що відстає від 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", String.Empty);

Як щодо методу розширення, який не використовує регулярний вираз.

Якщо ви дотримуєтесь одного з варіантів 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, "");
    }
}

Результат на моєму комп’ютері:
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();
        }