Regex test, есть ли лучший способ написать это?

Question

Следующий код работает, но мне интересно, если есть лучший способ достичь того же результата:

var regex = /define\((\s+)?['|"](PRIVATE_KEY)['|"],(\s+)?['|"](.*)['|"](\s+)?\)/i; 

Тест:

regex.test("define('PRIVATE_KEY', 'MYSECRETKEY');"); 

https://regex101.com/r/pW0qS0/4

Answer 1

Во-первых, я думаю, вы не 't нужно использовать pip в классе символов (если вы хотите совместить только одну цитату и двойную кавычку), а вместо (\s+)? вы можете использовать \s*:

/define\(\s*['"](PRIVATE_KEY)['"],\s*['"](.*)['"]\s*\)/i 

См демонстрационная https://regex101.com/r/dL1vF4/3

Answer 2

только фигурные скобки вокруг MYSECRETKEY части, потому что это часть, которую вы хотите захватить.

var regex = /^define\(\s*['"]PRIVATE_KEY['"]\s*,\s*['"](.*?)['"]\s*\);$/i;