「isset」かつ「!empty」を関数化しておきたい

Question

issetと!emptyは一緒に書くことが結構多いと思います。
これを関数化しておこうと思いました。

// このようにいつも一緒に書いているので関数化したい
if ( isset($v) && !empty($v) ) {
  echo 'issetで、かつ、emptyではない';
}

関数化したのがこちらですが、なぜかエラーです。

function isset_noempty($v){
    return isset($v) && !empty($v);
}
var_dump( isset_noempty($nodata) ); // -> Undefined variable: nodata

何がいけないのかわからず、三項演算子をやめてみたり、issetだけにしたりしてもダメでした。

// 三項演算子をやめてみた
function isset_noempty($v){
    return ( isset($v) && !empty($v) ) ? true : false;
}

// issetだけにしてみた
function isset_noempty($v){
    return isset($v);
}

しかし、issetの実行はできましたし、$nodataを宣言しておけばできました。

// issetはできた
var_dump( isset($nodata) );

// $nodataを宣言したらできた
$nodata = '';
var_dump( isset_noempty($nodata) );

原因と対策を知りたいです。
宜しくお願い致します。

Answer 1

まったく同じ動作をさせるのは無理です。

issetやemptyは、変数が宣言されていなくても警告を出さないという特殊な実装になっていますが、これはPHP組み込みのものなので、PHP内の関数で実装することはできません（渡そうとした時点で変数が未定義なので、E_NOTICEが発生します）。

Comments

[busi]

そうなんですね。みなさんいつも一緒に書いているのでしょうか？

[退会済みユーザー]

魔法の記号`@`があるかと。
function tmp($a){
var_export(isset($a));
}

tmp(@$a);

まぁ、使わんけどｗ

[maisumakun]

というより、!empty($arg)がtrueであれば、issetも必ずtrueなので、!emptyだけで用は足りるはずです（未定義でもemptyは処理可能です）。

[maisumakun]

> みなさんいつも一緒に書いているのでしょうか？

一緒に書く必要自体がないです（上記コメントを参照）。

[busi]

失礼致しました。ありがとうございました。

Answer 2

要件次第ですがnull合体演算子で対応できそうに思いますが、如何でしょうか。

Comments

[busi]

新機能とは勉強になりました。ありがとうございました。

Answer 3

該当する notice は、関数に変数を引数として渡そうとした段階で発生しています。
原因も対策もそれが理解できれば、実装可能かと。

バリデーションの一部としての実装ではないかと推測しますが、フレームワークに実装されている標準バリデーションをのぞいてみると勉強になると思いますよ。

Comments

[busi]

実装可能というと何がですか？実装不可能だと他の方から伺ったのですが、可能だとしたらコードを知りたいです。

[退会済みユーザー]

フレームワークの実装を参考にするといいって回答してるんですけど？

[busi]

そうなんですね。