Replace multiple strings with Javascript

Question

I'm trying to transform this string

.jpg,.gif,.png

into this (not dots and space after comma)

jpg, gif, png

I thought that something like PHP's str_replace for arrays in JS will do the trick, so I found this post, and specifically this answer. I tried it but is't not working as expected. I'm getting a blank string... Am I doing something wrong?

JS

String.prototype.replaceArray = function(find, replace)
{
    var replaceString = this;
    var regex;

    for (var i = 0; i < find.length; i++)
    {
        regex = new RegExp(find[i], "g");
        replaceString = replaceString.replace(regex, replace[i]);
    }

    return replaceString;
};

var my_string = ".jpg,.gif,.png";

alert(my_string.replaceArray([".", ","],["", ", "]));

Link to jsfiddle

@Hovercraft: javascript!, I've must accepted the wrong tag suggestion withoud reading — Matías Cánepa
– Matías Cánepa, Commented Sep 28, 2015 at 0:43
Is there a practical reason to favour arrays and regexes over two basic replaces? — Jongware
– Jongware, Commented Sep 28, 2015 at 0:56
hwnd: I don't think this is really the same as the question you linked to. If I'm not mistaken, OP would like to understand why he's getting a blank string instead of the expected output. — rossipedia
– rossipedia, Commented Sep 28, 2015 at 1:01

rossipedia · Accepted Answer · 2015-09-28 00:59:45Z

3

The first thing you're trying to replace is a period ("."), which is a regular expression for any character. You need to escape it: "\\."

answered Sep 28, 2015 at 0:59

rossipedia

59.8k10 gold badges95 silver badges94 bronze badges

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

1 Comment

rossipedia Over a year ago

Yeah you need the second slash because javascript think you're escaping the period before it event gets to the RegExp object.

Nocturno · Accepted Answer · 2015-09-28 01:00:48Z

0

I just did this:

var target = '.jpg,.gif,.png';
target = target.replace(/\\./g, '');
target = target.replace(/,/g, ', ');

I'm sure it can be done more efficiently, but this will get the job done.

answered Sep 28, 2015 at 1:00

Nocturno

10.1k5 gold badges34 silver badges41 bronze badges

Comments

Amit.S · Accepted Answer · 2015-09-28 01:06:56Z

0

You can change your fn to this :

function strToArr(str)
{
     var res = str.replace(/\./g, "");
     return res.split(",");
}

answered Sep 28, 2015 at 1:06

Amit.S

4412 silver badges9 bronze badges

Collectives™ on Stack Overflow

Replace multiple strings with Javascript

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related