RegExp: +-Modifier außerhalb eckiger Klammern wird nicht gewertet

[C++-Nerd]

Ich habe einen Regulären Ausdruck, um damit bei einer Usereingabe (genauer ein Dateiname) zu prüfen, ob wirklich auch nur A-Z, a-z, der Unterstrich _ , ein Hochkomma ' oder ein Strich - enthalten ist.

Wenn ich diesen aber mit der Funktion preg_match teste, wird mir bei allen Strings true (1) zurückgegeben:

$allowed_chars_regex = "/[a-zA-Z_\-']+/" ;

#=> Hier sollte false herauskommen, es kommt jedoch true
echo (preg_match($allowed_chars_regex, "öl%&/")); 

#=> Hier kommt richtigerweise true heraus
echo (preg_match($allowed_chars_regex, "rollo"));

Die RegExp muss jedoch stimmen, denn wenn ich sie auf RegExr: Online Regular Expression Testing Tool an beiden Strings teste, werden die richtigen Gruppen markiert.

Woran kann das liegen?

 

[struppi]

in beiden Zeichenketten kommen Buchstaben vor, du musst im ersten Fall das l rausnehmen.

 

[Commodore]

Negier einfach deinen RegEx:

$allowed_chars_regex = "/[^a-zA-Z_\-']+/";
//                        ^--- Damit wird alles gematcht, was nicht zu dem nachfolgenden gehört

(ungetestet)



Natürlich müsstest du dann entsprechend die Negation der Rückgabe von preg_match zurückgeben.

 

[C++-Nerd]

Es scheint, als hätte ich meine eigene RegExp bzw. den Aspekt nicht verstanden, dass nur ein Match dem RegExp entsprechen muss.

Und deswegen danke an struppi, aufgrunddessen mir eingeleuchtet ist, warum es nicht funktioniert und auch an Commodore, weil ich nun nicht nach einer funktionierenden Lösung mehr suchen muss (:

 

[Frank]

Ums nochmal klar zu machen

$allowed_chars_regex = "/[a-zA-Z_\-']+/" ;

#=> Hier sollte false herauskommen, es kommt jedoch true
echo (preg_match($allowed_chars_regex, "öl%&/"));

Dein Code ist schon richtig. Allerdings sagst du, dass an beliebieger Stelle mindestens eines der Zeichen a-z, A-z _ - und ' vorkommen muss.
Folglich stimmt die bedingung, denn in öl%&/ kommt ein "l" vor, dass auf diese Bedindung passt.

Was du im regex vergessen hast ist folgendes

$allowed_chars_regex = "/^[a-zA-Z_\-']+/$" ;

Damit sagst du ihm, dass deine Zeichenkette von ANFANG (^) bis zum ENDE ($) zutreffen muss, und nicht wie ich oben schrieb, nur ein beliebiger Teil.

 

[Gelöschtes Mitglied 3007]

$allowed_chars_regex = "/^[a-zA-Z_\-']+$/";

// 1 Fehler behoben