Question

У меня есть немного регулярных выражений в PHP, которые не делают то, что я хочу.

Что я хочу:

[segment type="segment_name"]more text here[/segment] 

быть изменен на:

[segmentclass='segment_name segment']more text here[/segment] 

и:

[segment]more text here[/segment] 

быть изменен на:

[segmentclass=' segment']more text here[/segment] 

Вот что у меня есть:

$pattern = '/(\[segment.*?)(type=\"(.+?)\")?(.*?\[\/segment\])/is'; 
$replacement = '$1class=\'$3 segment\'$4'; 
$content = preg_replace($pattern, $replacement, $content); 

Кто-нибудь есть какие-нибудь идеи? ? after (type = \ "(. +?) \") состоит в том, чтобы сделать этот блок необязательным. Кажется, что замена просто пропускает его, потому что это необязательно. Я думаю, что у меня есть проблема с жадностью где-то, но я не смог разобраться.

Ответьте, если у вас есть идеи по-другому, я тоже мог бы это сделать.

Answer 1

У вас нет закрывающего кронштейна ] для блока первого сегмента. Поскольку он не находит соответствия для этого закрывающего блока, .*? соответствует всем [/segment], игнорируя необязательную группу, которую вы там положили.

Вам следует избегать использования .* в любой форме, если это возможно, и это классическая ситуация, когда класс отрицательных символов был бы approriate. [^\]]* - лучший выбор.

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

Вход, который вы указали в своем вопросе, а также ваше конкретное соответствие segment заставляет меня поверить, что ваше регулярное выражение слишком сложно, а замена очень грязная. Этому обновленному регулярному выражению также не нужен флаг DOTALL.

$pattern = '/\[segment(?: type="([^"]+)")?\]/i'; 
$replacement = '[segmentclass=\'$1 segment\']';