JavaScript match and replace; what is the best way?

Question

So I've already got this working but I'm hoping someone can help me optimize it.

I have a ton of XML data that I'm grabbing and do not have control over. One of the tags has markup in it and instead of using >, < and " it uses the HTML names. I've setup a really simple function to return the string with the proper elements so that I can write it to the page.

I'm just wondering, can my cleanFeedDescription() function be done a little cleaner and more optimized?

<script type="text/javascript">
        var textString = '&lt;img src=&quot;/media/filter/small/transfer/a0/46/21ca7da7/3569/1774/cf3f/82634fc4/img/4th_of_july_209.jpg&quot; alt=&quot;&quot; /&gt; &lt;p class=&quot;long-description&quot;&gt;&lt;span class=&quot;uploaded-date&quot;&gt;Uploaded 2010-11-29 18:24:24&lt;/span&gt;&lt;span class=&quot;uploaded-by&quot;&gt;by gin_alvizo&lt;/span&gt;&lt;span class=&quot;uploaded-to&quot;&gt;part of Bull&#039;s-Eye Bold BBQ powered by ...&lt;/span&gt;&lt;/p&gt;';

        document.write(cleanFeedDescription(textString));

        function cleanFeedDescription(descString) {

            descString = descString.replace(/&lt;/g, '<');
            descString = descString.replace(/&gt;/g, '>');
            descString = descString.replace(/&quot;/g, '"');

            return descString;
        };
    </script>

score 2 · Answer 1 · edited May 23 '17 at 12:11

2

Here is a handy JS library that does the HTML entity encoding/decoding for you, no need to reinvent the wheel.

Alternately, if you're using jQuery, there's a similar question on SO with a jQuery-centric solution.

edited May 23 '17 at 12:11

Community

1
1

answered Dec 17 '10 at 18:59

Daniel DiPaolo

55,313
14
116
115

Vincent Robert · Accepted Answer · 2010-12-18T14:28:34.167

2

I suggest you to use only one replace with a function instead of many, this will be faster and easier to maintain.

function cleanFeedDescription(descString) 
{
    var replacements = { lt: '<', gt: '>', quot: '"' };

    return descString.replace(/&(\w+);/g, function(str, entity)
    {
        return replacements[entity] || "";
    });
};

edited Dec 18 '10 at 14:28

answered Dec 17 '10 at 19:08

Vincent Robert

35,564
14
82
119

This seems like a much more simple and cleaner solution than concatenating a bunch of .replace() methods or using bloated third party scripts to obtain my desired result. Thanks! p.s. I needed to modify your regExp... added "g" to the end of it! :) – Smccullough Dec 17 '10 at 19:21

score 0 · Answer 3 · answered Dec 17 '10 at 19:08

0

Well you don't really need to assign intermediate values to descString, so you could do it like this...

return descString.replace(/&lt;/g, '<')
                 .replace(/&gt;/g, '>')
                 .replace(/&quot;/g, '"');

answered Dec 17 '10 at 19:08

MooGoo

46,796
4
39
32

JavaScript match and replace; what is the best way?

3 Answers3