関数 empty は罠
PHPの関数 empty はとても使いやすく入力値の検査などに使いやすいですがこれが罠なんです。
・PHP: empty - Manual
empty の使いやすさ
PHPマニュアルにある empty の説明はこれだけです。
empty — 変数が空であるかどうかを検査する
| |
出力結果
| |
このように変数が未定義、文字数0や null の場合に true を返します。
わざわざ、変数未定義チェック、null チェック、文字数チェックなどを行う必要がなくとても簡単にチェックをしてくれます。
これだけだと 入力値がないものをに対してtrue を返す関数だと思ってしまいます。
emptyの罠
今度は数値で確認してみます。
| |
出力結果
| |
何か挙動が怪しいです。0 で true (空)扱いになります。
emptyの罠の原因
empty 関数について、内部で以下のように動いているためです。
表を見ていると false にも true や空の配列に true を返したりします。
| 式 | empty() |
|---|---|
| $x = “”; | true |
| $x = null; | true |
| var $x; | true |
| $x が未定義 | NULL |
| $x = array(); | true |
| $x = array(‘a’, ‘b’); | false |
| $x = false; | true |
| $x = true; | false |
| $x = 1; | false |
| $x = 42; | false |
| $x = 0; | true |
| $x = -1; | false |
| $x = “1”; | false |
| $x = “0”; | true |
| $x = “-1”; | false |
| $x = “php”; | false |
| $x = “true”; | false |
| $x = “false”; | false |
対策
対策としては empty の表を理解して使用する。
もしくはちゃんと意図した動作を明確に書くことです。
nullチェックでは is_null、文字数チェックでは strlenなど行いたい処理に合わして明確に処理を書いてください。
個人的な見解ですが empty だけだと処理の意図が伝わりづらい事があるので出来るだけ処理を明確に書く方がメンテナンス性が高い気がします。