Replacing array elements using regex and replace() in javascript

Question

I'm bit new to JavaScript, I'm trying to replacing the array element using regex that matches the string, here is a code which I tried

<button onclick="myFunction()">ClickHere</button>
<p id="demo"></p>
<script>
function myFunction() {
    var abc = ["deno", "eno","pqr","lenovo"];
    var i,text;
    for(i = 0; i < abc.length; i++) {
        text += abc[i].replace(/no/i, "po");
        document.getElementById("demo").innerHTML = text;
    }
}
</script>

I want to replace array element with "po" wherever it encounters "no" in the array element string.

This is what I expect:

abc["depo","epo","pqr","lepovo"]

Expect array or string in output ? You have a typo as , in loop..jsfiddle.net/pkjsdu9w — Rayon
– Rayon, Commented Mar 31, 2016 at 7:19

ROMANIA_engineer · Accepted Answer · 2016-03-31 07:45:44Z

1

You can do this for every element:

for(var i=0; i < abc.length; i++) {
    abc[i] = abc[i].replace('no', 'po');
}

or using one line

abc = abc.map(function(x){return x.replace('no', 'po');});

or using one line with "arrow functions":

abc = abc.map(x => x.replace('no', 'po'));

After you changed the array, you can convert it to a string using:

var text = 'abc['; 

for ( var i = 0 ; i < abc.length ; i++ ) {
    text+='\"'+abc[i]+'\"';
    if ( i != abc.length - 1) {
        text+=',';
    }
}
text += ']';

Test:

function myFunction() {
    var abc = ["deno", "eno","pqr","lenovo"];
    abc = abc.map(x => x.replace('no', 'po')); // see other 2 alternatives above

    var text = 'abc['; 

    for ( var i = 0 ; i < abc.length ; i++ ) {
        text+='\"'+abc[i]+'\"';
        if ( i != abc.length - 1) {
            text+=',';
        }
    }
    text += ']';
    document.getElementById("demo").innerHTML = text;
}

<button onclick="myFunction()">ClickHere</button>
<p id="demo"></p>

edited Mar 31, 2016 at 7:45

answered Mar 31, 2016 at 7:17

ROMANIA_engineer

57k30 gold badges211 silver badges207 bronze badges

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

2 Comments

Andre_k Over a year ago

After using your first solution ,its giving me this output denodepo,enoepo,pqrpqr,lenovolepovo which I think its taking the element first printing and then doing the replacement

ROMANIA_engineer Over a year ago

@deepesh , I added the conversion to text, where I don't use replace anymore, because I've already changed the array. You can also see a snippet integrating all those parts. Click on Run code snippet to test it.

Gibbs · Accepted Answer · 2016-03-31 07:18:44Z

0

var i, text;
for(i = 0; i < abc.length, i++) {
  text += abc[i].replace("no", "po");
}
  console.log(text);

answered Mar 31, 2016 at 7:18

Gibbs

23.1k14 gold badges84 silver badges147 bronze badges

Comments

kamal · Accepted Answer · 2016-03-31 07:29:54Z

0

There are three changes required in your code:

Initialiize text with empty string.Because it is undefined by default.
Change abc[i].length to abc.length.

Replace comma with a semicolon after abc[i].length in for loop.

var abc = ["deno", "eno","pqr","lenovo"];
var i;
var text = "";
for(i = 0; i < abc.length; i++) {
   text += abc[i].replace("no", "po");
}

edited Mar 31, 2016 at 7:29

answered Mar 31, 2016 at 7:23

kamal

841 silver badge6 bronze badges

2 Comments

Andre_k Over a year ago

To your solution this is the answer I got depoepopqrlepovo

Andre_k Over a year ago

I tried using "document.getElementById("demo").innerHTML = text + "<br>"; " to break line but it didnt gave the desired result

Collectives™ on Stack Overflow

Replacing array elements using regex and replace() in javascript

3 Answers 3

Test:

2 Comments

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Test:

2 Comments

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related