preg match compilation error

Do you realy want to use [ and ] as patern delimiters? My quess is that
you want to use it for defining character class. Then the only character
that must be escaped is -

'/[^a-z0-9,._+()\-]/i'

Luis Lebron wrote:
> I found a reqular expresion that I'm using to check for valid characters in
> file names.
>
> Here's the sample code:
>
> if(preg_match("[^a-zA-Z0-9,._\+\()\-]",$filename))
> {
> //blah, blah, blah
> }
>
> However, when I run the script I get the following error:
> Warning: Compilation failed: unmatched parentheses at offset 17 in
> /home/httpd/sigmarapid/html/test/upload.php on line 137 (the line with the
> preg_match)
>
> I'm new to regular expressions so any help would be greatly appreciated.
>
> Luis R. Lebron
> Sigmatech, Inc
>

I found a reqular expresion that I'm using to check for valid characters in
file names.

Here's the sample code:

if(preg_match("[^a-zA-Z0-9,._\+\()\-]",$filename))
{
//blah, blah, blah
}

However, when I run the script I get the following error:
Warning: Compilation failed: unmatched parentheses at offset 17 in
/home/httpd/sigmarapid/html/test/upload.php on line 137 (the line with the
preg_match)

I'm new to regular expressions so any help would be greatly appreciated.

Luis R. Lebron
Sigmatech, Inc

Start and end with same char, for example with a / or a #... Like
if(preg_match("#[^a-zA-Z0-9,._\+\()\-]#",$filename))


--
// DvDmanDT
MSN: dvdmandt¤hotmail.com
Mail: dvdmandt¤telia.com
##########################
Please, if you are using windows, you may be infected by Swen. Please go
here to find out more:
http://us.mcafee.com/virusInfo/defau...er&hcName=swen
http://securityresponse.symantec.com...swen.a@mm.html
##########################
"Luis Lebron" <llebron@sigmatech.com> skrev i meddelandet
news:CE5C3E60DCF0D5118C7F00A0C9EBCE23041550@SIGMA2 ...
> I found a reqular expresion that I'm using to check for valid characters
in
> file names.
>
> Here's the sample code:
>
> if(preg_match("[^a-zA-Z0-9,._\+\()\-]",$filename))
> {
> //blah, blah, blah
> }
>
> However, when I run the script I get the following error:
> Warning: Compilation failed: unmatched parentheses at offset 17 in
> /home/httpd/sigmarapid/html/test/upload.php on line 137 (the line with the
> preg_match)
>
> I'm new to regular expressions so any help would be greatly appreciated.
>
> Luis R. Lebron
> Sigmatech, Inc
>

* Thus wrote Luis Lebron (llebron@sigmatech.com):
> I found a reqular expresion that I'm using to check for valid characters in
> file names.
>
> Here's the sample code:
>
> if(preg_match("[^a-zA-Z0-9,._\+\()\-]",$filename))
> {
> //blah, blah, blah
> }

preg_* need a pattern deliminater (most commonly /,#, or !) so it should be:

preg_match("/[^a-zA-Z0-9,._\+\()\-]/",$filename)

Now you can get rid of all the escapes because mosts characters loose
their special meaning inside a character class:

preg_match("/[^a-zA-Z0-9,._+()-]/",$filename)

You can get away without escaping the '-' because it *can't* be
considered a range of characters since it is last (or if its
first).


That should work.


Curt
--
"My PHP key is worn out"

PHP List stats since 1997:
http://zirzow.dyndns.org/html/mlists/

* Thus wrote Marek Kilimajer (kilimajer@webglobe.sk):
> Do you realy want to use [ and ] as patern delimiters? My quess is that
> you want to use it for defining character class. Then the only character
> that must be escaped is -
>
> '/[^a-z0-9,._+()\-]/i'

actually, you don't even need that escape. It has the exception of
being the range character if it is either the first or last
character in the class.


Curt
--
"My PHP key is worn out"

PHP List stats since 1997:
http://zirzow.dyndns.org/html/mlists/