Почему не корректно работает регулярка?

Question

[Isaac Clark]

Здравствуйте, такой вопрос:
Есть регулярка, которая ищет в строке слово(hello) и если находит его, то обрамляет тегами стронг. Но почему то, она при нахождении hello, код захватывает и следующий символ.

function getStrongText(value) {
	var str = 'Hello world hello123',
		pattern = "([^{L}]|^)Hello([^{L}]|$)".replace(/{L}/g),
		regex = new RegExp(pattern, "gi"),		
		text = str.replace(regex, "<strong>$&</strong>");
		
	console.log(text); /* <strong>Hello </strong>world<strong> hello1</strong>23 - Как видите в последний hello попала цифра 1*/

}	
getStrongText();

Подскажите, почему так?

Comments

[Сергей Сергей]

че за .replace(/{L}/g)?

[Сергей Сергей]

в шаблоне Хелло обернуть в круглые скобки,
в замещении написать "$1$2$3.

[Isaac Clark]

Сергей: всмысле в замещении? вы имеете в виду так? text = str.replace(regex, "$1$2$3");
так не работает вообще

[Сергей Сергей]

Isaac Clark: парсер тостера съел тэги.
именно это замещение, да.
"$1<strong>$2<\/strong>$3"
$2 нужно обрамить тэгами, но после того, как хелло окажется в скобках.

Answer 1

[Pretor DH]

Используй онлайн тестер: Например.
И старайся упрощать... а не усложнять.

function getStrongText(value) {
  var str = 'Hello world hello123',
  regex = new RegExp(value, "gi");
  return str.replace(regex, '<strong>$&</strong>');
}	
console.log(getStrongText('Hello'));

Если нужно искать только слова, а не части слов то нужно делать слегка иначе.

function getStrongText(value) {
  var str = 'Hello, world hello123',
    pattern = "(\s|^)"+value+"(\W|\s|$)",
    regex = new RegExp(pattern, "gi");
    return str.replace(regex, "$1<strong>$2</strong>$3");   
}	
console.log( getStrongText('hello') );

В твоем примере замена производилась трёх значений. $& значит все найденные субпатерны в одной итерации, а у тебя три субпатерна.

<strong>$&</strong> все равно, что <strong>$1$2$3</strong>

Comments

[Isaac Clark]

супер, спасибо, а подскажите пожалуйста, если я хочу передать не строку а переменную в качестве параметру, то в паттерн, то как мне это сделать?
вот в таком варианте я так понимаю, что не правильно?
var value = "Hello";
text = str.replace(/([^]|^)(value)([^]|$)/gi, '$1$2$3');

[Pretor DH]

Смотри так как ты хотел?

[Isaac Clark]

Pretor DH: да, все верно, спасибо. Отметил решением. Подскажите, а если еще не учитывать пробелы в конце и в начале?

[Pretor DH]

Ворой вариант учитывает пробелы с двух сторон и знак препинания после искомой строки.

[Isaac Clark]

Pretor DH: а мне наоборот не нужно учитывать, но только в конце и в начале строки, в середине же учитывать нужно.

[Pretor DH]

Вначале и конце строки нужно убирать пробелы при замене, я тебя правильно понял?

[Isaac Clark]

извините, не так. Слово, которое мы ищем может иметь или не иметь один пробел с начала или с конца.

[Isaac Clark]

Pretor DH: то есть, все так же, то еще может быть пробел с начала или с конца, а может и не быть

[Pretor DH]

Hello OneHello, HelloTwo HelloEnd - Все hello дожны быть взяты в strong ???

[Isaac Clark]

Pretor DH: да

[Pretor DH]

Тoгда используй мой первый вариант... он все Hello обращает в strong.

[Isaac Clark]

Pretor DH: что-то не корректно работает, если я ищу в слове hello букву l, то он выделяет, только первую букву l , а вторую пропускает, а должен выделять обе l

[Isaac Clark]

Isaac Clark: а если буквы стоят не друг после друга, то все норм

[Pretor DH]

Скайп есть?

[Isaac Clark]

Pretor DH: уже поправил

[Isaac Clark]

Pretor DH: Здравствуйте, подскажите пожалуйста, проблема в том, что когда я пытаюсь искать символ ( то происходит синтаксическая ошибка .

[Isaac Clark]

сделал так
new RegExp(whatToLook.replace(/[[\]{}()*+?.\\|^$\-,\s]/g, "\\$&"), "gi"),

Answer 2

[Melkij]

при нахождении hello, код захватывает и следующий символ

И следующий и предыдущий.
Потому что у вас в регулярке так и написано:
([^{L}]|^) сохранить в подмаску не {L} или символ начала строки
И после ([^{L}]|$) - сохранить в другую подмаску не {L} или символ конца строки.
Здесь уместнее негативные ретроспективные и опережающие проверки. Но JS не умеет ретроспективную проверку.

Если нет желания читать "Регулярные выражения" Джеффри Фридла (хотя весьма рекомендую) то вот статья по позиционным проверкам: habrahabr.ru/post/159483

Comments

[Isaac Clark]

супер, спасибо за ссылку, а подскажите пожалуйста, как я могу передать переменную в патерн с текстом, вместо самого текста?

[Dmitriy Mozgovoy]

Isaac Clark: regexp = new RegExp("([^]|^)("+value+")([^]|$)", "gi")

[Melkij]

Добавлю к ответу Digital Brain, что если в value может попасть пользовательский ввод - то возможные спецсимволы регулярки надо будет экранировать. Штатного метода для этого, как обычно, в js не наблюдается, надо городить что-то вроде: stackoverflow.com/questions/3446170/escape-string-... да stackoverflow.com/questions/3561493/is-there-a-reg...

[Pretor DH]

А для чего вы огород городите за автором вопроса...
В вопросе достаточно простой глобальной замены:
regex = new RegExp(value, "gi");
str.replace(regex, '$&');

[Melkij]

Pretor DH: , мой хрустальный шар говорит, что автор хочет заменять слово hello, но не хочет заменять слова, в которые строка hello всего-навсего входит.

[Pretor DH]

Melkij: Тоже самое он получит и в своем варианте. А для того чтобы искать только слово нужно искать совсем ИНАЧЕ.

[Pretor DH]

Melkij: А именно патрен должен быть таким: (\s|^)(Hello)(\s|$)

[Melkij]

Pretor DH: во-первых, в \s не входят как минимум знаки препинания.
Во-вторых, в чём же это самое "совсем ИНАЧЕ"? Всё то же самое.
Вроде бы, JS умеет \b проверку. Надо выяснить только, что же считается за границу слова. Вроде учитывается только латиница.

[Pretor DH]

Melkij: Паттерн ([^]|^)(hello)([^]|$), применимо к нашему примеру равен просто hello. Но! Он излишне цепляет два суббпатерна, с чем у человека и били проблемы. Ибо они подключались в строку замены посредством $&, что равно в нашем примере $1$2$3.

[Pretor DH]

Melkij: А \b - точно не работает скирилицей...