[PHP] String controleren op ongeldige tekens *

[maartenvdv737]

Ik heb voor het aanmelden bij mijn website de volgende restricties op de gebruikersnaam gelegd:

PHP:

if(eregi("[^A-Za-z0-9_]+", "$gebruikersnaam")) {
 $error_message .= "Je hebt een ongeldig teken in je gebruikersnaam, 
je mag gebruiken: A-Z,a-z en 0-9";
}

Nu zijn er toch nog mensen met een ' of een * in hun gebruikersnaam.

Weet iemand waarom die door deze check heenkomen?

[Erik Jan]

Waarom deze niet werkt, weet ik eigenlijk ook niet. Als er 1 of meer chars in voorkomen die niet tot [A-Za-z0-9_] behoort, levert dit gewoon een extra $error_message op toch?

Ik gebruik meestal zoiets:

PHP:

if(!ereg('^[A-Za-z0-9_]+$', $str)) {
  //error
}

[curry684]

modbreak: als je heel de topiclijst hier vol ziet staan met [taal] voor de topictitle ben je dan echt te lam om zelf ook even [PHP] ervoor te typen zodat het overzichtelijk blijft voor iedereen?

[Sybr_E-N]

Allereerst haal die " om je $gebruikersnaam weg. Deze regular expression die jij hier geeft doet precies wat jij wil, laat allen A-Z, a-z, 0-9 en _ toe. Dus mijn glazen bol zegt dan dat ergens ander een foutje in je code zit.

Maar stopt je script ook als iemand een verkeerde gebruikersnaam invuld? Of gaat de PHP parser leuk door met zn werk en voert dan wellicht misschien de code uit om aan te melden?

[maartenvdv737]

curry684 schreef op 06 juli 2003 @ 21:49:
<span style="color:blue">modbreak: als je heel de topiclijst hier vol ziet staan met [taal] voor de topictitle ben je dan echt te lam om zelf ook even [PHP] ervoor te typen zodat het overzichtelijk blijft voor iedereen? :/</span>

Sorry, even vergeten. Doe het altijd, check mijn history.


Het script stopt natuurlijk met een

PHP:

if(strlen($error_message) > 0){
  //do iets
  exit();
}

[Verwijderd]

Ik heb het even getest en met je regexp is in ieder geval niets mis. Is het niet zo dat je de error-check pas uitvoerd na dat er een transactie naar de database toe plaatsvindt?

[Johnny]

maartenvdv schreef op 06 July 2003 @ 22:54:
[...]

Sorry, even vergeten. Doe het altijd, check mijn history.


Het script stopt natuurlijk met een

PHP:

if(strlen($error_message) > 0){
  //do iets
  exit();
}

Dan is if(isset($error_message)) {} mooier

Misschien is het ook nog handig de lengte van de gebruikersnaam in de regex op te nemen op deze manier: [A-Za-z0-9_]+${1,25} gebruiken, dat zorgt ervoor dat de string minimaal 1 teken en maximaal 25 tekens mag hebben.