日頃 CakePHP 関連のリポジトリーに送られてくるプルリクエストを眺めていると、実に多くの方が empty() を安易に使用しているのが目に留まります。実例をご覧ください。 ですが、空かどうかを検査するために empty() を使用するのは適切ではありません。代わりに ! 演算子の使用を推奨します。 !empty() の代わりには、何もせずに真偽コンテキストに渡すか (bool) による型変換が良いでしょう。これは CakePHP のコーディング規約にも書かれています。 empty() は検査対象が未定義であっても E_NOTICE レベルのエラーを発生させません。これは検査したい変数がひょっとすると未定義かもしれない場合には便利ですが、それ以外の場合には変数名の打ち間違いを見落としてしまう原因になりかねません。 もっとも、開発に IDE を利用している方は、自分がそんなミスをするはず
