RegEx funktioniert nicht

rs-web · #1 (**permalink**) 15.05.2010, 15:46

Hi, ich hab hier Mal ein Problem mit RegEx. Ich habe ein Suchmuster das nach Sonderzeichen sucht. Aber wenn im Namen nun einÖ vorkommt, kommt trotzdem true raus. Hier Mal die Funktion:

PHP-Code:


			
function usernameWritingCheck($username){

    if(strlen($username) > 15){

        $zulang = true; //Ja, er ist zu lang.

    }

    else{

        $zulang = false;//Nein, er passt.

    }

    $muster = "/[äöü\-@!éèà$%&\.\,\?\*\+\^\$\(\)\[\]]/"; //Liste der Verbotenen Zeichen

    if(preg_match($muster, $username)){

        $verbotenesZeichen = true; //Verbotenes Zeichen gefunden

    }

    else{

        $verbotenesZeichen = false; //Alles Ok

    }

    if($zulang == true){

        return false;

    }

    elseif($verbotenesZeichen == true){

        return false;

    }

    else{

        return true;

    }

}

Wenn der String länger als 15 Zeichen ist, kommt false zurück. Wenn jetzt aber "Benutzernamö" eingegeben wird, passiert nichts. Ich habe mir die Doku schon angesehen aber habe keinen Fehler finden können. Danke schon Mal.
LG rs-web

Karlo · #2 (**permalink**) 15.05.2010, 19:08

Hi,

der Modifikator (s|i|U) ist dein Freund...

rs-web · #3 (**permalink**) 15.05.2010, 19:56

Hi, danke es sieht jetzt wiefolgt aus:

PHP-Code:


			
function usernameWritingCheck($username){

    if(strlen($username) > 15){

        $zulang = true; //Ja, er ist zu lang.

    }

    else{

        $zulang = false;//Nein, er passt.

    }

    $muster = "/[äöü\-@!éèà$%&\.\,\?\*\+\^\$\(\)\[\]]/siU"; //Liste der Verbotenen Zeichen

    if(preg_match($muster, $username)){

        $verbotenesZeichen = true; //Verbotenes Zeichen gefunden

    }

    else{

        $verbotenesZeichen = false; //Alles Ok

    }

    if($zulang == true){

        return false;

    }

    elseif($verbotenesZeichen == true){

        return false;

    }

    else{

        return true;

    }

}

Aber es funktioniert immer noch nicht.
Danke schon mal

inta · #4 (**permalink**) 16.05.2010, 00:10

Du hast einen recht umständlichen Codestil, das geht viel kürzer und sehr viel verständlicher:

PHP-Code:


			
function usernameWritingCheck($username)
{
    if (strlen($username) > 15) {
        // Ja, er ist zu lang.
        return false;
    }
    // Liste der Verbotenen Zeichen
    $muster = "/[]äöü@!éèà$%&.,?*+^()[-]/iU";
    if (preg_match($muster, $username)) {
        // Verbotenes Zeichen gefunden
        return false;
    }
    return true;
}

Noch besser wird es, wenn du deutsch und englisch nicht mischst.

Der PHP-Code ist funktional identisch mit deinem, auch den regulären Ausdruck habe ich nur aufgeräumt (und den überflüssigen Modifikator "s" wieder rausgeschmissen). Die Funktion tut genau das was sie vermutlich soll, wenn der Name Ok ist gibt sie true zurück, ansonsten false.

