この正規表現の検索/置換は最適化できますか？

本当に簡単なことですが、HTMLスニペットの中のphp定数でタグを見つけて置き換える正規表現を書いています。私の解決策は、うまくいきません。どのように改善することができますか？この正規表現の検索/置換は最適化できますか？

preg_match_all('/\{CONSTANT_(.*)\}/', $final, $result, PREG_PATTERN_ORDER); 
      for ($i = 0; $i < count($result[1]); $i++) { 
       $final = str_replace($result[0][$i], constant($result[1][$i]),$final); 
      }

出典

2009-06-12 robjmills

'？（） 'に'？'が足りないのですか？（すなわち '（。*？）\}'でなければなりません）、またはU/PCRE_UNGREEDY修飾子を使用してください（http://www.php.net/manual/en/regexp.reference.php#regexp.reference.internal-optionsを参照）私はあなたが貪欲な検索をしていると思うので、 '' {CONSTANT_c1} {CONSTANT_c2} ''があれば、 '。*）'の捕捉グループは '' c1} {CONSTANT_c2 "'と一致します。 –

を、これは持っていますより長いマッチが可能な場合、それを食べないようにする効果。 ([^}]*)と同じ効果が得られるかもしれません。([a-zA-Z0-9_]+)

出典

2009-06-12 08:55:03

ええ、それは私が実際に必要なもののように見えます。よりエレガントで効率的なようです – robjmills

"[^}]}"は "（。*？）\}よりも優れています" – Tomalak

私は車輪の再発明に対して常にだ：あなたは（PHPがすでにある）テンプレートエンジンのいくつかの並べ替えが必要な場合Smartyを見てみましょう。

出典

2009-06-12 08:52:40 Keeper

これは当てはまりますが、Smartyを実装することは、この使用法のために残忍なように思えますし、とにかくそれ自身の固有のオーバーヘッドが付属するでしょう – robjmills

何

mixed preg_replace (mixed $pattern , mixed $replacement , mixed $subject [, int $limit= -1 [, int &$count ]])

は？

出典

2009-06-12 08:54:37 miku

正規表現で ".*"を使用する場合は、見つかったものすべてに貪欲に対応するように注意してください。たとえば、次の行に：

{CONSTANT_ONE} blah {CONSTANT_TWO}

上記の正規表現は、文字列をキャプチャします「ONE} blah {CONSTANT_TWO」

は、あなたの代わりに文字クラスを使用することができます。「}」文字以外のものと一致する：あなたは、私が.*?と.*非貪欲を作ったpreg_replace_callback

function getConstant($matches) 
{ 
    return constant($matches[1]); 
} 
$final=preg_replace_callback(
      '/\{CONSTANT_(.*?)\}/', 
      "getConstant", 
      $final);

注意と1本のヒットでそれをすべて行うことができます

/\{CONSTANT_([^}]*)\}/

出典

2009-06-12 08:54:43

良い点！ヘッドアップありがとう – robjmills

確かに、プレグでただ欲張りな演算子を使うことができます。。*？ –

この正規表現の検索/置換は最適化できますか？

答えて

関連する問題