phpのissetについて

Question

phpで返却値が未定義かどうか不定なものが多いため未定義な変数が来た場合
空に返却するような関数を作成しようと思うのですが、こう言った変換関数って調べても出てこないので
あまりよくないものなのでしょうか？
あと三項演算子を用いた記述は見づらいでしょうか。elseを使いたくないのでこの場合if文だと!issetみたいな記述になりそうで
可読性的にどうかなと思っております。

public function issetToBlank($request) {
   return isset($request) ? $request: '';
}

Answer 1

関数として実装すると、意図したようには動きません。というのも、未定義な変数を関数に引き渡そうとした時点で評価がなされてしまうので、そこでE_NOTICEを発生させてしまうからです。

その場でissetやempty、PHP7なら??などで書くしかありません（もちろん、E_NOTICEを気にしないならかまわないのですが、そのような書き方はおすすめしません）。

Comments

[enigumalu]

試しに実行してみましたが確かにNotice: Undefined index:
が発生していました完全に浅知恵でしたありがとうございます。
ソースが冗長的になりますがやはり個別に潰していくほかないのですね

[maisumakun]

配列での「Undefined index」であれば、「配列自体」と「キー」を分けて渡すことで、関数を呼ぶ時点での評価は避けることができます。

ただし、$_GETなどにはfilter_inputが、単なる配列にはfilter_var_arrayがありますので、自分で書く必要もないかもしれません。

Answer 2

phpで返却値が未定義かどうか不定なものが多いため未定義な変数が来た場合

空に返却するような関数を作成しようと思うのですが、こう言った変換関数って調べても出てこないので
あまりよくないものなのでしょうか？

エラーに気づきにくくなるので、利便性と天秤にかけて、状況に適した方法を選択するのが良いと思います。
絶対に避けるべき、ということはないです。

三項演算子については、書かれているような形で全く問題ないと思いますよ。

Answer 3

同じ処理を自分なら、以下のように書くと思います。

public function issetToBlank($request = '') {
   return $request;
}