RegExp range out of order with complex pattern

Question

This is my pattern:

var pattern = "/(?:https?:\/\/)?(?:www\.)?facebook\.com\/(?:(?:\w)*#!\/)?(?:pages\/)?(?:[\w\-]*\/)*([\w\-\.]*)/";
var matches = $("#search input").val().match(new RegExp(pattern));

When I use it, it gives me an error:

Uncaught SyntaxError: Invalid regular expression: //(?:https?://)?(?:www.)?facebook.com/(?:(?:w)*#!/)?(?:pages/)?(?:[w-]*/)*([w-.]*)//: Range out of order in character class

From reading on another similar issues it came to my attention that I need to double escape some characters, but I don't know which out of all from my pattern.

how are you using this in your javascript? – SajithNair Mar 06 '14 at 10:04 — SajithNair, Mar 06 '14 at 10:04

score 6 · Accepted Answer · answered Mar 06 '14 at 10:05

6

Remove unwanted double quotes from regex pattern:

 var pattern = /(?:https?:\/\/)?(?:www\.)?facebook\.com\/(?:(?:\w)*#!\/)?(?:pages\/)?(?:[\w\-]*\/)*([\w\-\.]*)/;

answered Mar 06 '14 at 10:05

Vicky Gonsalves

11,593
2
37
58

If you're willing to wait until StackOverflow allows me to do so. – Kid Diamond Mar 06 '14 at 10:09
2

Additionally, then you don't need the `new RegExp(pattern)` anymore; just use `pattern`. – Scimonster Mar 06 '14 at 10:12

score 0 · Answer 2 · answered Mar 06 '14 at 10:07

0

In a JavaScript String all backslashes should be replaced by double backslash

var pattern = "/(?:https?:\\/\\/)?(?:www\\.)?facebook\\.com\\/(?:(?:\\w)*#!\\/)?(?:pages\\/)?(?:[\\w\\-]*\\/)*([\\w\\-\\.]*)/";

answered Mar 06 '14 at 10:07

SajithNair

3,867
1
17
23

score 0 · Answer 3 · answered Mar 06 '14 at 10:09

0

if its just about getting the xyz part from the url http://www.facebook.com/xyz

why not use split instead ?

something like this

var str = "http://www.facebook.com/xyz";
var res = str.split('/').pop();
console.log(res);

answered Mar 06 '14 at 10:09

aelor

10,892
3
32
48

It's not about just getting the xyz part from the URL. :) – Kid Diamond Mar 06 '14 at 10:14

RegExp range out of order with complex pattern

3 Answers3

Linked