Regexp, capture between parentheses, javascript

Question

I have regexp that extracts values between parentheses.

It's working most of the time but not when it ends with a parentheses

var val = 'STR("ABC(t)")';
var regExp = /\(([^)]+)\)/;.
var matches = regExp.exec(val);
    
console.log(matches[1]); //"ABC(t"

What I want is "ABC(t)".

Any ideas how I can modify my regexp to Achive this?

Update The value is always inside the parentheses.

Some examples:

'ASD("123")'; => '123'
'ASD(123)'; => '123'
'ASD(aa(10)asda(459))'; => 'aa(10)asda(459)'

So first there is some text (always text). Then there is a (, and it always ends with a ). I want the value between.

You escaped the wrong closing ), so your capturing group ends and then you match the ) (when it should be the other way around). Voting off-topic because the problem is a typographic error. — Quentin
– Quentin, Commented Oct 27, 2016 at 8:20
Unless there are specific restrictions on the input, this is pretty tough to solve with a regex. Depending on what is allowable in the input and what isn't, only an actual parser may be able to do this properly. — deceze
– deceze ♦, Commented Oct 27, 2016 at 8:20
what should be matched in this case 'some text (num(10a ) ss) STR("ABC(t)")' ? — RomanPerekhrest
– RomanPerekhrest, Commented Oct 27, 2016 at 8:20
If the string is not part of a longer string, /\((.+)\)/ will do. More details about the input are necessary. — Wiktor Stribiżew
– Wiktor Stribiżew, Commented Oct 27, 2016 at 8:23

Wiktor Stribiżew · Accepted Answer · 2016-10-27 08:49:53Z

2

You may use greedy dot matching inside Group 1 pattern: /\((.+)\)/. It will match the first (, then any 1+ chars other than linebreak symbols and then the last ) in the line.

var vals = ['STR("ABC(t)")', 'ASD("123")', 'ASD(123)', 'ASD(aa(10)asda(459))'];
var regExp = /\((.+)\)/;
for (var val of vals) {
  var matches = regExp.exec(val);
  console.log(val, "=>", matches[1]);
}

Answering the comment: If the texts to extract must be inside nested balanced parentheses, either a small parsing code, or XRegExp#matchRecursive can help. Since there are lots of parsing codes around on SO, I will provide XRegExp example:

var str = 'some text (num(10a ) ss) STR("ABC(t)")';
var res = XRegExp.matchRecursive(str, '\\(', '\\)', 'g');
console.log(res);

<script src="https://cdnjs.cloudflare.com/ajax/libs/xregexp/2.0.0/xregexp-all-min.js"></script>

edited Oct 27, 2016 at 8:49

answered Oct 27, 2016 at 8:36

Wiktor Stribiżew

631k41 gold badges502 silver badges632 bronze badges

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

3 Comments

GreenAsJade Over a year ago

You should include some text (num(10a ) ss) STR("ABC(t)") in the test data set ;) :)

Wiktor Stribiżew Over a year ago

@GreenAsJade: No, OP has no such texts. Anyway, I can add a solution for this string, too. Need it?

GreenAsJade Over a year ago

I was just joking about the question in comments. The question in comments is designed to illustrate whether or not the regex is intended to parse matching parens, which I believe it is not.

Collectives™ on Stack Overflow

Regexp, capture between parentheses, javascript

1 Answer 1

3 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related