PHPの不等式に変数を使う

質問

いつもお世話になります

PHPの不等式に変数を使うことは出来ますでしょうか？
例えば

$a = 'huga';
$b = 'huga';
if($a == $b){
echo 'hoge';
}

結果
hoge

右辺と左辺の間、不等式を変数にしたいです

$a = 'huga';
$b = 'huga';
$c = '=='

if($a $c $b){
echo 'hoge';
}

期待する結果
hoge

上のソースはもちろんパースエラーとなり成り立ちません
型変換や考えられるキーワードで検索したのですが、出てきませんでした

下のように
strpos()、strstr（）等で$cの内容を解析して、条件分岐すれば期待の結果が出せるのは分かります

if( strpos($c,'==') !== false && $a == $b){
echo 'hoge';
}

(上記で成り立つのは分かります)
教えていただきたい内容は、「不等式を変数で成り立たせる方法」です
$c= '==';

PHPに詳しい方お教えいただけると嬉しいです

回答No.5

notnot · ベストアンサー

evalの対象文字列が、すべてプログラム中のリテラルなら問題ないですが、それだとevalを使う意味が無い。そのまま書けば良いので。

evalの対象文字列の一部に、ファイルから読んだ文字列とか、ウェブで送信された文字列とかを含むならば、それらをevalしても安全かどうかを解析しないと、むやみにevalすると何が起こるかわかりません。いわゆる、脆弱性（攻撃されるとファイルを消されたり情報を盗まれたりする）です。

そこまで解析する手間を掛けてevalするくらいなら、質問文中にお書きのように $c を調べてif文で分岐するなどする方がずっと簡単です。

お礼

なるほどですね、納得です！
evalの関数自体には問題がないことが分かったので安心しました。

1.$cはリテラルでないのではないかと仮定した
2.上の場合evalを使うのは乱暴である

初めに回答下さった方は、1番の説明を省略なさっていたので、僕には2番の意味が伝わらなかったのですね。

一応、preg_matchで通ったものを処理しますので
該当部分を悪意のあるユーザーに操作されることはないかと思います。
正規表現でやるか、全条件で分岐するか、あとは好みの問題ですかね。

回答No.4

kmee

・毎度構文解析、コンパイルするのでパフォーマンスが落ちる。
文法間違いも実行時でないと判断できない。

・悪意あるコマンドを実行されてしまう脆弱性がある。

・なにをしているのかがよくわからない。
常に「eval」なので、関数名から直感的に連想しにくい。

eval はどうしても代替手段が無い時にだけにしておいた方がよいです。

例えば、関数compareでも作ってしまえば (中身は
> if( strpos($c,'==') !== false && $a == $b){
とかでも構わない)

> if(compare($a,$c, $b)){
> echo 'hoge';
> }

とevalよりもはるかに直感的で安全なプログラムが書けます。
$cを()の外に出せる無名関数やクラスだと、より「二つの比較」であることがわかりやすくなります。

お礼

回答くださりありがとうございます
notnot様の回答により、
なぜevalが否定されたのかが分かりました！

kmee様が挙げてくださいました３つの条件で
2番、3番は僕には当てはまらないようです

僕にとって影響があるのは1番の「パフォーマンスが落ちる」
これが事実の場合はevalを使うのは控えなければなりません

evalは特に遅くないという実験結果はあったのですが
http://www.softel.co.jp/blogs/tech/archives/84

パフォーマンスが落ちる説明がされている実験結果などはございますでしょうか

回答No.3

himajin100000

まぁほかのコードを変えずに$cの内容に応じて処理を変えたい、ってだけのことなら
ふつうは、
http://ideone.com/Rk4ei
とか
http://ideone.com/iwMX6
とかやるわな。文字列で式を作らなくてもいいんなら
(匿名関数の対応がPHP 5.3.0以降なのでPHP 5.2.11のこのサイトでは実行できなかったけど)

お礼

わざわざコードまで書いていただきありがとうございます
私などの為に時間を割いて頂き深く感謝いたします。

圧倒的に簡単にコードが書けるeval()を教えていただいたのでそちらで行きたいと思います。

別の質問になってしまいますが、教えていただいたコードは、クラスや匿名関数を使っており、このようなソースは読むのが困難であると思うのが一般的だと思います。
本件で、圧倒的に理解しやすいコードが書けるevalを使わず
こちらを使ったのにはなにかメリットがあるからなのでしょうか。

回答No.2

yambejp

まず文字列の比較に「==」は使いません。
基本を理解しないと後で痛い目にあいます。

そのうえで何のために演算子を変数にしたいのかさっぱりわかりませんが
どうしてもというなら乱暴ですがevalを使えばなんとかなります

<?PHP
$a = 'huga';
$b = 'huga';
$c = '===';
if(eval('return $a '.$c.' $b;')){
echo 'hoge';
}
?>

お礼

ありがとうございます！
evalを使えばいんですね、正にこれが知りたかったです！

また、回答で「この方法は乱暴である」と付加情報をお教えいただきましたが
調べたところ、eval()は特に非推奨の関数にはなっていないようですが
具体的にどう乱暴になるのか、つっこんでお教えいただけるとありがたいです。

回答No.1

noname#158634

https://www.google.com/search?hl=ja&lr=lang_ja&ie=UTF-8&oe=UTF-8&q=php+eval&num=50

ただし、そういう処理を行わなくてはいけない状態になっている時点でそれはクソコードである、という自覚は持っていてください。

お礼

ありがとうございます。肝に銘じておきます。

これだけの少ない情報で「これはクソコードである」とお教えいただきました
上の回答者様もeval()を使うのは乱暴だと仰っています
eval()は特に非推奨の関数にはなっていないようですが
eval()を使うことにより「これはクソコードである」というネガティブな判断が下されるのはなぜなのかもお教えいただけると幸いです。