Почему не работает preg_replace?

Question

[Александр]

Здравствуйте, уважаемые форумчане! Почему у меня не срабатывает замена на preg_replace?
Исходный текст:

[quotemsg data-username="Александр З"]Здравствуйте, Подскажите как можно в исходном коде статей расставить классы для заголовков H2? Статей много - вручную не вариант. По сути надо в цикле пройтись по всем материалам и при встрече заголовка, в зависимости от его порядка в тексте, дать ему класс. Например классы header1, header2, header13 Сказать легко - сделать не могу[/quotemsg]Отвечаю на сообщение с первой страницы

Код, как обрабатываю:

if (preg_match("~\[quotemsg data-username=\"(.*?)\"\].*?\[/quotemsg\]~",$text ,$a))
 {  return preg_replace("~\[quotemsg data-username=\"(.*?)\"\](.*?)\[/quotemsg\]~", '$2', $text);}

Но, по результату так и вижу исходный текст

Comments

[Дмитрий]

echo '<pre>' . print_r($a, true) . '</pre>';
в $a[1] будет ваше имя, оно же будет в $1

[Александр]

Дмитрий, Моя задача убрать bbcode и оставить только содержимое внутри него

[Дмитрий]

Александр, оно убирает

[Александр]

Дмитрий, Если бы все работало, я бы не стал обращаться за помощью. У меня на странице видна исходная строка, после обработки, а не только текст из bbcode ((

[Дмитрий]

Александр, полный код киньте вашей функции

Answer 1

[YepBro]

Если тебе надо оставить то что между [quotemsg][/quotemsg] то preg_replace работает корректно.

А зачем делать return из if?

$text = preg_replace("~\[quotemsg data-username=\"(.*?)\"\](.*?)\[/quotemsg\]~", '$2', $text);

https://www.php.net/manual/ru/function.preg-replace.php

Comments

[YepBro]

Если же это часть функции, то смотри preg_match . Скорее всего он возвращает 0 или false.

[Александр]

YepBro, Это часть функции, там еще есть другие замены.
Попробовал так:

$text = preg_replace("~\[quotemsg data-username=\"(.*?)\"\](.*?)\[/quotemsg\]~", '$2', $text);
  return $text;

По итогу, получаю исходный текст, как описано в созданной теме
P.S пробовал даже убрать другие замены, оставил только этот код, но по факту, ничего не изменилось

[YepBro]

Значит причина в чем-то другом. Я проверил, вырезало именно что что между [quotemsg data-username=""][/quotemsg]

Какая версия php?

[Александр]

YepBro, PHP Version 7.1.33
Вот исходная функция

function removeDataFromMsg($text) {
  $text = preg_replace("~\[img\](.*?)\[/img\]~", '[*фото-контент*]', $text);
  $text = preg_replace("~\[video title=\"(.*?)\"\](.*?)\[/video\]~", '[*видео-контент*]', $text);
  $text = preg_replace("~\[spoiler title=\"(.*?)\"\](.*?)\[/spoiler\]~", '[*спойлер*]', $text);
  $text = preg_replace("~\[quotemsg data-username=\"(.*?)\"\](.*?)\[/quotemsg\]~", '$2', $text);
  return $text;
 }

Вот результат на скриншоте

[Дмитрий]

Александр, а вывод покажите, может забыли обработать вашу форму вашей функцией, или она идет после записи в бд

[Александр]

Дмитрий, $Mh['msg'] = removeDataFromMsg($Mh['msg']);

<div style="font-weight: 500;"><a href="<?=FORUM_LINK?><?=$Mh['forumID']?>/<?=$Mh['themeID']?>/"><?=$Mh['msg']?></a></div>

[YepBro]

https://regex101.com/r/MaDMii/1 вот тут видно что регулярное выражение рабочее

[YepBro]

Александр, а для теста ты откуда брал строку? Из базы или вот с этой страницы где скриншот делал? Если второе, то в базе может быть что-то слегка другое: задвоение пробелов, спецсимолы.

[Александр]

YepBro, мистика какая-то. Я конечно знаю что в программировании нет мистики, а есть косяки, но тут... Я вообще уже ничего не понимаю. Другие замены отрабатывают же, а эта ни в какую

[Александр]

YepBro, я вывожу из базы. А сюда копировал с страницы, которая с скриншота.
Вот из базы:

[quotemsg data-username="Александр З"]Здравствуйте,

Подскажите как можно в исходном коде статей расставить классы для заголовков H2?
Статей много - вручную не вариант.

По сути надо в цикле пройтись по всем материалам и при встрече заголовка, в зависимости от его порядка в тексте, дать ему класс.
Например классы header1, header2, header13

Сказать легко - сделать не могу[/quotemsg]Отвечаю на сообщение с первой страницы

[Александр]

YepBro, Убрал текст внутри тега и заменил на латиницу - замена сработала.
Написал прямо в базе текст "Ерунда какая-то". Замена сработала. Вставил опять текст "Здравствуйте,

Подскажите как можно в исходном коде статей расставить классы для заголовков H2?
Статей много - вручную не вариант.

По сути надо в цикле пройтись по всем материалам и при встрече заголовка, в зависимости от его порядка в тексте, дать ему класс.
Например классы header1, header2, header13

Сказать легко - сделать не могу" - опять не работает!

[YepBro]

Александр, так это большая разница - одна строка или несколько с переводами строк. Поэтому в реальности и не работает. Надо дорабатывать регулярку.

[YepBro]

Александр, точка захватывает любой символ кроме перевода строки (\n)

Добавляешь в регулярку опцию s и все работает.

https://regex101.com/r/rL2aAM/1

~\[quotemsg data-username=\"(.*?)\"\](.*?)\[/quotemsg\]~s

[Александр]

YepBro, Дмитрий, спасибо огромное. Век живи - век учись!
Действительно стало нормально работать с той строкой. А вот эта

[quotemsg data-username="Александр З"][quotemsg data-username="Александр З"]Переходим на новую страницу .... [/quotemsg]А вот тут могут возникнуть проблемы ((([/quotemsg]Это да!

Не хочет. Точнее, срабатывает, но только один подобный тег убирает

[YepBro]

Александр, в онлайн редакторе вставил, все работает. То есть проблема опять не в регулярке.

[Александр]

YepBro, Поправил предыдущее сообщение. Не так написал

[YepBro]

Александр, Все верно, твоя регулярка только на один раз :) Доработай ее или переделай.

Ведь можно сделать что бы из текста будут удаляться (заменяться на пустую строку) вот эти строки [quotemsg data-username="Александр З"] и [/quotemsg].

Результат тот же самый, будет поддерживать любое кол-во уровней и регулярка будет проще. Можно даже на две регулярки разбить (для начала бб-кода цитаты и для конца). Тогда во втором случае str_replace достаточно.

[Александр]

YepBro, Спасибо, надо подумать. Я просто с регулярками не очень дружу (((

Answer 2

[psiklop]

Плохой код.
Регулярное выражение выполняется два раза - это излишне.

Comments

[Александр]

Посмотрите в обсуждении выше. Код поправил, но проблема пока не ушла

[psiklop]

Александр, а так работает ?

$text = preg_replace("/\[quotemsg data-username=\".*\"\](.*?)\[\/quotemsg\]/u", '$1', $text);

[Александр]

psiklop, ерунда какая-то. Всё тоже самое (((

[psiklop]

Александр, у меня работает. А текст кириллицей в какой кодировке ? Используйте iconv в utf-8 если нужно. Но эта тема не гарантирую, что стабильная, у меня при парсинге всякое бывало.

[Александр]

psiklop, в базе msg text utf8_general_ci
Сам файл обработки в кодировке utf-8

[Александр]

Александр, Убрал текст внутри тега и заменил на латиницу - замена сработала.
Написал прямо в базе текст "Ерунда какая-то". Замена сработала. Вставил опять текст "Здравствуйте,

Подскажите как можно в исходном коде статей расставить классы для заголовков H2?
Статей много - вручную не вариант.

По сути надо в цикле пройтись по всем материалам и при встрече заголовка, в зависимости от его порядка в тексте, дать ему класс.
Например классы header1, header2, header13

Сказать легко - сделать не могу" - опять не работает!

[psiklop]

Александр, тут важнее откуда приходит такая строка, врядли ли она впечатана в файл обработки. Если это парсинг она придет в своей кодировке, если она из вашей базы то как она туда попала, вообще база может быть в utf, но это не значит что она конвертит в utf при записи.

[psiklop]

Александр, у меня с впечатанным в файл этим текстом кириллицей в utf 8 работает.

[Александр]

psiklop, строка изначально создается пользователем в редакторе сообщения (bbcode wysibb), затем аяксом отправляется в БД

[psiklop]

Александр, по идее в js все строки в utf. Где-то баг. Надо искать или менять логику. В js удалять bb-code. Уверен в самом плагине есть нормальная функция без bb-code. Тут скорее всего надо саму логику оптимизировать. Фигня все эти регулярки зачем тут на них полагаться.