RegExp constructor in a switch statement

Question

I have some regex patterns in an object literal which I would like to switch over when I look for certain patters like security numbers.

Why do I get a match on abc below?

Simplified version of my code:

var cst = {
    SECURITYNUMBER: /^\d{12}$/
};

function doStuff(query){

    $.each(query.split(' '), function(i, word){

        switch(true){
            case new RegExp(cst.SECIRUTYNUMBER).test(word): 
                console.log('match');
                break;
        }

    });
}

doStuff('198610054937 abc');

http://jsfiddle.net/t1ps0vdh/

freefaller · Accepted Answer · 2014-09-22 09:48:08Z

1

You've got a typo... instead of using SECURITYNUMBER, you've used SECIRUTYNUMBER.

Change...

case new RegExp(cst.SECIRUTYNUMBER).test(word):

To...

case new RegExp(cst.SECURITYNUMBER).test(word):

http://jsfiddle.net/t1ps0vdh/1/

answered Sep 22, 2014 at 9:48

freefaller

20.1k7 gold badges62 silver badges96 bronze badges

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

1 Comment

freefaller Over a year ago

I know the feeling @Johan, just needed a 2nd pair of eyes

Collectives™ on Stack Overflow

RegExp constructor in a switch statement

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related