如何替换ap标签内的所有“/”

Question

I would like to replace all forward slashes within ap tag but I cannot see what I am doing wrong.

 $(document).ready(function() { var content = $("p").text(); content.replace(/\\//g, 'ForwardSlash'); }); 

 <script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script> <p>YOYOYO / yoyoyoy / YIYYIYIYI ddwqd w/ dwdwdw/ dwd / dwd /</p> <div>YOYOYO / yoyoyoy / YIYYIYIYI ddwqd w/ dwdwdw/ dwd / dwd /</div> 

The reason that I want to do this is because a site I am making requires all the forward slashes to be a different colour to the text within the tag and as I am building the site within a CMS I am trying to make it easy to edit. The crude way of doing this would be to make the client paste spans with class names or inline styles. Hopefully I can do one of those things for him automatically using your help and the code above.

Answer 1

You were on the right track. But, know that replace() does not take a string with a reference and modify it. Rather, it returns a result and you will have to use that result

 $(document).ready(function() { $('p').each(function() { var content = $(this).text(); content = content.replace(/\\//g, 'ForwardSlash'); $(this).text(content); }); }); 

 <script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script> <p>YOYOYO / yoyoyoy / YIYYIYIYI ddwqd w/ dwdwdw/ dwd / dwd /</p> <p>YOYOYO / yoyoyoy / YIYYIYIYI ddwqd w/ dwdwdw/ dwd / dwd /</p> 

Answer 2

You forgot to apply the result back to the p (and to the content variable, as content itself does not get modified, it returns the modified string , so you would have to reassign the result to content if you want to do it in multiple steps).

Also, as @dman2306 mentioned, it might be better to use html() instead of text() as any other nodes are stripped when using text, including <em> and <strong> tags, which are often used inside <p> tags. Although this would require a regex pattern to not remove any slashes that were preceded by a < ...

 $(document).ready(function() { var content = $("p").text().replace(/\\//g, 'ForwardSlash'); $("p").text(content); }); 

 <script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script> <p>YOYOYO / yoyoyoy / YIYYIYIYI ddwqd w/ dwdwdw/ dwd / dwd /</p> <div>YOYOYO / yoyoyoy / YIYYIYIYI ddwqd w/ dwdwdw/ dwd / dwd /</div>