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

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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