How to use String.replace() to replace ALL occurrences of a pattern

Question

In many divs I have to change the class name by replacing spaces with periods....

So I tried this

$(jqueryElements).each(function(index)
{ 
    jqueryElements[index].className.replace(' ','.');
});

When the class has two words it works fine... But when the class name has 3 or more words it fails....

className='one word';
jqueryElement[index].className.replace(' ','.'); // console-> one.word
className='many many words';
jqueryElement[index].className.replace(' ','.'); // console-> many.many words

There is something wrong??

I'm using Chrome 25, Win7, jQuery 1.8

EDIT 2

I need to replace spaces to search all the span elements that have a particular class name.

So I use jquery in this way...

$('#span-container').find('span.'+jqueryElements[index].className.replace(' ','.').text('there are '+span_counter+'spans with this classname');

The result of this request shoud be:

 $('#span-container').find('span.many.many.words).text('there are '+span_counter+'spans with this classname');

Instead I have:

$('#span-container').find('span.many.many words).text('there are '+span_counter+'spans with this classname');

err, why are you doing this? spaces separate class names; combining them doesn't make any sense. — Eevee
– Eevee, Commented Feb 28, 2013 at 1:43

Barmar · Accepted Answer · 2013-02-28 01:54:58Z

3

Don't use replace, use split and join:

jqueryElements[index].className.split(' ').join('.');

answered Feb 28, 2013 at 1:54

Barmar

789k57 gold badges555 silver badges669 bronze badges

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

2 Comments

andrescabana86 Over a year ago

i agree with this... in this jsperf.com/split-vs-replace test case the split().join() function has the best performance in all browsers thank you for the answer

andrescabana86 Over a year ago

i make a new test case, see the jsperf.com/split-vs-replace/2 and give me an opinion of this... thank you

Explosion Pills · Accepted Answer · 2013-02-28 01:43:52Z

3

Use .replace(/ /g, '.') instead. The g means global (as in "global replace"). I am a bit skiptical about what you're trying to do though.

answered Feb 28, 2013 at 1:43

Explosion Pills

192k56 gold badges341 silver badges417 bronze badges

5 Comments

andrescabana86 Over a year ago

global means all chars??? including '-' or '_'?? because i want to replace only spaces, white spaces...

jahroy Over a year ago

Global means "perform this substitution multiple times on each piece of input".

andrescabana86 Over a year ago

great! this works fine! thank you, im not sure if i have to erase the question... because is simple question, but in all tutorials the answer is ".replace(' ','.'), this is a different aproach...

jahroy Over a year ago

No need to remove the question. I recommend doing some reading on JavaScript regex, though. Here's a good place to start

andrescabana86 Over a year ago

i make a new test case, see the jsperf.com/split-vs-replace/2 and give me an opinion of this... thank you –

Collectives™ on Stack Overflow

How to use String.replace() to replace ALL occurrences of a pattern

2 Answers 2

2 Comments

5 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

5 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related