if文を使うほうが良い気がします。(PHP)

Question

PHPを始めたばかりの者です。
JavaScriptの経験は一年ほどあります。

インターネットで見つけたPHPのサンプルコードの一部分で、

if ( !empty($var) ) {
    // code
}

というソースコードを見かけたのですが、ifとemptyは判定結果が対になるので、

if( $var ) {
    // code
}

の形式でもいい気がするのですが、あえて!emptyにする必要はあるのでしょうか？

どちらの方法で記述してもエラーが発生するといったような致命的なものではないのですが、定石や慣習等のように意味のある構文なのかとふと疑問に思い質問しました。

よろしくお願いします。

Answer 1

どちらの方法で記述してもエラーが発生するといったような致命的なものではない

間違っています。
PHP 型の比較表

注意:

$xが定義されていない状態で単に if ($x)としてしまうとE_NOTICE レベルのエラーが発行てしまいます。代わりに、empty()や isset()を使うかあるいは変数を初期化するように してください。

Comments

[showggg]

ありがとうございます！
変数に何か値が予め代入されていれば、ifでも!emptyでも挙動は変わらないのでしょうか？

[退会済みユーザー]

一般的に、当初の質問のコードは「変数に何か値が入っているかどうか」を確認するために使用されるので、コメントの質問に意義を感じません。
また、コメントの質問の回答は、リンク先を読めば理解できる範囲のモノなので、大変残念に思います。

[showggg]

すいません、そういう意味があったのですね..!
リンク先はこの質問自体をする前にも確認してはいたのですが、単純に(意味がないと思い込んでいた)emptyは必要性がわからず、短く記述できるif文のほうが有用性があると感じていました。
思えばemptyの使い方も調べれば出てきたと思うので、この質問した自分自身に欠陥があったと思います。
大変勉強になりました！ありがとうございます！

Answer 2

$varがbooleanだったら
if($var)はtrue/falseを見てるのか空を判定してるのか分からない。

「動かした結果としてたまたま期待通りに動く」と「コードの意図が明確」はかなり違う。
if($var)が許されたのは昔のゆるふわなPHPの頃だけ。
今はemptyでコードの意図を明確にしたほうがいい。
if($var)と書かれてたら今はbooleanと認識することが多い。

本来はifの反対はunlessだけどPHPにはないのでif(!とするしかない。
素のPHPにはないのでフレームワークレベルでfilled()やblank()を作ったり
https://readouble.com/laravel/8.x/ja/helpers.html#method-filled
@unlessを作ったりしている。
https://readouble.com/laravel/8.x/ja/blade.html#if-statements
if(!が分かりにくいことは当然分かっているので対策してる。

if(filled($var)) {

}

Comments

[Zuishin]

if (!empty($var)) は、$var が false のときどうなりますか？