[PHP] ereg_replace probleempje

[Thunk]

Hey, voor een $voornaam wil ik alleen de volgende karakters toestaan in een string:

a-z, A-Z, spatie en - (streepje)

De volgende functie werkt niet helemaal naar behoren:

PHP:

$voornaam = trim(ereg_replace("[^a-z,A-Z,-,[:space:]]","",$_POST['voornaam']));

a-z, A-Z, en spatie worden goed weergegeven.
Alleen haalt PHP nog altijd het streepje (-) weg uit mijn string.
Kan iemand me vertellen wat ik nog over het hoofd zie?

[ Voor 6% gewijzigd door Thunk op 01-02-2006 16:54 ]

[arieleks]

Had je al geprobeerd om een \ (backslash) voor dat streepje te zetten?

PHP:

$voornaam = trim(ereg_replace("[^a-z,A-Z,\-,[:space:]]","",$_POST['voornaam']));

[ Voor 41% gewijzigd door arieleks op 01-02-2006 16:57 ]

[NMe]

Die komma's in je character class slaan nergens op; je class matcht nu ook komma's. Daarnaast vind ik een char class binnen een char class ook nogal raar overkomen, maar ik ben niet bekend met POSIX, dus misschien moet dat inderdaad wel zo. Wat jij doet zou ik persoonlijk zo aanpakken:

PHP:

$voornaam = preg_replace("/[^a-z\s-]/i","",$_POST['voornaam']);

Die trim slaat natuurlijk ook nergens op aangezien je regexp al ervoor zorgt dat er geen spaties meer in staan, dus ook niet aan het begin of einde van je string.

[MrNGm]

Volgens de Pattern Syntax in de PHP docs (weliswaar Perl-compatible regex) staat dat het streepje '-' een karakter reeks aanduidt. Probeer anders het volgende:

$voornaam = trim(ereg_replace("[^a-z,A-Z,\-,[:space:]]","",$_POST['voornaam']));

Misschien is het ook beter om Perl-compatible (dus preg_replace) regexen te gebruiken.

*ik zie nu dat een aantal mensen al sneller zijn

[arieleks]

offtopic:
Persoonlijk zou ik voor de tip van -NMe- gaan. Die werkt namelijk gewoon. Die van mij niet, maar dat komt omdat mijn hoofd is vervuild met VB6 en VBA. Tsja, 't is maar waar je je geld mee verdient...

[Thunk]

Bedankt voor de reacties.. zat al de hele middag te prutsen met mogelijke werkende combinaties.
Een \ voor het streepje werkt ook niet (Warning: ereg_replace() [function.ereg-replace]: REG_ERANGE)

Maar, de oplossing van -NMe- werkt perfect.. thanks!

[Thunk]

-NMe- schreef op woensdag 01 februari 2006 @ 16:57:Die trim slaat natuurlijk ook nergens op aangezien je regexp al ervoor zorgt dat er geen spaties meer in staan, dus ook niet aan het begin of einde van je string.

Die trim sloeg wel ergens op, want die verwijderde de spaties aan het begin en eind van de string.
De regexp die ik er op losliet, zorgde ervoor dat er WEL spaties BINNEN de string waren toegestaan (en dat werkte ook prima).

[NMe]

Thunk schreef op woensdag 01 februari 2006 @ 17:26:
Die trim sloeg wel ergens op, want die verwijderde de spaties aan het begin en eind van de string.
De regexp die ik er op losliet, zorgde ervoor dat er WEL spaties BINNEN de string waren toegestaan (en dat werkte ook prima).

Ow? De regexp die je zelf postte zou, als je het streepje en de overbodige komma's even wegdenkt, het volgende doen:

"Jan Klaassen" -> "JanKlaassen"
"  Pietje" -> "Pietje"
"   " -> ""

Enz.

Zoals je die regexp hier postte heb je die trim simpelweg niet nodig.

[Thunk]

-NMe- schreef op woensdag 01 februari 2006 @ 17:31:
[...]

Ow? De regexp die je zelf postte zou, als je het streepje en de overbodige komma's even wegdenkt, het volgende doen:

"Jan Klaassen" -> "JanKlaassen"
"  Pietje" -> "Pietje"
"   " -> ""

Enz.

Zoals je die regexp hier postte heb je die trim simpelweg niet nodig.

Volgens mij wel, want het volgende stukje

code:

"[^a-z,A-Z,-,[:space:]]"

Zorgt er voor dat alle expressies die er tussen de [ ] staan, WEL toegestaan zijn ( door ^ )... (dus a-z, A-Z, - (dat niet werkte), en spatie, de rest wordt weggefilterd)
Als je dan zonder trim zou werken, zouden er ook spaties voor en na de string worden doorgelaten.. en de trim haalt die juist weg (?!)

[NMe]

Ow, inderdaad, je hebt gelijk... Ik kijk weer eens niet goed.

[Thunk]

;-)
Thanks for the help tho