1

Нужно заменить все вхождения [br] в заголовках h1 на тег </br>. Пишу следующий код:

jQuery('h1').each(function() {
    var text = jQuery(this).text();
    text.replace(/\[br]/g, '</br>');
    jQuery(this).text(text);
});

https://jsfiddle.net/cjq263ex/

Работать не хочет, хотя регулярное выражение, вроде, правильное.

2 ответа 2

3

replace возвращает новую строку, не меняя текущую.

Этот метод не изменяет объект String, на котором он вызывается. Он просто возвращает новую строку.

нужно

jQuery(this).text(text.replace(/\[br\]/g, '</br>'));

как заметил @walik, если нужно вставить html вместо функции text нужно использовать html

jQuery(this).html(text.replace(/\[br\]/g, '</br>'))
2
  • И самое важное забыли, метод .text() выведет html "как он есть", нужно использовать .html() для того что бы все сработало.
    – walik
    Commented 17 нояб. 2015 в 13:09
  • @walik, добавил замечание :-)
    – Grundy
    Commented 17 нояб. 2015 в 13:12
2

В вашем коде есть ряд проблем:

  1. Вы не правильно осуществляете замену. Метод String.prototype.replace возвращает обновленную строку, а не проводит замены в текущей.

  2. Метод jQuery.text экранирует все теги из переданной строки, заменяя управляющие символы HTML-сущностями. Вам нужно использовать метод jQuery.html

  3. Тег br имеет вид <br /> а не </br> (обратите внимание на положение слеша). Ваш вариант не соответствует стандартам, хотя и должен работать в браузерах.

Таким образом, правильный код имеет вид:

jQuery('h1').each(function() {
    var text = jQuery(this).text();
    text = text.replace(/\[br]/g, '<br />');
    jQuery(this).html(text);
});

JSFiddle с работающим примером.

5
  • 1
    вообще говоря я проверил с текущим выражением - тоже работает.
    – Grundy
    Commented 17 нояб. 2015 в 13:12
  • @Grundy, как бы то ни было, управляющие символы лучше эскейпировать. Ну и это не единственная проблема в коде ТС Commented 17 нояб. 2015 в 13:13
  • Метод .text() ничего не вырезает он экранирует символы кода HTML.
    – walik
    Commented 17 нояб. 2015 в 13:14
  • @walik, вы правы, обновил ответ Commented 17 нояб. 2015 в 13:15
  • Символ ] нужно экранировать только в символьном классе. Здесь не нужно этого делать. Экранировать надо только то, что необходимо экранировать. Commented 17 нояб. 2015 в 20:45

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.