How to clean string from word characters, Javascript

Question

I'm trying to clean strings which has been transformed from word text but I'm stuck on removing special character '…'

By click on button "clean", script removes all dots and only one special character, however I need to remove all of them

Where is my mistake?

Here is my code and plunker with struggles

    $scope.string = "My transformed string &hellip;&hellip;&hellip;&hellip;.........&hellip;...."

 $scope.removeDots = function () {
        var em = document.getElementsByTagName('em');
        var reg = /\./g;
        var hellip = /&hellip;/g

        angular.forEach(em, function (item) {

            if(item.innerText.match(reg)){
                 item.innerText = process(item.innerText)
            } 
            if (item.innerText.match(hellip)){
              item.innerText = item.innerText.replace("&hellip;", "")
            }
        });
    };

    function process( str ) {
        return str.replace( /^([^.]*\.)(.*)$/, function ( a, b, c ) {
            return b + c.replace( /\./g, '' );
        });
    }

Instead of `replace("…", "")`, use the regex you have there: `replace(hellip, "")`, which is global. — trincot, Jan 20 '17 at 21:58
@antonyboom I was wrong. There's more going on here than I initially thought. — p.s.w.g, Jan 20 '17 at 22:01

score 2 · Accepted Answer · answered Jan 20 '17 at 22:04

There's a few problems here, but they can all be resolved by simply reducing the code to a single regex replace within process that will handle both periods and … entities:

 $scope.removeDots = function () {
        var em = document.getElementsByTagName('em');
        angular.forEach(em, function (item) {
              item.innerText = process(item.innerText)
        });
    };

    function process( str ) {
        return str.replace( /\.|&hellip;/g, '');
    }
});

Plunker demo

score 1 · Answer 2 · answered Jan 20 '17 at 22:05

1

You replace every occurrence of . in process, but only replace … once.

I don't see why don't you just do something like .replace(/(\.|…)/g, ''); the g modifier makes sure every match is replaced.

answered Jan 20 '17 at 22:05

9000

39,899
9
66
104

score 1 · Answer 3 · answered Jan 20 '17 at 22:05

You can do both replacements by first replacing the occurrences of … with one point (because it might be the only thing you find), and then replacing any sequence of points by one:

function process( str ) {
    return str.replace(/&hellip;/g, '.').replace(/\.\.+/g, '.');
}

var test="My transformed string &hellip;&hellip;&hellip;&hellip;.........&hellip;....";

console.log(process(test));

One of the reasons your code did not replace everything, is that you used a string as find argument, which will result in one replacement only. By using the regular expression as find argument you can get the effect of the g modifier.

You're welcome. Make sure you use a solution that will also work when your string is like `test…`, without any other point. You'll want to turn that into `test.` with the point. ;-) — trincot, Jan 20 '17 at 22:10

How to clean string from word characters, Javascript

3 Answers3