Javascript Regex doesn't match with my String

Question

I have the following String :

var resultLine= "[UT] - GSM incoming call : STEP 1 - Simulate reception from server (1)Rerun3713 msAssertion ok"

And the following code which is responsible to check of the String matched with the Regex :

var resultRE = /^([ \w-]*: )?(.+) \((\d+), (\d+), (\d+)\)Rerun/;
var resultMatch = resultLine.match(resultRE);
if (resultMatch) {
   return true;
} else {
   return false;
}

In this case, i have an error in my Regex because i always get "false". Where is my mistake ?

Your string bears almost no resemblance to the pattern. Why would you ever expect that to match? — p.s.w.g
– p.s.w.g, Commented Sep 24, 2014 at 18:20
For starters, it appears that you are trying to find something with (NUM, NUM, NUM)Rerun in it, but your source string has only (NUM)Rerun. — abiessu
– abiessu, Commented Sep 24, 2014 at 18:20
It helps if you explain what exactly the pattern is supposed to be. — Matt Burland
– Matt Burland, Commented Sep 24, 2014 at 18:20

abiessu · Accepted Answer · 2014-09-24 18:55:31Z

1

I would recommend the following pattern based on what it appears you are looking for:

var resultRE = /^([\[ \w\]-]*: )(.+) $([0-9, ]*)$Rerun(.*)$/

This should force all capture groups to exist, even if they are empty, and will allow for multiple numbers before Rerun as you seem to expect.

edited Sep 24, 2014 at 18:55

answered Sep 24, 2014 at 18:29

abiessu

1,9471 gold badge18 silver badges16 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Gary Over a year ago

In your case you really do need to escape the dash in the character class (since it's being interpreted as a range here). Since the OP was so vague, I don't know if it matters, but the first group will match things like "UT[[][]". However, this fits what was provided.

Gary · Accepted Answer · 2014-09-24 18:58:12Z

1

This matches nothing in your string

([ \w-]*: )?

Since it was optional, that doesn't matter because it gets caught by the all inclusive

(.+)

If you were trying to match the [UT] part with it's separator, it would look something like this

(\[\w+\][\s\-]*)?

As noted in the comments, you only have one number in parentheses but your regex requires three sets of them, separated by commas. This will allow any number of numbers, separated by commas indefinitely (I don't know if there's a limit or not).

\((\d+,\s)*(\d+)\)

If you need something more specific, you'll have to be more specific about what template your matching, not a specific case. But the best I can figure with what you've provided is

^(\[\w\][\s\-]*)?(.+)\((\d+,\w)*(\d+)\)Rerun

edited Sep 24, 2014 at 18:58

answered Sep 24, 2014 at 18:29

Gary

14k18 gold badges54 silver badges73 bronze badges

4 Comments

abiessu Over a year ago

I think your second regexp might need to have the dash and optional spacing outside the escaped [ and ], if I'm reading the source line correctly...

Gary Over a year ago

I don't think the dash needs to be escaped being on the end but at the very least, you're right it should be. As far as the brackets, they're escaped, I just forgot to remove the colon. Thank for the input

abiessu Over a year ago

I just meant that the dash in the line to be matched appears outside the [ and ] surrounding the UT portion... Maybe something like (\[\w\]\s*-)??

Gary Over a year ago

Oh, I see what you mean. I didn't notice why the dash was there to begin with. I guess it really wouldn't make sense to allow the line to start with " - ".

w35l3y · Accepted Answer · 2014-09-24 18:28:28Z

0

var resultRE = /\((\d+)(?:, (\d+))?(?:, (\d+))?\)Rerun/;
if (resultRE.test(resultLine)) {
  var num1 = RegExp.$1,
  num2 = RegExp.$2,
  num3 = RegExp.$3;
}

answered Sep 24, 2014 at 18:28

w35l3y

8,8333 gold badges41 silver badges53 bronze badges

Collectives™ on Stack Overflow

Javascript Regex doesn't match with my String

3 Answers 3

1 Comment

4 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

4 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related