Question

Using .each() to go through an array, and trying to use replace() to find all the “<br>” and replace with ” ” (pretty simple) but nothing happens – no errors, it just doesn’t replace.

//get values of each td in row
var values = '';
var tds = $(this).find('td');

$.each(tds, function(index, tditem) {
    values = values + "td" + ':' + tditem.innerHTML + '\n';
    values = values.replace(/<br>/i, " ");
});

Acknowledging that it would be better to manipulate a DOM element (and selecting one of those answers as the correct answer) I ended up adding this to my function as it just makes my life easier in the short-term:

values = values.replace(/<br>/i, " ");

Thanks @all

Answer 1

You don’t need to do string processing to achieve this result. Here is the best (right) way to do that:

$(this).find('td br').remove();

The general rule is: always use the DOM to reach and manipulate the elements, avoid string analysis for this kind of operation.

---

Edit:

In case you don’t want to change the real DOM elements, just clone then to a variable like so:

var HTML = '';
var tds = $(this).find('td').clone();
$(tds).find('br').remove();
HTML = $(tds).html();
alert(HTML);