Ersetzen Sie nicht numerische Zeichen durch leere Zeichenfolgen

Schnelle Add-On-Anforderung in unserem Projekt. Ein Feld in unserer Datenbank, in dem eine Telefonnummer gespeichert ist, darf nur 10 Zeichen zulassen. Wenn ich also "(913) -444-5555" oder etwas anderes übergeben bekomme, gibt es eine schnelle Möglichkeit, eine Zeichenfolge durch eine spezielle Ersetzungsfunktion zu führen, bei der ich eine Reihe von Zeichen übergeben kann, um dies zuzulassen?

Regex?

Auf jeden Fall Regex:

string CleanPhone(string phone)
{
    Regex digitsOnly = new Regex(@"[^\d]");   
    return digitsOnly.Replace(phone, "");
}

oder innerhalb einer Klasse, um zu vermeiden, dass der reguläre Ausdruck ständig neu erstellt wird:

private static Regex digitsOnly = new Regex(@"[^\d]");   

public static string CleanPhone(string phone)
{
    return digitsOnly.Replace(phone, "");
}

Abhängig von Ihren realen Eingaben möchten Sie möglicherweise eine zusätzliche Logik, um beispielsweise führende 1 (für große Entfernungen) oder alles, was hinter einem x oder X (für Erweiterungen) liegt, zu entfernen.

Sie können es einfach mit Regex tun:

string subject = "(913)-444-5555";
string result = Regex.Replace(subject, "[^0-9]", ""); // result = "9134445555"

Sie müssen Regex nicht verwenden.

phone = new String(phone.Where(c => char.IsDigit(c)).ToArray())

Hier ist die Methode der Erweiterungsmethode.

public static class Extensions
{
    public static string ToDigitsOnly(this string input)
    {
        Regex digitsOnly = new Regex(@"[^\d]");
        return digitsOnly.Replace(input, "");
    }
}

Mit den Regex-Methoden in .NET sollten Sie in der Lage sein, jede nicht numerische Ziffer mit \ D wie folgt abzugleichen:

phoneNumber  = Regex.Replace(phoneNumber, "\\D", String.Empty);

Wie wäre es mit einer Erweiterungsmethode, die keinen regulären Ausdruck verwendet?

Wenn Sie sich an eine der Regex-Optionen halten, verwenden Sie diese zumindest RegexOptions.Compiledin der statischen Variablen.

public static string ToDigitsOnly(this string input)
{
    return new String(input.Where(char.IsDigit).ToArray());
}

Dies baut auf der Antwort von Usman Zafar auf, die in eine Methodengruppe konvertiert wurde.

Versuchen Sie Folgendes, um die beste Leistung und einen geringeren Speicherverbrauch zu erzielen:

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, "");
    }
}

Das Ergebnis auf meinem Computer ist:
Init ...
Zeit: 307
Zeit: 2178

Ich bin mir sicher, dass es einen effizienteren Weg gibt, aber ich würde dies wahrscheinlich tun:

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();
}

Versuche dies

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();
        }