pregmatchで変数を使いつつ、正規表現も使いたい。

Question

$file = 'my_file_123456789';
$needle = 'my_file_'

$pattern = '/' . preg_quote($needle, '/') . '/';
if ( preg_match($pattern, $file) ){
	echo 'マッチします。';
}

こんな感じに書いています。

pregmatchのとこで、patterを使っていますがここで完全一致の判定を入れたいんですが変数との共存の書き方が分かりません。

'/' . '\A' . preg_quote($needle, '/') . '\z' . '/';

としてみましたがだめでした。

Comments

[maisumakun]

完全一致であれば、「preg_matchを使わずに、$needle === $fileで済ませる」という方法も考えられます。

preg_matchを使わなければならない理由はどのようなものでしょうか？

[momolength]

そうですね...。
いえ特に。正規表現はpregmatchという倣い的なものと、今後の拡張性も含めて使っていました。

[m.ts10806]

パターンケースをすべて洗い出して整理しないことには抜けが出ます。
できたと思ったらできてなくて同じ質問を繰り返すことになります。
質問前に要件を整理してください。

[maisumakun]

> 正規表現はpregmatchという倣い的なものと

使う道具は適宜選びましょう。不必要に機能の多い関数を使うのは、バグの要因ともなって好ましくありません。

[momolength]

>不必要に機能の多い関数を使うのは、バグの要因ともなって好ましくありません。
そうですね。ありがとうございます。

Answer 1

ワイルドカードも使わない完全一致は正規表現でやるべきではありません

$file = 'my_file_123456789';
$needle = 'my_file_123456789';
if($file===$needle){
print "完全一致";
}

Comments

[momolength]

なぜでしょう？

[maisumakun]

「実装の手間が増える」「余計な機能のある関数を使うことで、余計な動作をさせてしまう危険がある」「動作も遅くなる」と、何1つメリットがありません。

[m.ts10806]

つまり、練習目的以外で積極採用するものではないです。

[momolength]

> yambejp maisumakun
>余計な機能のある関数を使うことで、余計な動作をさせてしまう危険がある
納得です。シンプルなもので代用できるならそうした方がよいということですね。ありがとうございます。
ただ、"この場合の書き方"の知識が欲しかったんです。他の書き方を知りたかったんですが、今回はこれにしておきます。

[maisumakun]

> ただ、"この場合の書き方"の知識が欲しかったんです。

でしたら、質問段階でわかるようにしておいてください（実際に、「他の関数のオプションとして指定する関係上、正規表現がいちばん楽」ということもありえます）。

特に条件がなければ、コード全体の枠組みごと見直す回答になる場合もありえます。

[momolength]

>でしたら、質問段階でわかるようにしておいてください
>共存の書き方が分かりません。
と書いているので、その答えが欲しかったです。
次はわかり易く質問します。ありがとうございました。

[maisumakun]

> 変数との共存の書き方が分かりません。

ただ単に===で比較する場合も、元の変数をそのまま使えばいいだけなので問題ないと判断しました。

Answer 2

それであっていると思います。手元では期待どおり動きました。
「うまくいかない」ことを再現する、実行可能なスクリプトを示してください。

以下でうまく動いているように見えます。ちょっとだけ変えました。

<?php
$file = 'my_file_123456789';
$needle = 'my_file_123456789';

$pattern = '/\A' . preg_quote($needle, '/') . '\z/';
if ( preg_match($pattern, $file) ){
    echo 'マッチします。';
}

Comments

[momolength]

一緒にできたんですね。良く理解できてなかったです、ありがとうございます。
ただ、他の方の意見も参考にしてそちらを使わせていただくことになりました。