Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Why is my javascript regexp test function not working?

I've got a Javascript function that tests a regexp that I'm intending to validate positive decimals up to a precision of two:

function isPositiveDecimalPrecisionTwo(str) {
    return /^\d*(\.\d{1,2}$)?/.test(str);
}

I'm not getting the results I'm expecting when I call it from code.

For example:

var a = isPositiveDecimalPrecisionTwo("1");         //   expect t, returns t
var b = isPositiveDecimalPrecisionTwo("1.2");       //   expect t, returns t
var c = isPositiveDecimalPrecisionTwo("1.25");      //   expect t, returns t
var d = isPositiveDecimalPrecisionTwo("1.257");     // * expect f, returns t *
var e = isPositiveDecimalPrecisionTwo("1.2575");    // * expect f, returns t *
var f = isPositiveDecimalPrecisionTwo(".25");       // * expect t, returns f *
var g = isPositiveDecimalPrecisionTwo("d");         //   expect f, returns f
var h = isPositiveDecimalPrecisionTwo("d1");        //   expect f, returns f
var i = isPositiveDecimalPrecisionTwo("1d");        // * expect f, returns t *
var j = isPositiveDecimalPrecisionTwo("1d1");       // * expect f, returns t *
var k = isPositiveDecimalPrecisionTwo("-1");        //   expect f, returns f
var l = isPositiveDecimalPrecisionTwo("-1.5");      //   expect f, returns f

There are three issues:

  1. Validating "1.257" or "1.2575" returns true, which I though would return false due to the \d{1,2}
  2. Validating ".25" returns false, which I thought would return true due to the ^\d*
  3. Validating "1d" or "1d1" return true. It looks like \. is reading as any character when it looks to me like it's a properly escaped "." (dot).

However, when I use a tool like regexpal.com, the same regexp appears to be validating the way I expect:

http://regexpal.com/?flags=g&regex=%5E%5Cd%2B(%5C.%5Cd%7B1%2C2%7D%24)%3F&input=

What am I missing?

like image 923
dfran02 Avatar asked Sep 02 '26 00:09

dfran02


1 Answers

The problem is that your $ is inside the parenthesis. You want it to be after the group.

/^\d*(\.\d{1,2})?$/

This should work the way you expect.

With the $ inside, it was matching everything that started with a number, since the end of string ($) was "optional".

like image 108
Rocket Hazmat Avatar answered Sep 03 '26 12:09

Rocket Hazmat



Donate For Us

If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!