PHPのeval内での関数使用がうまくいかない

Question

下記のように、連想配列から[ID]が1のものを抜き出す処理を書きました。

$dataarray = array_filter($dataarray, function($v) {
return preg_match('/^1$/', $v['ID']);
});

ここまで正常に動くことを確認したうえで、
return箇所にevalを使用したいと思い、下記のように書き換えたのですが、
[ID]が1のものがうまく取得出来ず、結果の配列が空になってしまいます。

どのように修正すれば良いでしょうか。

$dataarray = array_filter($dataarray, function($v) {
eval("return preg_match('/^1$/', $v['ID']);");
});

Answer 1

まずもって、設計段階で突っ込みどころ（不必要に事態を複雑化させていること）が満載です。

• 完全一致を判定するのにpreg_matchは不要です。(string)$v['ID'] === '1'で十分です。
• なぜ「return箇所にevalを使用したい」のでしょうか。「eval is evil」なんて言葉もあるように、それしか手段がないような場合を除いて、evalは使うべきではありません。
• evalの中でreturnすると、それは「evalを抜ける」という意味になるので、外側の関数はreturnしません。結果、array_filterのコールバックはどの値に対しても「返り値なし」となります。

Comments

[hiyokoman]

回答どうもありがとうございます。説明が足りなく申し訳ございません。

(1)については、実際のソースは完全一致・部分一致・後方一致・部分一致全て使用できるようにしたいと考えております。ですが、こちらでご相談するにあたり現象の再現が可能なよりシンプルなコードに書き換えておりますため、完全一致になっています。どうしてもeval内でpreg_matchを使用したく思っております。

(2)については、あくまで個人的な利用目的のプログラムであり、evalの中身は他者から書き換えられる事も無いため、evalを使用することにしております。

(3)ソースを下記のように書き換えましたが、症状変わらずです。
eval("$kekka = preg_match('/^1$/', $v['ID']);");
return $kekka;

[maisumakun]

「完全に固定の文字列でevalを行う」ことに全くメリットがないと思うのですが、それでも使いたい理由は何でしょうか。

[hiyokoman]

そちらも説明足らずで申し訳ございません。
(1)に記載しました通り、問題を明確化するため現象が再現可能なよりシンプルなコードに書き換えております。実際のソースは「完全に固定の文字列」ではありません。
preg_matchの箇所は他のファイルから取得するような作りになっています。あくまで個人使用のプログラムですので、セキュリティに拘りは無い物になります。よろしくお願い致します。

[hiyokoman]

またpreg_matchの引数の中身のみ他のファイルから取得するといった仕様にするのも構造上難しく、どうしてもこのような作りになった次第です。よろしくお願い致します。