正規表現のネストがうまくいかない

Question

独自形式の文字列を、htmlタグに変換する処理を行っています。

例えば、以下の文字列に対し、

$str = "{img:sample1.jpg}";

以下の処理を行い、

while( preg_match('/{([^{}]+)}/', $str, $match) ){
    list($key, $p1, $p2) = explode(':', $match[1]);
    switch($key){
        case 'img':
            $v = '<img src="'.$p1.'">';
            $str = str_replace($match[0], $v, $str);
            break;
        case 'tr':
            $v = '<tr><th>'.$p1.'</th><td>'.$p2.'</td></tr>';
            $str = str_replace($match[0], $v, $str);
            break;
    }
}

これは、以下のように期待通りに変換されます。

<img src="sample.jpg">

一応、{} をネストできるようにしたつもりですので

$str = '<table>'
    . '{tr:サンプル１:{img:sample1.jpg}}'
    . '{tr:サンプル２:{img:sample2.jpg}}'
    . '</table>';

を処理すれば、以下のようになることを期待していますが、

<table>
<tr><th>サンプル１</th><td><img src="sample1.jpg"></td></tr>
<tr><th>サンプル２</th><td><img src="sample2.jpg"></td></tr>
</table>

以下のようになってしまいます。

<table>
<tr></tr>
<tr></tr>
</table>

どこが間違っているのかがわからず困っています。
間違っている点をご指摘いただければありがたいです。

Comments

[m.ts10806]

の前に
> ：
そもそもこのコード、ポツンとある↑でエラー出てますよ。

[ippya]

ごめんなさい。
「他にも同じような処理がある」という意味で、省略したつもりでした。
（途中省略）に変更します。

[m.ts10806]

エラーで困っていないのでしたらエラーが出ないコードをご提示ください。
あまり省略されすぎると再現確認できないので、半端に省略するくらいなら改めて再現できる最小構成のコードを組んでご提示ください。

[ippya]

省略をなくしました。

[otn]

> 一応、{} をネストできるようにしたつもりですので
ネスト化前と間違えてコピペしていませんか？

[otn]

ああ、失礼しました。正規表現で再帰をやるんじゃなくて、ループで再帰を処理しているんですね。
PHP 7.3.12 ではうまくいきました。

[ippya]

PHP 7.3.12 ではうまくいくんですね。検証ありがとうございました。＾＾
こちらは PHP 7.3.9 までしか利用できない環境です。
やはり PHPのバージョンの問題なんですかね。

[mix-peach]

7.2.17と、7.3.8で動作確認をしましたが、どちらも期待通りの正しい結果になりましたので、バージョン差異に問題はなく、提示されたコード以外に原因があるのかもしれないなと思いました。

お手元で現象が確認できるコードを分割せずに、「独自形式の文字列の定義」「正規表現によるHTMLへの置換処理」「画面に出力」までの通したコードでご提示いただけると良いかもしれません。

[yambejp]

正規表現うんぬんよりも、jsonなどもっと効率的なデータの持ち方にすれば
無駄に悩む必要はないと思いますが・・・

[otn]

あ、もっと古いバージョン(5とか)なのかと思っていました。mix-peachさんの通り、バージョンの問題ではなさそうです。

[ippya]

提示させていただいた内容を、そのまま実行したら、皆さんおっしゃるように、期待通りの結果になりました。
元々のコードは、定義部分を json 化しており、変換処理を汎用化しようとして、かなり長文化しているため、全文を掲載するには無理がありますので、要点を提示したつもりでしたが、その部分には問題ではなかったようです。
他の部分を見直します。
みなさん、お騒がせしました。申し訳ありません。

[otn]

長いコードは、抜粋して、実行して、解決したい現象が出ることを確認してから、コピペしましょう。

[kyoya0819]

ippyaさん
解決しましたら下のフォームに解決方法を入力いただき自己解決とすることで同じ悩みを待っている方が迅速に的確に解を得られやすくなります。
よろしくお願いいたします。

Answer 1

表題に「正規表現のネストがうまくいかない」と書きましたが、
正規表現とは関係ない、私の ミス による勘違いでした。

$str = '<table>'
    . '{tr:サンプル１;{img:sample1.jpg:200px:150px}}'
    . '{tr:サンプル２;{img:sample2.jpg}}'
    . '</table>';

while( preg_match('/{([^{}]+)}/', $str, $match) ){
    list($key, $other) = explode(':', $match[1], 2);
    $v = '';
    switch($key){
        case 'img':
            list($src, $w, $h) = explode(':', $other);
            $s = [];
            if( $w!="" ) $s[] = 'width:'.$w; // *****ミス*****
            if( $h!="" ) $s[] = 'height:'.$h; // *****ミス*****
            $style = (count($s)>0) ? ' style="'.implode('; ', $s).'"' : '';
            $v = '<img src="'.$src.'"'.$style.'>';
            break;
        case 'tr':
            list($text, $cls) = explode(':', $other);
            $d = explode(';', $text);
            $cols = '';
            foreach( $d, $v1 ){
                $cols .= '<td>'.$v1.'</td>';
            }
            $v = '<tr>'.$cols.'</tr>';
            break;
    }
    $str = str_replace($match[0], $v, $str);
}

変換対象の文字列を「 : 」で分割して処理をしていたのに、変換後の内容にも「 : 」を含めてしまい、意図しない結果になっていました。お騒がせしました。