Regex match error in javascript

Question

I have seen Return positions of a regex match() in Javascript? but not sure how to implement it in my case

I have an array item_array and I want to filter all items containing text val

This is the code I'm using :

I have made a function matchdata to filter the results

var matchdata = function(text_to_match, item_array) {
    var reg = new RegExp(text_to_match.split('').join('\\w*').replace(/\W/, ""), 'i');
    return item_array.filter(function(item) {
        if (item.match(reg)) {
            return item;
        }
    });
};

And I'm using it like

var matching_items = matchdata(val, item_array);

Now I have 2 questions :

When val ( item to find in array ) contains more than one \, it starts giving this error in console :

Uncaught SyntaxError: Invalid regular expression: /\w*\w*/: \ at end of pattern(…)

Why is this happening ? and is there any way to rectify this ?

Is there any way the filter function can return the matched item index along with the matched item ( assuming there can be duplicate entries in the array )

When text_to_match = "\\\\", your split & join will try to create an invalid regex — Washington Guedes
– Washington Guedes, Commented Nov 16, 2016 at 16:34
It would be more informative if you can provide example inputfr for text_to_match and items_array. — nu11p01n73R
– nu11p01n73R, Commented Nov 16, 2016 at 16:37
@nu11p01n73R : you can see this jsfiddle.net/pjtnpLn7/3 for an example — mrid
– mrid, Commented Nov 16, 2016 at 16:43
@Guedes : thanks for informing, do you have a solution for this ? and by index, i mean the array index of the matched item — mrid
– mrid, Commented Nov 16, 2016 at 16:44

Washington Guedes · Accepted Answer · 2016-11-16 17:15:09Z

1

This function is your 'regex idea' coded in a for-loop:

function isSubstring (input, text) {
    input = input.toLowerCase();
    text = text.toLowerCase();
    var found = 0;
    var nextChar = input.charAt(found);
    for (var i=0, l=text.length; i<l; i++) {
        if (text.charAt(i) === nextChar) {
            found++;
            if (found === input.length)
                return true;
            nextChar = input.charAt(found);
        }
    }
};

Basically, it will look for a substring with zero or more characters between each matched letter:

isSubstring('BZL', 'Brazil');  // true
//                  ^  ^ ^

Then, your code will look like:

var matchdata = function(text_to_match, item_array) {
    var result = item_array.map(function(item, index) {
        if (isSubstring(text_to_match, item)) {
            return [item, index];
        }
        return 0;
    });
    return result.filter(isNaN);
};

The result will be a matrix:

matchdata('bzl', ['BraZiL', 'LiZarB', 'BraZiL']);
// [ ['brazil', 0], ['brazil', 2] ]

Hope it helps :)

answered Nov 16, 2016 at 17:15

Washington Guedes

4,3753 gold badges35 silver badges59 bronze badges

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

2 Comments

mrid Over a year ago

what does result.filter(isNaN) do ?

Washington Guedes Over a year ago

The map function returns 0 when isSubstring(text_to_match, item) is not true. Since isNaN(0) === false, you can use isNaN to filter off the zeroes.

Collectives™ on Stack Overflow

Regex match error in javascript

1 Answer 1

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related