Beefy Boxes and Bandwidth Generously Provided by pair Networks
Do you know where your variables are?

Re^2: Need a regex..

by littlemonk (Sexton)
on May 21, 2012 at 13:55 UTC ( #971616=note: print w/replies, xml ) Need Help??

in reply to Re: Need a regex..
in thread Need a regex..

it should match non-decimal value also ...example 12,123, and it should not match more than 3 digits before and after decimal..i have one regex(/^\- \+{0,1}0-9{0,3}(\.){0,1}0-9{0,3}\%{0,1}$/) but its accepting 1234(more than 3 digits before decimal) ... plz help me

Replies are listed 'Best First'.
Re^3: Need a regex..
by kennethk (Abbot) on May 21, 2012 at 14:33 UTC

    Please wrap code it <code> tags to keep things legible. As you can see, your character classes got linkified.

    1234 is passing because it's being read as (123)(4). You should set the whole part following and including the decimal in a non-capturing group with a conditional: /^[-+]?[0-9]{0,3}(?:\.[0-9]{0,3})?%?$/ I've also removed a bunch of unnecessary escaping, and changed your {0,1} to the shorter ? synonym.

    A read through of perlretut may be helpful.

    #11929 First ask yourself `How would I do this without a computer?' Then have the computer do it the same way.

Re^3: Need a regex..
by Neighbour (Friar) on May 21, 2012 at 14:40 UTC
    Edit: Lost the race to Re: Need a regex.., which is also slightly better :)
    Try a more grouped approach:
    First the start of the line: ^
    then an optional + or - sign, with the + enclosed in [] to avoid interpretation: ([+]|-)?
    then 0-3 digits: [0-9]{0,3}
    then an optional group of a period followed by 1-3 digits, with the period escaped in another way to avoid interpretation: (\.[0-9]{1,3})?
    and finally the end of the line: $

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://971616]
and all is quiet...

How do I use this? | Other CB clients
Other Users?
Others examining the Monastery: (6)
As of 2018-06-24 20:58 GMT
Find Nodes?
    Voting Booth?
    Should cpanminus be part of the standard Perl release?

    Results (126 votes). Check out past polls.