Javascript Replacing Spaces within an Array

Question

I'm ok at PHP, but no nothing about Javascript, so I have no idea how to proceed here. I'm trying to replace spaces with a "+" in line 3. Anybody know why it's not working? Thanks!

var tn_dv_suggestions = new Array();
for(var tn_counter=0; tn_counter < tn_top_performers.length; tn_counter++)
tn_top_performers[tn_counter]=tn_top_performers[tn_counter].replace(" ","+");
tn_dv_suggestions.push("<a style='font-family: Verdana, Arial; font-size: 14px;' target='_blank' href='http://www.<?=$siterow['Domain']?>/Buy-"+escape(tn_top_performers[tn_counter]) +"-<?=urlencode($siterow['CitySearchName'])?>-Tickets' >"+tn_top_performers[tn_counter] +"</a><br />");
document.getElementById('tn_dv_suggestions089hZ').innerHTML=tn_dv_suggestions.join('');

Can you give us some example input (values in tn_top_performers before the loop) and output (values in tn_top_performers after the loop)? — Greg
– Greg, Commented Mar 28, 2011 at 17:01

Mike Lewis · Accepted Answer · 2011-03-28 17:02:43Z

3

Here is a solution using array.map:

var replaceInArray = function(str){
  return str.replace(/\s+/g, "+")
}

var arr = ["Summer is Great", "Winter is terrible"]

arr.map(replaceInArray);
// returns => ["Summer+is+Great", "Winter+is+terrible"]

Your problem was that you were only replacing the first instance of " ". To fix this, use the global flag, by using g with your regex.

answered Mar 28, 2011 at 17:02

Mike Lewis

64.3k20 gold badges144 silver badges111 bronze badges

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

Comments

wildcard · Accepted Answer · 2011-03-28 17:00:12Z

1

you probably replace only first space found. to replace all of them, you'll need global flag. try .replace(/\ /g, "+");

answered Mar 28, 2011 at 17:00

wildcard

7,5253 gold badges30 silver badges25 bronze badges

3 Comments

Justin Over a year ago

Thanks for your response. I just tried that, but it only displays "undefined"

wildcard Over a year ago

well, the replace itself is fine then. your problem must be somewhere else - possibly tn_top_performers[tn_counter] is undefined?

Justin Over a year ago

I'm sorry, I'm pretty green at javascript. When I take out the replace line, the results are displayed fine (but still of course have spaces). So that would mean they're defined, right?

Oliver Moran · Accepted Answer · 2011-03-28 17:20:52Z

0

Your use of String.replace() is fine. The problem is that you are missing curly brackets surrounding all of the statements you want in the loop.

Fixed code:

var tn_dv_suggestions = new Array();
for (var tn_counter=0; tn_counter < tn_top_performers.length; tn_counter++) {
  tn_top_performers[tn_counter]=tn_top_performers[tn_counter].replace(" ","+");
  tn_dv_suggestions.push("<a style='font-family: Verdana, Arial; font-size: 14px;' target='_blank' href='http://www.<?=$siterow['Domain']?>/Buy-"+escape(tn_top_performers[tn_counter]) +"-<?=urlencode($siterow['CitySearchName'])?>-Tickets' >"+tn_top_performers[tn_counter] +"</a><br />");
}
document.getElementById('tn_dv_suggestions089hZ').innerHTML=tn_dv_suggestions.join('');

answered Mar 28, 2011 at 17:20

Oliver Moran

5,1874 gold badges34 silver badges45 bronze badges

1 Comment

Justin Over a year ago

That did it! Thanks so much for noticing that!

lucian.pantelimon · Accepted Answer · 2011-03-28 17:04:31Z

0

Tested on FF3 and Chrome.

tn_top_performers[tn_counter]=tn_top_performers[tn_counter].replace(/ /g,"+");

Edit: Don't forget the " "(space) between the forward slashes.

answered Mar 28, 2011 at 17:04

lucian.pantelimon

3,6774 gold badges31 silver badges46 bronze badges

Collectives™ on Stack Overflow

Javascript Replacing Spaces within an Array

4 Answers 4

Comments

3 Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

3 Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related