js regex replace multiple words

Question

I need to replace ${conferance name} with ABC, ${conference day} with Monday in the following sentence. Could some one help me with the regex.

var text = "<td>${conference name}</td><td>${conference day}</td>"

var list = ["${conferance name}", "${conference day}" ]

for (var j = 1; j < list.length; j++) {
            //Extracting the col name
            var colName  =   list[j].split("${");
            colName = colName.split("}")[0];
            //Replacing the col name
            text = text.replace(new RegExp('\\$\\{' + colName + '\\}', 'g'), "ABC");

        }

The above code repalces fine if i have ${conference_name}, but it fails when i have a space in between.

The list is a dynamic array. And the Replace texts are also dynamic. I just simulated them as objects here for fitting them in the Regex Statement.

Thanks in Advance.

Pardon me for the syntax errors. It could be a typo. But this is teh logic that i use in a bigger chunk of code. — Raghav
– Raghav, Commented Dec 19, 2012 at 16:57
Also, there is no syntax error on the Regex, which is the core answer i want — Raghav
– Raghav, Commented Dec 19, 2012 at 16:57

Marcus Mendes · Accepted Answer · 2014-05-15 13:11:27Z

2

Here's a version that lets you parameterize the replacements explicitly:

function doReplace(repl, str) {
  var regexStr = Object.keys(repl).map(function(s) {
    return s.replace(/([^\w\s])/g, '\\$1');
  }).join('|');
  return str.replace(new RegExp(regexStr, 'g'), function(m) {
    return repl[m];//it changed to "repl"  
  });
}

var replacements = {
  "${conference name}": "ABC",
  "${conference day}": "Monday"
};

doReplace(replacements, text) // => "<td>ABC</td><td>Monday</td>"

edited May 15, 2014 at 13:11

Marcus Mendes

33 bronze badges

answered Dec 19, 2012 at 17:29

maerics

157k47 gold badges277 silver badges299 bronze badges

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

Comments

VisioN · Accepted Answer · 2012-12-19 17:14:18Z

1

How about that?

var r = {
        "conference name": "ABC",
        "conference day" : "Monday"
    },
    reg = new RegExp("\\$\\{" + Object.keys(r).join("\\}|\\$\\{") + "\\}", "g"),
    str = "<td>${conference name}</td><td>${conference day}</td>";

str = str.replace(reg, function(v) {
    return r[v.substring(2, v.length - 1)];
});

answered Dec 19, 2012 at 17:14

VisioN

146k35 gold badges287 silver badges291 bronze badges

Comments

Niet the Dark Absol · Accepted Answer · 2012-12-19 17:19:59Z

1

There is absolutely no need for a regex, since you know the exact string to search for.

var str = "<td>${conference name}</td><td>${conference day}</td>";
var list = {
  "conference name":"ABC",
  "conference day":"Monday"
};
var i;
for( i in list) {
  if( list.hasOwnProperty(i)) {
    str = str.replace("${"+i+"}",list[i]);
  }
}

The only catch is if you might have more than one of the placeholders in your search string, but in that case the innermost code can be:

do {
  oldstr = str;
  str = str.replace("${"+i+"}",list[i]);
} while(oldstr != str);

edited Dec 19, 2012 at 17:19

answered Dec 19, 2012 at 16:56

Niet the Dark Absol

326k86 gold badges480 silver badges604 bronze badges

9 Comments

VisioN Over a year ago

str.replace("${" + i + "}", list[i]);

Raghav Over a year ago

this would repalce only one instance. I went in for the regex, coz i need to replace all instaces in the text.

VisioN Over a year ago

@Raghav Use this: str.replace(new RegExp("${" + i + "}", "g"), list[i]);.

Raghav Over a year ago

sorry @Vision this is not replacing any text

Niet the Dark Absol Over a year ago

Because for regexes to work you have to make sure you escape every single symbol with regex meaning. Here, though, any string is fine.

|

Community · Accepted Answer · 2020-06-20 09:12:55Z

1

    var str = "<td>${conference name}</td><td>${conference day}</td>";
    var list = ["${conference name}", "${conference day}" ];
    var listResult = ["ABC", "Monday" ];

Generic model

See it working http://jsbin.com/atehun/6/watch

Construct str, list, listResult as shown in above example.

function genericReplace(str, list, listResult) {    

    var escape = function(text) {
        return text.replace(/[-[\]{}()*+?.,\\^$|#\s]/g, "\\$&");
    };
    
    var escList = [];
    for (var i=0; i < list.length; i++) {
      escList.push( escape(list[i]) );
    }
    
    var regEx = new RegExp(escList.join('|'), "g");

    return str.replace(regEx, function(word) {
      return listResult[list.indexOf(word)];
    });
}

edited Jun 20, 2020 at 9:12

CommunityBot

11 silver badge

answered Dec 19, 2012 at 17:02

closure

7,4521 gold badge25 silver badges23 bronze badges

2 Comments

Raghav Over a year ago

Thanks. Can u pls do the replace with list[i]? list being string array. list[i] = "${conference name}"

closure Over a year ago

Edited! Also see the fiddle!

Collectives™ on Stack Overflow

js regex replace multiple words

4 Answers 4

Comments

Comments

9 Comments

Generic model

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

9 Comments

Generic model

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related