Regex nur für Zahlen und Löschtaste

Hallo,

brauche mal wieder Hilfe 🙁 bei C# Formulare

Habe folgendes geschrieben:

public static bool nurZahlen(KeyPressEventArgs e)
        {

            return Regex.IsMatch(e.KeyChar.ToString(),@"\b[0-9]");
}

nur leider tut das net 🙁 Zahlen kann ich keine eingeben .. dafür aber buchstaben 🙁
Kann mir bitte jemand helfen?

Wobei dies aber funktioniert:

 public static bool nurZahlen(KeyPressEventArgs e)
        {

            if ("1234567890\b".IndexOf(e.KeyChar.ToString()) < 0)
                return true;
            else
                return false;
        }

Würde es aber gerne mit einem Regulären Ausdruck machen! Kann mir bitte jemand helfen?

Lieben Gruß
TigerCrow

Du willst einen RegEx-Ausdruck haben um ein Zeichen (char) zu überprüfen. Habe ich das so richtig verstanden?

Findest du das nicht etwas oversized?

Hast Du richtig verstanden und ja gebe Dir recht das dies wie Du so schön sagst oversized ist

Aber Du kennst mich ja inzwieschen 😉

Und wie schreibe ich das jetzt mit einem Regulären ausdruck?

Sir Rufo hat schon recht: was Du da machst ist einfach nur Quatsch, verschenkte Zeit und verschenkte Ressourcen.
Regex ist kostenintensiv im Sinne der Rechenzeit. Das ist einfach - sorry - alles andere als intelligent, das mit Regex zu machen 😉

Aber um die Frage zu beantworten:

\b[0-9] matched einwandrei auf Ziffern, aber nicht auf Zahlen.
Was Du hier überprüfst ist auch eigentlich eine Ziffer (da Du auch nur ein Char und kein String als Eingang hast).

Ich bezweifle also, dass es hier am Regex liegt.
Aber das kannste ja prima selbst mit dem Debugger herausfinden.
[Artikel] Debugger: Wie verwende ich den von Visual Studio?

  if ("1234567890\b".IndexOf(e.KeyChar.ToString()) < 0)
                return true;

sieht für mich logisch gesehen auch falsch aus.
Wenn es weniger als 0 (also wenn IndexOf nichts findet und -1 liefert), dann lieferst Du true zurück. Ist es das, was Du wirklich willst? 🤔
Vermutlich liegt der Fehler eher in Deiner Logik des Aufrufs von "nurZahlen".

Dazu auch gleich:
[Artikel] C#: Richtlinien für die Namensvergabe

Und wenn eine Methode "NurZahlen" heisst, dann sollte sie auch "nur Zahlen" verarbeiten und nicht noch \b.
Sonst macht der Name absolut keinen Sinn.

Zudem mischt Du UI mit Logik => zukünftige Probleme vorprogrammiert
[Artikel] Drei-Schichten-Architektur

Zum Prüfen ob ein char eine Ziffer darstellt gibt es sogar schon eine eingebaute Methode Char.IsDigit()

Hallo TigerCrow,

KeyPress ist der falsche Ansatzpunkt. Das führt zu mehreren Problemen und stellt im schlimmsten Fall nicht mal das gewünschte Verhalten sicher. Richtig ist TextChanged. Alles weiter steht in [FAQ] In einer TextBox nur bestimmte Zeichen/Eingaben zulassen.

herbivore

Danke für eure Infos
ein kleiner Code hätte es auch getan 🙂

aber so geht es:

public static bool nurZahlen(KeyPressEventArgs e)
        {

            return !Regex.IsMatch(e.KeyChar.ToString(),@"\b[0-9]");
}

Ihr wisst das man manchmal was sieht und das machen möchte, nur dann wenn es nicht geht, warum auch immer, fragt man sich halt warum.
Das mit dem char.isDigit wusste ich nicht .. Danke dafür.

Was mit den Regulären ausdrücken ist ... habe ich immer gedacht die wären besonders schnell .. lag ich wohl falsch ... aber wieder was gelernt 🙂

vielen Dank

Gruß
TigerCrow

Hallo TigerCrow,

aber so geht es:

nein, so geht es eben nicht. Zumindest hat die Behandlung in KeyPress vorzunehmen unerwünschte Nebenwirkungen. Wie ich schon schrieb, siehe [FAQ] In einer TextBox nur bestimmte Zeichen/Eingaben zulassen.

herbivore