Du solltest generell deinen Ansatz überdenken und statt auf verbotene Zeichen lieber prüfen, ob nur erlaubte verwendet wurden. Trotz deiner Prüfung sind immer noch eine Menge von Sonderzeichen möglich (keine Ahnung ob das so von dir gewollt ist.

rs-web · #5 (**permalink**) 16.05.2010, 11:04

Hi,
also ich hab das Mal 1:1 kopiert, doch es klappt nicht. Wenn ich "Benutzernamö" eingebe gibt er true zurück. Wenn ich jetzt aber "Benutzernameeeeee" eingebe gibt er false zurück. Ich dachte man kann mit regEx nur Blacklists erstellen, weil der User muss ja z.B. keine Großbuchstaben verwenden. Es soll aber, wenn z.B. ein "@" vorkommt eine Fehlermeldung geben. Ich hoffe ihr wisst was ich meine..

LG rs-web

rs-web · #6 (**permalink**) 16.05.2010, 11:25

Irgendwie kapier ich das nicht mehr. Ich habe die Funktion wiefolgt geändert:

PHP-Code:


			
function usernameWritingCheck($username)

{

    if (strlen($username) > 15) {

        // Ja, er ist zu lang.

        return false;

    }

    if (strlen($username) < 4) {

        return false;

    }

    // Liste der Verbotenen Zeichen

    $muster = "/[]äöü@!éèà$%&.,?*+^()[-]/iU";

    if (preg_match($muster, $username)) {

        // Verbotenes Zeichen gefunden

        return false;

    }

    return true;

}

Wenn der Name "Bei" lautet wird false zurückgegeben, bei "Beö" true. Das kapier ich nicht mehr.

inta · #7 (**permalink**) 16.05.2010, 12:23

Zitat:

Zitat von rs-web

also ich hab das Mal 1:1 kopiert, doch es klappt nicht. Wenn ich "Benutzernamö" eingebe gibt er true zurück.

Wie testest du das?

Das hier:

PHP-Code:


			
<?php



function usernameWritingCheck($username)

{

    if (strlen($username) > 15) {

        // Ja, er ist zu lang.

        return false;

    }

    // Liste der Verbotenen Zeichen

    $muster = "/[]äöü@!éèà$%&.,?*+^()[-]/iU";

    if (preg_match($muster, $username)) {

        // Verbotenes Zeichen gefunden

        return false;

    }

    return true;

}



var_dump(usernameWritingCheck('Benutzernamö'));

ergibt bool(false).

Ich vermute du hast ein Zeichenkodierungsproblem. Woher kommen deine Daten? Testest du das schon mit einem Formular? Lass dir mal ausgeben was genau im PHP-Skript ankommt.

Zitat:

Zitat von rs-web

Ich dachte man kann mit regEx nur Blacklists erstellen, weil der User muss ja z.B. keine Großbuchstaben verwenden. Es soll aber, wenn z.B. ein "@" vorkommt eine Fehlermeldung geben. Ich hoffe ihr wisst was ich meine..

Das geht sogar noch viel besser über den umgekehrten Weg, indem du mit deinem regulären Ausdruck nur testest ob der Benutzername deinen Kriterien entspricht, zum Beispiel so:

PHP-Code:


			
function usernameWritingCheck($username)

{

    if (preg_match('/^[a-z0-9]{4,15}$/i', $username)) {

        return true;

    }

    return false;

}

Hier wären zum Beispiel alle Buchstaben von A - Z und alle Zahlen erlaubt, der Benutzername muss zwischen 4 und 15 Zeichen lang sein.

rs-web · #8 (**permalink**) 16.05.2010, 12:37

Hi,
ja ich teste schon mit einem Formular. Vielen Dank mit der letzten Funktion klappt alles wunderbar. Wie es scheint, muss ich mich mit RegEx erst noch anfreunden

LG rs-web

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Ansicht
Linear-Darstellung Zur Hybrid-Darstellung wechseln Zur Baum-Darstellung wechseln

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Mouseover funktioniert nicht (externe Methode)	blackhtml	CSS	2	07.04.2009 17:49
RegEx funktioniert, aber eigentlich auch nicht - warum?	Boris	Serveradministration und serverseitige Scripte	2	04.10.2007 12:47
dtd und css in php funktioniert im IE nicht!!!	da-lick	CSS	17	09.06.2007 15:44
Regex Problem ("]" killt die regex)	psycho_dmr	Serveradministration und serverseitige Scripte	7	29.04.2007 16:49
hier eine leichte: regex über seite bauen	bernard_hinault	CSS	13	01.09.2006 20:33