Regex match javascript/php function name in str

Question

I'm trying to highlight code (and eventually sanitize the HTML) but my regex is not matching just the function name and params. I am no good at regex, it kinda boggles me to begin with. Also when I try to use .replace() on my matched result to sanitize the HTML and add the <pre> brackets, it gives me the error Uncaught TypeError: undefined is not a function which I am guessing is because it's not returning a basic string?

var content = $('#content'),
    html = content.html(),
    result = html.replace(/\s.*\(.*\)\s/gi, "<pre>$&</pre>");

    // When trying to add the <pre> tags in the last line of code
    // And use this to sanitize my html.match() I get a error

    // escapedRes = result.replace(/&/g, "&amp;")
    //           .replace(/</g, "&lt;")
    //           .replace(/>/g, "&gt;")
    //           .replace(/"/g, "&quot;")
    //           .replace(/'/g, "&#039;");

    // Uncaught TypeError: undefined is not a function 

content.html(result);

JSFiddle Example

Broken Sanitize Fiddle

var content = $('#content'),
    html = content.html(),
    result = html.match(/\w+\(.+?\)/g);
var escapedRes = result.replace(/&/g, "&amp;")
    .replace(/</g, "&lt;")
    .replace(/>/g, "&gt;")
    .replace(/"/g, "&quot;")
    .replace(/'/g, "&#039;")
    .replace(/(/g, "&#40;")
    .replace(/)/g, "&#41;")
    .replace(/\*/g, "&#42;")
    .replace(/$/g, "&#36;");
var result = escapedRes.replace(result, '<pre>'+escapedRes+'</pre>');
content.html(result);

JSFiddle Example

Amit Joki · Accepted Answer · 2014-04-20 08:25:48Z

3

Use this regex:

/\w+\(.+?\)/g

DEMO

Update:

On your sanitizing part, you need to do

 result = html.match(/\w+\(.+?\)/g)[0];

as match() returns an array.

Also, you'll need to escape ( and ) and $ with backslash, as they have special meaning in regex.

edited Apr 20, 2014 at 8:25

answered Apr 20, 2014 at 7:07

Amit Joki

59.4k7 gold badges80 silver badges96 bronze badges

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

10 Comments

WASasquatch Over a year ago

Thanks, that's part of the problem.

Amit Joki Over a year ago

@WASasquatch, what is the other part?

WASasquatch Over a year ago

It's in my question. I updated code to show problematic code. Which is based on using a match() in result insidead of replace

WASasquatch Over a year ago

Please re-read my comment and question. This is based on the html.match(/\w+$.+?$/g) code as the <pre> is added afterwards.

Amit Joki Over a year ago

@WASasquatch, .maatch() will only give you the matched part which is function(){ ... }. Why are you using that?

|

Collectives™ on Stack Overflow

Regex match javascript/php function name in str

Broken Sanitize Fiddle

1 Answer 1

Update:

10 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

Broken Sanitize Fiddle

1 Answer 1

Update:

10 Comments

Your Answer

Sign up or log in

Post as a guest

Related